[libvirt] [PATCH 4/6] virsh: Create macro for "overwrite" and no-overwrite" options
John Ferlan
jferlan at redhat.com
Wed Dec 16 15:45:20 UTC 2015
On 12/16/2015 10:32 AM, John Ferlan wrote:
>
> [...]
>
>>>
>>> So then in your opinion and using the same logic, patch 3 is also nixed?
>>> This one will get used in patch 6.
>>
>> With that use it will result in total 4 uses. I won't object if you fix
>> the macro name and remove the comma
>>
>
> Hmm. by name change do you mean "OPT_POOL_FILE_COMMON",
> "OPT_POOL_OVERWRITE_COMMON", etc.? I guess I was thinking less globally
> available macros, e.g. module specific rather than adding them to some
> more generally included virsh*.h file.
>
nm... patch 2 comment... you'd rather see VSH_POOL_OPT_COMMON than
OPT_POOL_COMMON
John
More information about the libvir-list
mailing list