Source-Changes-D archive

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

Re: CVS commit: src/external/bsd/atf/dist/atf-c



In article <20111107202432.GA7502%britannica.bec.de@localhost>,
Joerg Sonnenberger  <joerg%britannica.bec.de@localhost> wrote:
>On Mon, Nov 07, 2011 at 04:06:30PM +0000, Christos Zoulas wrote:
>> Well, I tried to print the failing pattern in t_expand, and it silently
>> got truncated. dprintf(3) has been part of TOG since 2006:
>> (http://pubs.opengroup.org/onlinepubs/9699919799/functions/dprintf.html)
>> So it would be preferable to implement it in terms of asprintf/write +
>> or snprintf/malloc/write or even fdopen/fprintf instead of open coding it.
>
>From reading the patch, there are three different brances, right?
>Why don't you use writev for this with up to 6 elements in the vector?

Done.

christos



Home | Main Index | Thread Index | Old Index