[libvirt] [PATCH v2 1/5] introduce virSnprintf helper
Pavel Hrdina
phrdina at redhat.com
Mon Sep 15 12:06:00 UTC 2014
On 09/12/2014 08:47 PM, John Ferlan wrote:
>
>
> On 09/10/2014 08:08 AM, Pavel Hrdina wrote:
>> Signed-off-by: Pavel Hrdina <phrdina at redhat.com>
>> ---
>> src/libvirt_private.syms | 1 +
>> src/util/virstring.c | 25 +++++++++++++++++++++++++
>> src/util/virstring.h | 20 ++++++++++++++++++++
>> 3 files changed, 46 insertions(+)
>>
>
> Not sure why this is necessary - I do see calls to 'snprintf' from
> within existing libvirt code. The usage you have in patch 5 could
> easily use snprintf. There is no need for variable argument list.
>
> I'm not against it, but if it's created shouldn't other snprintf's use it?
>
> Hopefully someone else has an opinion on this...
>
> John
OK, I'll drop it for this patch series.
[..]
More information about the libvir-list
mailing list