Re: [xml] vsnprintf

[Adrian Bunk <bunk stusta de>]

On Wed, Dec 15, 2010 at 03:38:14PM -0600, Peter O'Gorman wrote:
> ...
> --- a/Makefile.am
> +++ b/Makefile.am
> @@ -31,17 +31,6 @@ libxml2_la_LDFLAGS = @CYGWIN_EXTRA_LDFLAGS@ @WIN32_EXTRA_LDFLAGS@ \
>                    -version-info @LIBXML_VERSION_INFO@ \
>                    @MODULE_PLATFORM_LIBS@
>  
> -if WITH_TRIO_SOURCES
> -libxml2_la_SOURCES = SAX.c entities.c encoding.c error.c parserInternals.c  \
> -             parser.c tree.c hash.c list.c xmlIO.c xmlmemory.c uri.c  \
> -             valid.c xlink.c HTMLparser.c HTMLtree.c debugXML.c xpath.c  \
> -             xpointer.c xinclude.c nanohttp.c nanoftp.c DOCBparser.c \
> -             catalog.c globals.c threads.c c14n.c xmlstring.c \
> -             xmlregexp.c xmlschemas.c xmlschemastypes.c xmlunicode.c \
> -             triostr.c trio.c xmlreader.c relaxng.c dict.c SAX2.c \
> -             xmlwriter.c legacy.c chvalid.c pattern.c xmlsave.c \
> -             xmlmodule.c schematron.c
> -else
>  libxml2_la_SOURCES = SAX.c entities.c encoding.c error.c parserInternals.c  \
>               parser.c tree.c hash.c list.c xmlIO.c xmlmemory.c uri.c  \
>               valid.c xlink.c HTMLparser.c HTMLtree.c debugXML.c xpath.c  \
> @@ -51,6 +40,8 @@ libxml2_la_SOURCES = SAX.c entities.c encoding.c error.c parserInternals.c  \
>               xmlreader.c relaxng.c dict.c SAX2.c \
>               xmlwriter.c legacy.c chvalid.c pattern.c xmlsave.c \
>               xmlmodule.c schematron.c
> +if WITH_TRIO_SOURCES
> +libxml2_la_SOURCES += trio.c triostr.c
>  endif
> ...

This looks like a nice cleanup, but unrelated to the problem you are 
fixing.

> --- a/configure.in
> +++ b/configure.in
> @@ -479,6 +479,25 @@ AC_CHECK_FUNCS(stat _stat signal)
>  dnl Checking the standard string functions availability
>  AC_CHECK_FUNCS(printf sprintf fprintf snprintf vfprintf vsprintf vsnprintf sscanf,,
>                 NEED_TRIO=1)
> +AC_CACHE_CHECK([if vsnprintf is POSIX compliant],[xml_cv_working_vsnprintf],
> ...

Referring to my comment to your first email, I'd suggest "C99" instead 
of "POSIX".

cu
Adrian

-- 

       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed