bug-gnulib
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: new module: xprintf


From: Jim Meyering
Subject: Re: new module: xprintf
Date: Sat, 20 Oct 2007 16:20:37 +0200

Bruno Haible <address@hidden> wrote:
> Thanks; this module is necessary.
>
> Two suggestions:
>
> - Add a call to va_end() that matches the va_start() call. POSIX says:
>   "Each invocation of the va_start() and va_copy() macros shall be matched
>    by a corresponding invocation of the va_end() macro in the same function."
>
> - I would make it clearer in the error message that the error is about
>   formatted output. I don't feel that "write error" is good enough, given
>   that the usual reasons for failing printf(), when !ferror(stream), are:
>     EINVAL - invalid format string
>     EILSEQ - cannot convert a wide character string
>     ENOMEM - out of memory
>   and these are not immediately related to writing on the stream.

Thanks.
I applied your patch.




reply via email to

[Prev in Thread] Current Thread [Next in Thread]