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 <cferg...@redhat.com>
> wrote:
> > On Fri, Dec 02, 2011 at 05:34:15PM +0200, Zeeshan Ali (Khattak) wrote:
> >> From: "Zeeshan Ali (Khattak)" <zeesha...@gnome.org>
> >>
> >> + 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
pgpk71CSCHKn7.pgp
Description: PGP signature
-- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
