> 
> ACK with one nit:
> 
>> diff --git a/src/conf/storage_conf.c b/src/conf/storage_conf.c
>> index b070448..a8e9876 100644
>> --- a/src/conf/storage_conf.c
>> +++ b/src/conf/storage_conf.c
>> @@ -1150,76 +1150,90 @@ virStoragePoolSourceFormat(virBufferPtr buf,
> 
> --- 8< ---
> 
>> -    virBufferAdjustIndent(&buf, -2);
>> -    virBufferAddLit(&buf, "</pool>\n");
>> +    virBufferAdjustIndent(buf, -2);
>> +    virBufferAddLit(buf, "</pool>\n");
>> +
>> +    return 0;
>> +
>> + error:
>> +    return -1;
> 
> The error label is not necessary - just return -1; directly (unless you
> plan to add some cleanup code there in the near future).
> 
> Jan
> 

Thanks, now pushed.

Erik

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to