diff options
author | Daniel Black <daniel@mariadb.org> | 2023-01-10 10:45:03 +1100 |
---|---|---|
committer | Andrew Hutchings <andrew@linuxjedi.co.uk> | 2023-01-10 17:10:43 +0000 |
commit | 56948ee54c9d113f07f725ebdc560d1919fc6676 (patch) | |
tree | ec9661fb5f822176475aca5590a194615c423f13 /storage/connect/zip.c | |
parent | d7f447915c96681c85abf00d22de589a5332a6da (diff) | |
download | mariadb-git-56948ee54c9d113f07f725ebdc560d1919fc6676.tar.gz |
clang15 warnings - unused vars and old prototypes
clang15 finally errors on old prototype definations.
Its also a lot fussier about variables that aren't used
as is the case a number of time with loop counters that
aren't examined.
RocksDB was complaining that its get_range function was
declared without the array length in ha_rocksdb.h. While
a constant is used rather than trying to import the
Rdb_key_def::INDEX_NUMBER_SIZE header (was causing a lot of
errors on the defination of other orders). If the constant
does change can be assured that the same compile warnings will
tell us of the error.
The ha_rocksdb::index_read_map_impl DBUG_EXECUTE_IF was similar
to the existing endless functions used in replication tests.
Its rather moot point as the rocksdb.force_shutdown test that
uses myrocks_busy_loop_on_row_read is currently disabled.
Diffstat (limited to 'storage/connect/zip.c')
-rw-r--r-- | storage/connect/zip.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/storage/connect/zip.c b/storage/connect/zip.c index 4bbe31ab7dd..d796fe09b87 100644 --- a/storage/connect/zip.c +++ b/storage/connect/zip.c @@ -1471,11 +1471,6 @@ extern int ZEXPORT zipWriteInFileInZip (zipFile file,const void* buf,unsigned in { uLong uTotalOutBefore = zi->ci.stream.total_out; err=deflate(&zi->ci.stream, Z_NO_FLUSH); - if(uTotalOutBefore > zi->ci.stream.total_out) - { - int bBreak = 0; - bBreak++; - } zi->ci.pos_in_buffered_data += (uInt)(zi->ci.stream.total_out - uTotalOutBefore) ; } |