18.05.2016 10:54, Nikolay Shirokovskiy пишет:


On 17.05.2016 22:02, Maxim Nestratov wrote:
22.04.2016 10:20, Nikolay Shirokovskiy пишет:

vz puts uuids into curly braces. Simply introduce new contstant to reflect this
and get rid of magic +2 in code.

Signed-off-by: Nikolay Shirokovskiy <[email protected]>
---
   src/vz/vz_sdk.c   | 12 ++++++------
   src/vz/vz_utils.h |  2 ++
   2 files changed, 8 insertions(+), 6 deletions(-)
I bet we will confuse VZ_UUID_STRING_BUFLEN and VIR_UUID_STRING_BUFLEN 
eventually
That's why I doubt we need this patch

I see, what if I use VIR_UUID_STRING_BRACED_BUFLEN instead?

OK to me. But if you use VIR prefix, then it's better to define it beside VIR_UUID_STRING_BUFLEN

--
libvir-list mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to