On Sun, Aug 13, 2006 at 12:07:02PM +0200, Markus Raab wrote: > Am Freitag, 11. August 2006 22:50 schrieb Patrice Dumas: > > On Fri, Aug 11, 2006 at 02:50:28PM +0200, Markus Raab wrote: > > > Am Dienstag, 1. August 2006 01:27 schrieb Patrice Dumas: > > > For me -ansi add more warnings, since it complains for functions that > > are not in C99 but are in previous standards. -pedantic don't change
I got it backward. I wanted to say 'it complains for functions that are in C99 but weren't in previous standards'. Looks like I was wrong, however, and those functions don't seem to be in C99. > And for me reaching ISO C99 Code is an aim (but not the most important...) It will help portability quite a lot. > > -Wall if compiler is gcc and CFLAGS is empty. But I removed -pedantic, > > -ansi (and -Wall) everywhere in the Makefile.am. > > Which will move us away from the aim to have C99 code once in the future;) -ansi is for C89, not C99. -std=c99 is for C99. I personally think that we should try to have C99 compatibility, and add portability for C89 only if there a platform which we port to, which lacks some C99 features. I think that fixing all the -ansi -pedantic issues is doing too much work for old proprietary platforms which will never embrace elektra anyway. I may be wrong, however... What is your opinion on that? > If you use C99. If you use all gcc features you may have luck with the intel > compiler, but others? solaris 8 cc compiles, installs libelektra, libelektratools and filesys backend. > mmh, make install failes here too;) What is your platform? (note that this is not a c portability issue ;) > test -e libelektra-ddefault.so || \ > ln -s libelektra-berkeleydb.so libelektra-ddefault.so > ln: ,,libelektra-ddefault.so": file existing > Because the C gcc accepts (without that flags) is something else then C99. > And > something else then any other compiler supports (see linux source, but even > they restrict to some features). Ok, so we could add -std=c99. After some testing it appears very similar with -ansi, so we need some work here... (it breaks in berkeleydb backend for me). But I still think that this should be set in CFLAGS when running ./configure and not hardcoded. -- Pat ------------------------------------------------------------------------- Using Tomcat but need to do more? Need to support web services, security? Get stuff done quickly with pre-integrated technology to make your job easier Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642 _______________________________________________ Registry-list mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/registry-list
