[libvirt] [PATCHv3 06/10] storage: simplify fd handling
Eric Blake
eblake at redhat.com
Fri Mar 4 13:00:36 UTC 2011
On 03/04/2011 01:15 AM, Paolo Bonzini wrote:
> On 03/04/2011 01:37 AM, Eric Blake wrote:
>> - unsigned int flags ATTRIBUTE_UNUSED)
>> + unsigned int flags)
>
> Uhm, you're still not using the flags, you're just using the argument as
> a place to write the flags for virFileOperation. Perhaps the "inner"
> flags variable should be renamed?
Actually, I did use the flags - the virCheckFlags(0,-1) added at the top
of the function is a macro that uses the incoming argument. But yes,
repurposing the incoming argument is confusing, I'll rename the second
use of flags to something else.
--
Eric Blake eblake at redhat.com +1-801-349-2682
Libvirt virtualization library http://libvirt.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 619 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20110304/ec0a1d87/attachment-0001.sig>
More information about the libvir-list
mailing list