> > 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@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list