[Scummvm-devel] g++ 3.4 patch

[Max Horn]

Am Montag, 18.08.03 um 23:19 Uhr schrieb Falk Hueffner:

> Hi,
>
> I just tried to compile CVS. g++ 3.4 with -pedantic won't accept
> spurious semicolons. It also warns about -Wstrict-prototypes (which
> doesn't make sense for C++). Patch attached.
>
> Also, I noticed that SCUMM_NEED_ALIGNMENT tries to detect whether
> unaligned accesses "work". Unfortunately, this is bogus; many systems,
<snip>
No this is not bogus. See also  
https://sourceforge.net/tracker/ 
?func=detail&atid=418820&aid=791741&group_id=37116 (which is the more  
appropriate place to continue this discussion anyway).


>
> Oh, and if you're that interested in performance, you should compile
> with -O2 or -O3, -O doesn't really do a lot of optimizations.
>
We only compile with -O for the sake of the -Wuninitialized warning.  
Otherwise, it would actually be preferable to compile without any -O  
option. Debugging with gdb is much more painful with -O.
Release builds may be built with -O2 / -O3, but this is up to the  
porter (I usually do the Mac OS X binaries using -O3).


Bye,

Max