Compilation Warnings Linux

[Auntie Mangos]

Actually with the chnages i made and config options i have here --> http://getmangos.eu/community/showthread.php?12411-High-Performance-Tips/page2 it runs smoothly for i dont know how many hours. Population around 30 PCs.

[cyrex]

Compilation Warnings on Linux (Ubuntu 32Bit)

Mangos Core - 9407

SD2 - 1583

K.Tpo -c ../../../../dep/ACE_wrappers/ace/LSOCK.cpp  -fPIC -DPIC -o .libs/libACE_la-LSOCK.o
../../../../dep/ACE_wrappers/ace/LSOCK.cpp: In member function ‘ssize_t ACE_LSOCK::send_handle(ACE_HANDLE) const’:
../../../../dep/ACE_wrappers/ace/LSOCK.cpp:62: warning: dereferencing type-punned pointer will break strict-aliasing rules
../../../../dep/ACE_wrappers/ace/LSOCK.cpp: In member function ‘ssize_t ACE_LSOCK::recv_handle(ACE_HANDLE&, char*, ssize_t*) const’:
../../../../dep/ACE_wrappers/ace/LSOCK.cpp:126: warning: dereferencing type-punned pointer will break strict-aliasing rules
mv -f .deps/libACE_la-LSOCK.Tpo .deps/libACE_la-LSOCK.Plo


la-LSOCK_Stream.Tpo -c ../../../../dep/ACE_wrappers/ace/LSOCK_Stream.cpp  -fPIC -DPIC -o .libs/libACE_la-LSOCK_Stream.o
../../../../dep/ACE_wrappers/ace/LSOCK_Stream.cpp: In member function ‘ssize_t ACE_LSOCK_Stream::send_msg(const iovec*, size_t, ACE_HANDLE)’:
../../../../dep/ACE_wrappers/ace/LSOCK_Stream.cpp:87: warning: dereferencing type-punned pointer will break strict-aliasing rules
../../../../dep/ACE_wrappers/ace/LSOCK_Stream.cpp: In member function ‘ssize_t ACE_LSOCK_Stream::recv_msg(iovec*, size_t, ACE_HANDLE&)’:
../../../../dep/ACE_wrappers/ace/LSOCK_Stream.cpp:123: warning: dereferencing type-punned pointer will break strict-aliasing rules
mv -f .deps/libACE_la-LSOCK_Stream.Tpo .deps/libACE_la-LSOCK_Stream.Plo


../../../../dep/ACE_wrappers/ace/POSIX_Proactor.cpp: In member function ‘virtual int ACE_POSIX_AIOCB_Proactor::get_result_status(ACE_POSIX_Asynch_Result*, int&, size_t&)’:
../../../../dep/ACE_wrappers/ace/POSIX_Proactor.cpp:1242: warning: null argument where non-null required (argument 1)
../../../../dep/ACE_wrappers/ace/POSIX_Proactor.cpp:1246: warning: null argument where non-null required (argument 1)
../../../../dep/ACE_wrappers/ace/POSIX_Proactor.cpp: In member function ‘virtual int ACE_POSIX_AIOCB_Proactor::start_aio_i(ACE_POSIX_Asynch_Result*)’:
../../../../dep/ACE_wrappers/ace/POSIX_Proactor.cpp:1433: warning: null argument where non-null required (argument 1)
../../../../dep/ACE_wrappers/ace/POSIX_Proactor.cpp:1437: warning: null argument where non-null required (argument 1)


CXX    genrevision.o
../../../../src/tools/genrevision/genrevision.cpp: In function ‘void extractDataFromSvn(FILE*, bool, RawData&)’:
../../../../src/tools/genrevision/genrevision.cpp:41: warning: ignoring return value of ‘char* fgets(char*, int, FILE*)’, declared with attribute warn_unused_result
../../../../src/tools/genrevision/genrevision.cpp:42: warning: ignoring return value of ‘char* fgets(char*, int, FILE*)’, declared with attribute warn_unused_result
../../../../src/tools/genrevision/genrevision.cpp:43: warning: ignoring return value of ‘char* fgets(char*, int, FILE*)’, declared with attribute warn_unused_result
../../../../src/tools/genrevision/genrevision.cpp:44: warning: ignoring return value of ‘char* fgets(char*, int, FILE*)’, declared with attribute warn_unused_result
../../../../src/tools/genrevision/genrevision.cpp:45: warning: ignoring return value of ‘char* fgets(char*, int, FILE*)’, declared with attribute warn_unused_result
../../../../src/tools/genrevision/genrevision.cpp:46: warning: ignoring return value of ‘char* fgets(char*, int, FILE*)’, declared with attribute warn_unused_result
../../../../src/tools/genrevision/genrevision.cpp:47: warning: ignoring return value of ‘char* fgets(char*, int, FILE*)’, declared with attribute warn_unused_result
../../../../src/tools/genrevision/genrevision.cpp:48: warning: ignoring return value of ‘char* fgets(char*, int, FILE*)’, declared with attribute warn_unused_result
../../../../src/tools/genrevision/genrevision.cpp:49: warning: ignoring return value of ‘char* fgets(char*, int, FILE*)’, declared with attribute warn_unused_result
../../../../src/tools/genrevision/genrevision.cpp:50: warning: ignoring return value of ‘char* fgets(char*, int, FILE*)’, declared with attribute warn_unused_result
 CXXLD  genrevision


 CXX    dotconfpp.o
../../../../src/shared/Config/dotconfpp/dotconfpp.cpp: In member function ‘int DOTCONFDocument::setContent(const char*)’:
../../../../src/shared/Config/dotconfpp/dotconfpp.cpp:425: warning: ignoring return value of ‘char* fgets(char*, int, FILE*)’, declared with attribute warn_unused_result
 CXX    mempool.o


 CXX    ModelContainer.o
../../../../src/shared/vmap/ModelContainer.cpp: In member function ‘bool VMAP::ModelContainer::readFile(const char*)’:
../../../../src/shared/vmap/ModelContainer.cpp:258: warning: ignoring return value of ‘size_t fread(void*, size_t, size_t, FILE*)’, declared with attribute warn_unused_result
 CXX    SubModel.o


 CXX    VMapManager.o
../../../../src/shared/vmap/VMapManager.cpp: In member function ‘bool VMAP::VMapManager::_existsMap(const std::string&, unsigned int, int, int, bool)’:
../../../../src/shared/vmap/VMapManager.cpp:294: warning: ignoring return value of ‘size_t fread(void*, size_t, size_t, FILE*)’, declared with attribute warn_unused_result
 AR     libmangosvmaps.a


 CXX    DBCStores.o
../../../src/game/DBCStores.cpp: In function ‘bool ReadDBCBuildFileText(const std::string&, const char*, std::string&)’:
../../../src/game/DBCStores.cpp:191: warning: ignoring return value of ‘size_t fread(void*, size_t, size_t, FILE*)’, declared with attribute warn_unused_result
 CXX    debugcmds.o


 CXX    Map.o
../../../src/game/Map.cpp: In static member function ‘static bool Map::ExistMap(uint32, int, int)’:
../../../src/game/Map.cpp:84: warning: ignoring return value of ‘size_t fread(void*, size_t, size_t, FILE*)’, declared with attribute warn_unused_result
../../../src/game/Map.cpp: In member function ‘bool GridMap::loadData(char*)’:
../../../src/game/Map.cpp:1167: warning: ignoring return value of ‘size_t fread(void*, size_t, size_t, FILE*)’, declared with attribute warn_unused_result
../../../src/game/Map.cpp: In member function ‘bool GridMap::loadAreaData(FILE*, uint32, uint32)’:
../../../src/game/Map.cpp:1220: warning: ignoring return value of ‘size_t fread(void*, size_t, size_t, FILE*)’, declared with attribute warn_unused_result
../../../src/game/Map.cpp:1228: warning: ignoring return value of ‘size_t fread(void*, size_t, size_t, FILE*)’, declared with attribute warn_unused_result
../../../src/game/Map.cpp: In member function ‘bool GridMap::loadHeightData(FILE*, uint32, uint32)’:
../../../src/game/Map.cpp:1237: warning: ignoring return value of ‘size_t fread(void*, size_t, size_t, FILE*)’, declared with attribute warn_unused_result
../../../src/game/Map.cpp:1248: warning: ignoring return value of ‘size_t fread(void*, size_t, size_t, FILE*)’, declared with attribute warn_unused_result
../../../src/game/Map.cpp:1249: warning: ignoring return value of ‘size_t fread(void*, size_t, size_t, FILE*)’, declared with attribute warn_unused_result
../../../src/game/Map.cpp:1257: warning: ignoring return value of ‘size_t fread(void*, size_t, size_t, FILE*)’, declared with attribute warn_unused_result
../../../src/game/Map.cpp:1258: warning: ignoring return value of ‘size_t fread(void*, size_t, size_t, FILE*)’, declared with attribute warn_unused_result
../../../src/game/Map.cpp:1266: warning: ignoring return value of ‘size_t fread(void*, size_t, size_t, FILE*)’, declared with attribute warn_unused_result
../../../src/game/Map.cpp:1267: warning: ignoring return value of ‘size_t fread(void*, size_t, size_t, FILE*)’, declared with attribute warn_unused_result
../../../src/game/Map.cpp: In member function ‘bool GridMap::loadLiquidData(FILE*, uint32, uint32)’:
../../../src/game/Map.cpp:1280: warning: ignoring return value of ‘size_t fread(void*, size_t, size_t, FILE*)’, declared with attribute warn_unused_result
../../../src/game/Map.cpp:1294: warning: ignoring return value of ‘size_t fread(void*, size_t, size_t, FILE*)’, declared with attribute warn_unused_result
../../../src/game/Map.cpp:1299: warning: ignoring return value of ‘size_t fread(void*, size_t, size_t, FILE*)’, declared with attribute warn_unused_result
 CXX    MapInstanced.o


 CXX    SpellEffects.o
../../../src/game/SpellEffects.cpp: In member function ‘void Spell::EffectAddHonor(uint32)’:
../../../src/game/SpellEffects.cpp:4065: warning: format ‘%u’ expects type ‘unsigned int’, but argument 4 has type ‘double’
 CXX    SkillDiscovery.o


 CXX    instance_violet_hold.lo
../../../../src/bindings/ScriptDev2/scripts/northrend/violet_hold/instance_violet_hold.cpp: In member function ‘virtual void instance_violet_hold::SetData(uint32, uint32)’:
../../../../src/bindings/ScriptDev2/scripts/northrend/violet_hold/instance_violet_hold.cpp:183: warning: ' ' flag used with ‘%u’ gnu_printf format
 CXX    violet_hold.lo

[cyrex]

This is to help a little bit of cleaning on the code. Like the many fixes NoFantasy and Vladimir made some revisions ago which made a lot of cleaning. The only one lefts are this ones which is good.

[dereka]

-Wall -Wextra -Wuninitialized -Wconversion

try compile with this flags

[silverleet]

-Wall -Wextra -Wuninitialized -Wconversion

try compile with this flags

that would fix the warnings?

[VladimirMangos]

no, this will add more

[cyrex]

no, this will add more

hahahahhahahhahahah

[silverleet]

no, this will add more

it's a bit hard to find out what makes mangos crash so much in ubuntu. since 9200+ it does

mangos code bug maybe?