NeverEnding #825

Remove all warnings when compiling Ryzom

Added by vl about 5 years ago. Updated over 4 years ago.

Status:New Start date:04/30/2010
Priority:Normal Due date:
Assignee:- % Done:

0%

Category:Build
Target version:-

warning_v2.diff (48.8 kB) Magnifier Letto, 05/12/2010 09:25 am

warnings_v3.diff (55.3 kB) Magnifier Letto, 05/14/2010 04:20 pm

warnings_v4.diff (49.8 kB) Magnifier Letto, 05/15/2010 01:30 pm

warn_linux_gcc.patch (83.8 kB) Magnifier abelgar, 09/22/2010 11:07 am

nel-warning-1.diff (4.9 kB) Magnifier Naush, 10/04/2010 09:18 pm


Related issues

duplicated by Ryzom - NeverEnding #29: Fix compiler warnings Closed 09/03/2008

History

#1 Updated by Letto about 5 years ago

  • File warnings.diff added

I managed to get off almost all the warnings when the client compiles with cmake with Debug up to 40%
I'm pretty sure those that remain are bugs.

#2 Updated by Letto about 5 years ago

Updated, patch after cmake-stuff.diff
Compile in Debug mode
http://pastebin.com/nkwLZHL9

#3 Updated by jayme about 5 years ago

  • File warnings-seven_zip.patch added

Removed some warnings in seven_zip files...

#4 Updated by ThibG about 5 years ago

  • File neldebug.patch added

#5 Updated by ThibG about 5 years ago

  • File nl_ct_debug.patch added

#6 Updated by ThibG about 5 years ago

  • File nel-warnings.patch added

#7 Updated by kervala about 5 years ago

  • File deleted (nel-warnings.patch)

#8 Updated by kervala about 5 years ago

Patch nel-warnings.patch applied and removed :) (to be sure it won't be applied a second time :p)

Thanks a lot ThibG :)

I will continue to apply them.

#9 Updated by kervala about 5 years ago

  • File deleted (warnings.diff)

#10 Updated by Letto about 5 years ago

  • File nldebug2.patch added

Here's a better nldebug patch that forces trailing ;

#11 Updated by jayme about 5 years ago

  • File sp_type_string-const-to-char_warnings.patch added

Removed a "deprecated conversion from string constant to ‘char*’" warning from code/ryzom/common/src/game_share/sp_type.cpp

#12 Updated by Letto about 5 years ago

I managed to compile the client with these.
Includes vhelsing's patches.

#13 Updated by Letto about 5 years ago

#14 Updated by vl about 5 years ago

We applied some of warning, can you provide another patch with the latest repo?

#15 Updated by Letto about 5 years ago

you have added lines like this wich i deleted in the latest version
nlwarning("Unknown light cycle state reached.");
Beware that this causes the client to continually print that message. Do you intend to fix that in some other way?

#16 Updated by ThibG about 5 years ago

Please forget my neldebug and nl_ct_debug patches. They use variadic macros, which are part of the C99 specifications, but not part of C++.

#17 Updated by jayme about 5 years ago

  • File string_constant_to_char.patch added

Removed some more "deprecated conversion from string constant to ‘char*’"-warnings

#18 Updated by vl about 5 years ago

string_constant_to_char.patch was applied on the repo

#19 Updated by kervala about 5 years ago

  • File deleted (string_constant_to_char.patch)

#20 Updated by kervala about 5 years ago

  • File deleted (warnings-seven_zip.patch)

#21 Updated by kervala about 5 years ago

  • File deleted (sp_type_string-const-to-char_warnings.patch)

#22 Updated by kervala about 5 years ago

  • Subject changed from Remove all warning when compiling Ryzom on Linux to Remove all warning when compiling Ryzom

#23 Updated by kervala about 5 years ago

  • File deleted (nldebug2.patch)

#24 Updated by kervala about 5 years ago

  • File deleted (nl_ct_debug.patch)

#25 Updated by kervala about 5 years ago

  • File deleted (neldebug.patch)

#26 Updated by kervala almost 5 years ago

  • Subject changed from Remove all warning when compiling Ryzom to Remove all warnings when compiling Ryzom

#27 Updated by abelgar almost 5 years ago

Here some warns check.
In diff, file code/ryzom/server/src/ai_service/ai_entity_physical_inline.h is weird. So be careful. Others are good.

#28 Updated by kervala almost 5 years ago

Thanks for your patch abelgar :)

#29 Updated by Naush almost 5 years ago

Some more, for NeL

#30 Updated by kervala over 4 years ago

  • Tracker changed from Feature to NeverEnding

Also available in: Atom PDF