On Sat, Dec 03, 2011 at 12:11:04AM +0200, Zeeshan Ali (Khattak) wrote:
> On Fri, Dec 2, 2011 at 6:34 PM, Christophe Fergeau <cfergeau@xxxxxxxxxx> wrote:
> > On Fri, Dec 02, 2011 at 05:34:15PM +0200, Zeeshan Ali (Khattak) wrote:
> >> From: "Zeeshan Ali (Khattak)" <zeeshanak@xxxxxxxxx>
> >>
> >> + if (virStorageVolDelete(vol->priv->handle, flags) < 0) {
> >> + g_set_error (err,
> >> + GVIR_STORAGE_VOL_ERROR,
> >> + 0,
> >> + "Unable to delete storage volume");
> >
> > I think gvir_error_new_literal should be used here
>
> g_set_error() checks if error is NULL and doesn't set the error if so.
I know, but gvir_error_new_literal automatically appends the error message
from libvirt so this should be used here. But yeah for now you have to
check for NULL errors by yourself. Or you can wait until the
gvir_set_error_literal patches go in.
Christophe
Attachment:
pgpj1aBNhTpo1.pgp
Description: PGP signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list