Re: [PATCHv2 1/2] DUPLICATE: util: Add function to check if a virStorageSource is "empty"

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

 



On 09/12/2014 01:43 AM, Peter Krempa wrote:

>>
>>> + * Returns true if @src points to an empty storage source.
>>> + */
>>> +bool
>>> +virStorageSourceIsEmpty(virStorageSourcePtr src)
>>
>> Maybe the comment is better as:
>>
>> Returns true if the guest disk has no associated host storage source
>> (such as an empty cdrom drive).
>>
> 
> Sounds better. I've changed the comment and pushed this patch.

Even more bikeshedding ideas (but don't worry about it, since you
already pushed):

virStorageSourceNoMedia() Returns true if a guest disk has no media
(that is, no associated host storage source)

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

--
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]