>> By the way, we should add -Wall in configure.in (it was once, who removed >> it?). And perhaps also use -O2 instead of -O3 and add -g. Those were the >> previous default values I think. > > In the HEAD cvs, -Wall should be added, agreed. -O3 (as opposed to > -O2 -g) was chosen because a user who knows how to use gdb will be > able to override compile settings. An average user will see a crash > and know nothing of how to solve it. When debugging with them, we can > simply tell them how to compile with -g. Other users, especially on > low cpu systems, can benifet from the noticable speed increase that > comes with -O3. If people strongly feel the differ, we can change > those options on the next release.
I think "-O3" isn't much better than "-O2". We could use "-ffast-math" for PerlinNoise.cpp. With "-g -O2" users can make backtraces without having to compile glob2 at all. We could measure speed, to see if it is worth it. "-O3" in HEAD is bad. It increases compilation time considerably without much speed improvements. Given the frequency of compilation this is no good. I like "-Wall". -- Kai Antweiler _______________________________________________ glob2-devel mailing list [email protected] http://lists.nongnu.org/mailman/listinfo/glob2-devel
