On 08.05.2011 11:44, Daniel Veillard wrote: > On Fri, May 06, 2011 at 05:49:20PM +0300, Stefan Kost wrote: >> On 04.05.2011 12:18, Daniel Veillard wrote: >>> On Wed, May 04, 2011 at 11:43:03AM +0300, Stefan Kost wrote: >>>> On 03.05.2011 18:14, Stefan Kost wrote: >>>>> hi, >>>>> >>>>> I am still not happy with the time it takes for gtk-doc to generate the >>>>> html. I did some profiling and tired various changes. Its not too >>>>> impressive. Just posting them here for feedback and comment on the >>>>> directions of what would be acceptable or not. I technically have commit >>>>> rights, so I can push things once they are ready. If patch 0002 looks >>>>> like a good idea, I can extend it for other accessors. >>>> Daniel, btw. are you fine with me pushing a couple of obvious patches >>>> that only fix compiler warnings (like unused variables and missing >>>> casts). I would do that for libxml2 and libxslt. >> There are a lot of warnings like this: >> ISO C90 does not support "long long" > Ignore all compilers where we need this have it 0029-configure-add-Wno-long-long-to-CFLAGS.patch >> ISO C forbids assignment between function pointer and 'void *' > Leave them, they come from an API definition, we can't really change > it as it's public, and trying to hide the issue isn't good either, > we need to learn to live with those.
It is only 4 of them and they can be easily found by grepping for XML_CAST_FPTR where you tried to hide them in the past. The build looks really good now. Stefan
>From 3995f5e705b14771b1abf2e88bc90f6d8e9960b5 Mon Sep 17 00:00:00 2001 From: Stefan Kost <[email protected]> Date: Tue, 10 May 2011 10:55:07 +0300 Subject: [PATCH 29/29] configure: add -Wno-long-long to CFLAGS --- configure.in | 7 +++++-- 1 files changed, 5 insertions(+), 2 deletions(-) diff --git a/configure.in b/configure.in index 7502e7b..58741be 100644 --- a/configure.in +++ b/configure.in @@ -664,8 +664,11 @@ else # CFLAGS="${CFLAGS} -fexceptions" fi - - CFLAGS="${CFLAGS} -pedantic -W -Wformat -Wunused -Wimplicit -Wreturn-type -Wswitch -Wcomment -Wtrigraphs -Wformat -Wchar-subscripts -Wuninitialized -Wparentheses -Wshadow -Wpointer-arith -Wcast-align -Wwrite-strings -Waggregate-return -Wstrict-prototypes -Wmissing-prototypes -Wnested-externs -Winline -Wredundant-decls" + + # warnings we'd like to see + CFLAGS="${CFLAGS} -pedantic -W -Wformat -Wunused -Wimplicit -Wreturn-type -Wswitch -Wcomment -Wtrigraphs -Wformat -Wchar-subscripts -Wuninitialized -Wparentheses -Wshadow -Wpointer-arith -Wcast-align -Wwrite-strings -Waggregate-return -Wstrict-prototypes -Wmissing-prototypes -Wnested-externs -Winline -Wredundant-decls" + # warnings we'd like to supress + CFLAGS="${CFLAGS} -Wno-long-long" case "${host}" in alpha*-*-linux* ) CFLAGS="${CFLAGS} -mieee" -- 1.7.1
_______________________________________________ xslt mailing list, project page http://xmlsoft.org/XSLT/ [email protected] http://mail.gnome.org/mailman/listinfo/xslt
