Zandronum Chat on our Discord Server Get the latest version: 3.1
Source Code

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0001974Zandronum[All Projects] Bugpublic2014-10-27 10:232020-05-05 22:05
ReporterEdward-san 
Assigned To 
PrioritynormalSeveritytweakReproducibilityalways
StatusclosedResolutioninvalid 
PlatformOSOS Version
Product Version1.3 
Target VersionFixed in Version 
Summary0001974: Collection of various compiler warnings
DescriptionThis ticket is intended to post all the warnings GCC, Clang, MSVC, etc are found, while ignoring the GZDoom codebase ones.

To filter them, the only way I know is to compile gzdoom r900 (from this git repository:'https://github.com/doomtech/gzdoom [^]' ) in Release mode and save the warnings to a file; then compile zandronum 2.0 beta in Release mode and save the warnings to a file; compare them and remove the common warnings from both files.

There's a problem while compiling gzdoom r900 in non-Windows targets and it can be fixed with the attached patch compile-gzdoom900-linux.diff.txt (it's available only for linux targets for now).

I suggest to avoid using the old gzdoom r323 as codebase, because it doesn't compile with linux and requires some workarounds.

Currently, it's not possible to compile gzdoom and zandronum with clang, because its support in gzdoom was added a lot later.

It's possible to post non-default warnings, if they make sense.
Attached Filestxt file icon compile-gzdoom900-linux.diff.txt [^] (1,070 bytes) 2014-10-27 10:30 [Show Content]
txt file icon zandronum_2.x_warns_gcc.txt [^] (18,478 bytes) 2014-10-27 13:02 [Show Content]
txt file icon zandronum_2.x_warns_gcc_castqual.txt [^] (23,725 bytes) 2014-10-27 23:29 [Show Content]

- Relationships

-  Notes
User avatar (0010704)
Edward-san (developer)
2014-10-27 13:15
edited on: 2014-10-27 23:40

Attached what I get myself with gcc 4.9.1 on ubuntu linux x86_64.

All the reported -Wmaybe-uninitialized warnings are false positives (the one in cl_main.cpp is idiocy, the ones in g_raven/a_artitele.cpp are because the function I_Error (I_FatalError, too) does not have the 'noreturn' attribute).

[edit]Added also what I get if I add -Wcast-qual to the compiler options.

User avatar (0021314)
Edward-san (developer)
2020-05-05 22:05

This can be closed. Some of these warnings were fixed over the course of time. It lost its momentum, given that the mentioned warnings rarely refer to 'broken code', so it's fine as is.

Issue Community Support
This issue is already marked as resolved.
If you feel that is not the case, please reopen it and explain why.
Supporters: No one explicitly supports this issue yet.
Opponents: No one explicitly opposes this issue yet.

- Issue History
Date Modified Username Field Change
2014-10-27 10:23 Edward-san New Issue
2014-10-27 10:23 Edward-san Status new => assigned
2014-10-27 10:23 Edward-san Assigned To => Edward-san
2014-10-27 10:30 Edward-san Description Updated View Revisions
2014-10-27 10:30 Edward-san File Added: compile-gzdoom900-linux.diff.txt
2014-10-27 10:31 Edward-san Description Updated View Revisions
2014-10-27 13:02 Edward-san File Added: zandronum_2.x_warns_gcc.txt
2014-10-27 13:15 Edward-san Note Added: 0010704
2014-10-27 23:29 Edward-san File Added: zandronum_2.x_warns_gcc_castqual.txt
2014-10-27 23:40 Edward-san Note Edited: 0010704 View Revisions
2020-05-05 22:05 Edward-san Note Added: 0021314
2020-05-05 22:05 Edward-san Status assigned => closed
2020-05-05 22:05 Edward-san Assigned To Edward-san =>
2020-05-05 22:05 Edward-san Resolution open => invalid






Questions or other issues? Contact Us.

Links


Copyright © 2000 - 2024 MantisBT Team
Powered by Mantis Bugtracker