Re: [PATCH v6 REBASE 5/5] vz: cleanup: define vz format of uuids

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

 



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 <nshirokovskiy@xxxxxxxxxxxxx>
---
   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
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]