[libvirt] [PATCH] virDomainDefFormatInternal: Drop useless check
Martin Kletzander
mkletzan at redhat.com
Mon Feb 22 09:58:33 UTC 2016
On Mon, Feb 22, 2016 at 10:06:34AM +0100, Erik Skultety wrote:
>On 22/02/16 08:57, Michal Privoznik wrote:
>> There's a check if a domain definition has any graphics card and
>> if so, we iterate over each one of them. This makes no sense,
>> because even if it has none we can still iterate over.
>>
>> Signed-off-by: Michal Privoznik <mprivozn at redhat.com>
>> ---
>> src/conf/domain_conf.c | 8 +++-----
>> 1 file changed, 3 insertions(+), 5 deletions(-)
>>
>> diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
>> index e47d613..56bd1aa 100644
>> --- a/src/conf/domain_conf.c
>> +++ b/src/conf/domain_conf.c
>> @@ -22363,11 +22363,9 @@ virDomainDefFormatInternal(virDomainDefPtr def,
>> goto error;
>> }
>>
>> - if (def->ngraphics > 0) {
>> - for (n = 0; n < def->ngraphics; n++)
>> - if (virDomainGraphicsDefFormat(buf, def->graphics[n], flags) < 0)
>> - goto error;
>> - }
>> + for (n = 0; n < def->ngraphics; n++)
>> + if (virDomainGraphicsDefFormat(buf, def->graphics[n], flags) < 0)
>> + goto error;
>>
You could also add the curly brackets so it complies with our coding
style. And also add similar fixes from other files. But I see you've
already pushed it, so I'll send a follow-up.
>> for (n = 0; n < def->nsounds; n++)
>> if (virDomainSoundDefFormat(buf, def->sounds[n], flags) < 0)
>>
>
>ACK
>
>Erik
>
>--
>libvir-list mailing list
>libvir-list at redhat.com
>https://www.redhat.com/mailman/listinfo/libvir-list
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20160222/3c36671a/attachment-0001.sig>
More information about the libvir-list
mailing list