[libvirt] [libvirt-glib] [PATCH 1/4] GVirDomain: Fix some doc comments of the snapshot API

Christophe Fergeau cfergeau at redhat.com
Mon Aug 18 09:58:27 UTC 2014


Hey,

On Fri, Aug 15, 2014 at 12:05:44PM +0200, Timm Bäder wrote:
> Fixes some warnings from g-ir-scanner.
> ---
>  libvirt-gobject/libvirt-gobject-domain.c | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/libvirt-gobject/libvirt-gobject-domain.c b/libvirt-gobject/libvirt-gobject-domain.c
> index feac6f0..aa2a170 100644
> --- a/libvirt-gobject/libvirt-gobject-domain.c
> +++ b/libvirt-gobject/libvirt-gobject-domain.c
> @@ -1529,10 +1529,10 @@ gvir_domain_create_snapshot(GVirDomain *dom,
>   * gvir_domain_fetch_snapshots:
>   * @dom: The domain
>   * @list_flags: bitwise-OR of #GVirDomainSnapshotListFlags
> - * @cancellable: (allow-none)(transfer-none): cancellation object
> - * @error: (allow-none): Place-holder for error or NULL
> + * @cancellable: (allow-none) (transfer none): cancellation object
> + * @error: (allow-none): Place-holder for error or %NULL
>   *
> - * Returns: TRUE on success, FALSE otherwise.
> + * Returns: %TRUE on success, %FALSE otherwise.
>   */
>  gboolean gvir_domain_fetch_snapshots(GVirDomain *dom,
>                                       guint list_flags,
> @@ -1647,10 +1647,10 @@ static void _fetch_snapshots_async_thread(GTask *task,
>  
>  
>  /**
> - *
> + * gvir_domain_fetch_snapshots_async:
>   * @dom: The domain
>   * @list_flags: bitwise-OR of #GVirDomainSnapshotListFlags
> - * @cancellable: (allow-none)(transfer-none): cancellation object
> + * @cancellable: (allow-none) (transfer none): cancellation object

Is g-ir-scanner complaining about the lack of space? or just about
'transfer-none'? Just being curious, patch looks good to me, ACK. For
future commits, c&p-ing the g-ir-scanner warnings that you are fixing in
the commit log can be useful)

Christophe
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20140818/bf2a502c/attachment-0001.sig>


More information about the libvir-list mailing list