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 <[email protected]>
---
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.
[..]
--
libvir-list mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/libvir-list
