[libvirt] [PATCH] [3/4] Implement managed save operations for qemu driver
Chris Lalancette
clalance at redhat.com
Fri Apr 2 20:51:15 UTC 2010
On 04/02/2010 04:26 PM, Eric Blake wrote:
>> -static int qemudDomainSave(virDomainPtr dom,
>> - const char *path)
>> +static int qemudDomainSaveFlag(virDomainPtr dom, const char *path,
>> + int compressed)
>
> Should that be bool compressed, since we are only using it as a binary?
> Or are there plans to extend it in the future to allow choice between
> multiple acceptable compression algorithms, in which case it is better
> as an unsigned int?
No, it's definitely not a bool. It's meant to take a number of different
arguments (like compressed-gzip, compressed-bzip2, etc). So an unsigned
int seems better.
--
Chris Lalancette
More information about the libvir-list
mailing list