Re: speeding up xslt in gtk-doc

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

_______________________________________________
xslt mailing list, project page http://xmlsoft.org/XSLT/
[email protected]
http://mail.gnome.org/mailman/listinfo/xslt

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