Re: [libvirt] [PATCH v2 6/7] qemu: Switch over command line capabilities to virBitmap

This is done for two reasons: - we are getting very close to 64 flags which is the maximum we can use with unsigned long long - by using LL constants in enum we already violates C99 constraint that enum values have to fit into int --- Notes: Version 2: - rebased to current master - qemuCapsFree added to a list of free-like functions in cfg.mk - don't fail qemuCapsParseHelpStr() if there is no memory for bitmap string used for a debug message

+++ b/cfg.mk @@ -135,7 +135,8 @@ useless_free_options = \ --name=virThreadPoolFree \ --name=xmlFree \ --name=xmlXPathFreeContext \ - --name=xmlXPathFreeObject + --name=xmlXPathFreeObject \ + --name=qemuCapsFree

@@ -1029,10 +1029,13 @@ int qemuCapsParseHelpStr(const char *qemu, *version = (major * 1000 * 1000) + (minor * 1000) + micro; - *flags = qemuCapsComputeCmdFlags(help, *version, *is_kvm, *kvm_version); + qemuCapsComputeCmdFlags(help, *version, *is_kvm, *kvm_version, flags); + + strflags = virBitmapString(flags); + VIR_DEBUG("Version %u.%u.%u, cooked version %u, flags %s", + major, minor, micro, *version, NULLSTR(strflags)); + VIR_FREE(strflags); - VIR_DEBUG("Version %u.%u.%u, cooked version %u, flags 0x%llx", - major, minor, micro, *version, *flags);