On Thu, Jan 03, 2013 at 02:16:21PM -0500, John Ferlan wrote: > --- > tools/virsh.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/tools/virsh.c b/tools/virsh.c > index 283194a..9f834e4 100644 > --- a/tools/virsh.c > +++ b/tools/virsh.c > @@ -544,7 +544,7 @@ vshTreePrintInternal(vshControl *ctl, > false, indent) < 0) > goto cleanup; > } > - virBufferTrim(indent, " ", -1); > + ignore_value(virBufferTrim(indent, " ", -1)); > > /* If there was no child device, and we're the last in > * a list of devices, then print another blank line */ > @@ -552,7 +552,7 @@ vshTreePrintInternal(vshControl *ctl, > vshPrint(ctl, "%s\n", virBufferCurrentContent(indent)); > > if (!root) > - virBufferTrim(indent, NULL, 2); > + ignore_value(virBufferTrim(indent, NULL, 2)); > ret = 0; > cleanup: > return ret; I must say I don't see the point in the return value for virBufferTrim. I think I'd recommend turning it into a 'void' function Daniel -- |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| |: http://libvirt.org -o- http://virt-manager.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :| -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list