mirror of
https://github.com/MariaDB/server.git
synced 2025-01-30 18:41:56 +01:00
3e32ba3ff1
Add some extra error output and code cleanup in an attempt to fix/debug a rare random testsuite problem in check_warnings, where the exit code from mysqltest is somehow corrupted inside mysql-test-run.pl. include/my_global.h: Fix compiler warnings on some platforms. mysql-test/lib/My/SafeProcess.pm: Move dereference of $? subprocess exit code closer to where it is generated, to make the code more robust and on the chance that this will fix the occasional problems in check_warnings we see in Buildbot. mysql-test/mysql-test-run.pl: When check_warnings failed, also log the mysqld server for which it failed. sql/sql_lex.cc: Fix compiler warning about possibly uninitialised value, by rewriting a for() loop that is always executed at least once into a do .. while() loop with an assert. sql/table.cc: Fix compiler warning about uninitialised value. storage/federatedx/ha_federatedx.cc: Fix uninitialised variable. storage/maria/ma_delete.c: Fix compiler warning about uninitialised value. storage/maria/ma_loghandler.c: Fix compiler warning about uninitialised value. storage/myisam/ft_stopwords.c: Fix compiler warning. storage/myisam/mi_write.c: Fix compiler warning about possibly uninitialised value, by rewriting a while() loop that is always executed at least once into a do .. while() loop with an assert. storage/xtradb/btr/btr0cur.c: Fix compiler warning about possibly uninitialised value. support-files/compiler_warnings.supp: Fix warning suppression to cover all cases in yassl. vio/viossl.c: Fix compiler warning.
140 lines
5.1 KiB
Text
140 lines
5.1 KiB
Text
#
|
||
# This file contains compiler warnings that can
|
||
# be ignored for various reasons.
|
||
#
|
||
|
||
integer.cpp: .*control reaches end of non-void function.*: 1288-1427
|
||
integer.cpp: .*no return statement in function returning non-void.*: 1288-1427
|
||
DictTabInfo.cpp : .*invalid access to non-static.*
|
||
DictTabInfo.cpp : .*macro was used incorrectly.*
|
||
DbdihMain.cpp : .*unused variable.* : 6666-7013
|
||
DbtupExecQuery.cpp : .*unused variable.* : 1823
|
||
kernel_types.h : .*only defines private constructors and has no friends.* : 51
|
||
Dbtup.hpp: .*only defines private constructors and has no friends.*
|
||
diskpage.hpp: .*only defines private constructors and has no friends.*
|
||
tuppage.hpp: .*only defines private constructors and has no friends.*
|
||
NdbScanOperation.cpp: .*unused variable '__align'.* : 1190-1200
|
||
sql_yacc.cc : .*label 'yyerrlab1' defined but not used.*
|
||
kernel_types.h : .*'struct Local_key' only defines private constructors and has no friends.*
|
||
lgman.hpp : .*'struct Lgman::Buffer_idx' only defines private constructors and has no friends.*
|
||
SchemaFile.hpp : .*'struct SchemaFile::TableEntry' only defines private constructors and has no friends.*
|
||
sql_yacc.cc : .*switch statement contains 'default' but no 'case' labels.*
|
||
|
||
#
|
||
# Things that can be ignored in InnoDB
|
||
#
|
||
pars0grm.tab.c: .*'yyerrorlab' : unreferenced label.*
|
||
_flex_tmp.c: .*not enough actual parameters for macro 'yywrap'.*
|
||
pars0lex.l: .*conversion from 'ulint' to 'int', possible loss of data.*
|
||
btr/btr0cur\.c: .*value computed is not used.*: 3175-3375
|
||
include/buf0buf\.ic: unused parameter ‘mtr’
|
||
fil/fil0fil\.c: pointer targets in passing argument.*differ in signedness
|
||
fil/fil0fil\.c: comparison between signed and unsigned : 3100-3199
|
||
fil/fil0fil\.c: unused parameter
|
||
log/log0recv\.c: unused variable
|
||
os/os0file\.c: unused parameter
|
||
os/os0file\.c: pointer targets in assignment differ in signedness
|
||
handler/i_s\.cc: unused variable
|
||
sync/sync0rw\.c: unused parameter
|
||
sync/sync0sync\.c: unused parameter
|
||
sync/sync0sync\.c: unused variable
|
||
ut/ut0ut\.c: ignoring return value of
|
||
|
||
#
|
||
# bdb is not critical to keep up to date
|
||
#
|
||
.*/bdb/.* : .*discards qualifiers from pointer target type.*
|
||
.*/bdb/.* : .*unused parameter.*
|
||
.*/bdb/.* : .*may be used uninitialized.*
|
||
.*/bdb/.* : .*empty body in an if-statement.*
|
||
db_vrfy.c : .*comparison is always false due to limited range of data type.*
|
||
|
||
#
|
||
# readline is not maintained by us
|
||
#
|
||
.*/cmd-line-utils/readline/.* : .*
|
||
|
||
#
|
||
# Ignore some warnings in libevent, which is not maintained by us.
|
||
#
|
||
.*/extra/libevent/.* : .*unused parameter.*
|
||
.*/extra/libevent/select\.c : .*comparison between signed and unsigned.* : 270-280
|
||
|
||
#
|
||
# Ignore warnings from system libraries
|
||
#
|
||
/usr/share/aclocal/audiofile.m4 : .*
|
||
|
||
#
|
||
# Ignore all conversion warnings on windows 64
|
||
# (Is safe as we are not yet supporting strings >= 2G)
|
||
#
|
||
.* : conversion from '__int64' to .*int'.*
|
||
.* : conversion from '__int64' to 'uint8'.*
|
||
.* : conversion from '__int64' to 'uint32'.*
|
||
.* : conversion from '__int64' to 'u.*long'.*
|
||
.* : conversion from '__int64' to 'long'.*
|
||
.* : conversion from '__int64' to 'off_t'.*
|
||
.* : conversion from '.*size_t' to .*int'.*
|
||
.* : conversion from '.*size_t' to 'TaoCrypt::word32'.*
|
||
.* : conversion from '.*size_t' to 'u.*long'.*
|
||
.* : conversion from '.*size_t' to 'uint32'.*
|
||
.* : conversion from '.*size_t' to 'off_t'.*
|
||
.* : conversion from '.*size_t' to 'size_s'.*
|
||
.* : conversion from '.*size_t' to 'DWORD'.*
|
||
.* : conversion from '.*size_t' to 'uLongf'.*
|
||
.* : conversion from '.*size_t' to 'UINT'.*
|
||
.* : conversion from '.*size_t' to 'uInt'.*
|
||
.* : conversion from '.*size_t' to 'uint16'.*
|
||
|
||
#
|
||
# The following should be fixed by the ndb team
|
||
#
|
||
.*/ndb/.* : .*used uninitialized in this function.*
|
||
.*/ndb/.* : .*unused variable.*
|
||
.*/ndb/.* : .*defined but not used.*
|
||
|
||
#
|
||
# Maria warning that is ok in debug builds
|
||
#
|
||
storage/maria/ma_pagecache.c: .*'info_check_pin' defined but not used
|
||
|
||
#
|
||
# I think these are due to mix of C and C++.
|
||
#
|
||
storage/pbxt/ : typedef.*was ignored in this declaration
|
||
ha_pbxt\.cc : variable.*might be clobbered by.*longjmp
|
||
|
||
#
|
||
# Yassl
|
||
include/runtime.hpp: .*pure_error.*
|
||
.*/extra/yassl/.*taocrypt/.*: comparison with string literal
|
||
.*/extra/yassl/taocrypt/src/blowfish\.cpp: array subscript is above array bounds
|
||
.*/extra/yassl/taocrypt/src/file\.cpp: ignoring return value
|
||
.*/extra/yassl/taocrypt/src/integer\.cpp: control reaches end of non-void function
|
||
|
||
#
|
||
# Groff warnings on OpenSUSE.
|
||
#
|
||
.*/dbug/.*(groff|<standard input>) : .*
|
||
|
||
#
|
||
# Unexplanable (?) stuff
|
||
#
|
||
listener.cc : .*conversion from 'SOCKET' to 'int'.*
|
||
net_serv.cc : .*conversion from 'SOCKET' to 'int'.*
|
||
set_var.cc: right-hand operand of comma has no effect : 1000-1400
|
||
|
||
|
||
# allow a little moving space for the warning below
|
||
mi_packrec.c : .*result of 32-bit shift implicitly converted to 64 bits.* : 560-600
|
||
ma_packrec.c : .*result of 32-bit shift implicitly converted to 64 bits.* : 550-650
|
||
|
||
#
|
||
# Wrong compiler warnings
|
||
#
|
||
.* : .*no matching operator delete found; memory will not be freed if initialization throws an exception.*
|
||
ctype-simple.c : .*unary minus operator applied to unsigned type, result still unsigned.*
|
||
|
||
# Wrong warning due to GCC bug: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=29478
|
||
regexec\.c : .*passing argument 3 of.*matcher.* discards qualifiers from pointer target type.*
|