John Ferlan wrote: > On 07/21/2014 01:39 PM, Roman Bogorodskiy wrote: > > virStorageBackendLogicalCreateVol contains a piece like: > > > > if (vol->target.path != NULL) { > > /* A target path passed to CreateVol has no meaning */ > > VIR_FREE(vol->target.path); > > } > > > > The 'if' is useless here, but 'syntax-check' doesn't catch that > > because of the comment, so drop the 'if'. > > --- > > src/storage/storage_backend_logical.c | 6 ++---- > > 1 file changed, 2 insertions(+), 4 deletions(-) > > > > ACK Thanks; pushed. Roman Bogorodskiy
Attachment:
pgpbyXZ7WoLAz.pgp
Description: PGP signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list