On 07.05.2013 05:24, Eric Blake wrote: > As pointed out during my (ongoing) review of Michal's series, I > think that letting VIR_STRDUP take a NULL source can make it > easier to use; I also think that documenting the guarantee that > it evaluates arguments exactly once is worth enforcing. > > Hmm, I guess that means I should write a 3/2 patch that enhances > the testsuite to intentionally do a side effect and show that it > happens exactly once. > > Eric Blake (2): > alloc: make VIR_APPEND_ELEMENT safer > string: make VIR_STRDUP easier to use > > HACKING | 19 ++++++++++++------- > docs/hacking.html.in | 16 ++++++++++++---- > src/util/viralloc.c | 9 ++++++--- > src/util/viralloc.h | 36 +++++++++++++++++++++++++++--------- > src/util/virstring.c | 12 ++++++++---- > src/util/virstring.h | 22 ++++++++++++++++------ > 6 files changed, 81 insertions(+), 33 deletions(-) > ACK series Michal -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list