[virt-tools-list] [libosinfo v2 1/3] Add missing documentation

Christophe Fergeau cfergeau at redhat.com
Mon Dec 17 09:47:48 UTC 2012


On Mon, Dec 17, 2012 at 07:24:34AM -0200, Fabiano Fidêncio wrote:
> Annotations for:
> - osinfo_install_script_generate_finish()
> - osinfo_install_script_generate_output_finish()

and osinfo_media_get_os and osinfo_media_set_languages (thanks for adding
these, I had totally overlooked that). Did you get warnings about the
missing _set_languages annotation? the symbol is not exported.

Christophe

> ---
>  osinfo/osinfo_install_script.c | 16 ++++++++++++++++
>  osinfo/osinfo_media.c          | 15 +++++++++++++--
>  osinfo/osinfo_os.c             |  8 +++++++-
>  3 files changed, 36 insertions(+), 3 deletions(-)
> 
> diff --git a/osinfo/osinfo_install_script.c b/osinfo/osinfo_install_script.c
> index 2229843..da92850 100644
> --- a/osinfo/osinfo_install_script.c
> +++ b/osinfo/osinfo_install_script.c
> @@ -889,6 +889,14 @@ static gpointer osinfo_install_script_generate_finish_common(OsinfoInstallScript
>      return g_simple_async_result_get_op_res_gpointer(simple);
>  }
>  
> +/**
> + * osinfo_install_script_generate_finish:
> + * @script: the install script
> + * @res:    a #GAsyncResult
> + * @error:  The location where to store any error, or NULL
> + *
> + * Returns: (transfer full): the generated script, or NULL on error
> + */
>  gchar *osinfo_install_script_generate_finish(OsinfoInstallScript *script,
>                                               GAsyncResult *res,
>                                               GError **error)
> @@ -898,6 +906,14 @@ gchar *osinfo_install_script_generate_finish(OsinfoInstallScript *script,
>                                                          error);
>  }
>  
> +/**
> + * osinfo_install_script_generate_output_finish:
> + * @script: the install script
> + * @res:    a #GAsyncResult
> + * @error:  The location where to store any error, or NULL
> + *
> + * Returns: (transfer full): a file containing the script, or NULL on error
> + */
>  GFile *osinfo_install_script_generate_output_finish(OsinfoInstallScript *script,
>                                                      GAsyncResult *res,
>                                                      GError **error)
> diff --git a/osinfo/osinfo_media.c b/osinfo/osinfo_media.c
> index 0bb29fa..2821e7f 100644
> --- a/osinfo/osinfo_media.c
> +++ b/osinfo/osinfo_media.c
> @@ -1114,8 +1114,11 @@ gint osinfo_media_get_installer_reboots(OsinfoMedia *media)
>              (OSINFO_ENTITY(media), OSINFO_MEDIA_PROP_INSTALLER_REBOOTS, 1);
>  }
>  
> -/* osinfo_media_get_os:
> - * Returns: (transfer full):
> +/**
> + * osinfo_media_get_os:
> + * @media: an #OsinfoMedia instance
> + *
> + * Returns: (transfer full): the operating system, or NULL
>   */
>  OsinfoOs *osinfo_media_get_os(OsinfoMedia *media)
>  {
> @@ -1156,6 +1159,14 @@ GList *osinfo_media_get_languages(OsinfoMedia *media)
>      return osinfo_entity_get_param_value_list(OSINFO_ENTITY(media), OSINFO_MEDIA_PROP_LANG);
>  }
>  
> +/**
> + * osinfo_media_set_languages:
> + * @media: an #OsinfoMedia instance
> + * @languages: (element-type utf8): a #GList containing the list of the UI
> + * languages this media supports.
> + *
> + * Sets the #OSINFO_MEDIA_PROP_LANG parameter
> + */
>  void osinfo_media_set_languages(OsinfoMedia *media, GList *languages)
>  {
>      GList *it;
> diff --git a/osinfo/osinfo_os.c b/osinfo/osinfo_os.c
> index 2be7f64..786e596 100644
> --- a/osinfo/osinfo_os.c
> +++ b/osinfo/osinfo_os.c
> @@ -538,7 +538,13 @@ void osinfo_os_add_recommended_resources(OsinfoOs *os,
>                      OSINFO_ENTITY(resources));
>  }
>  
> -
> +/**
> + * osinfo_os_find_install_script:
> + * @os:      an operating system
> + * @profile: the install script profile
> + *
> + * Returns: (transfer full): A new #OsinfoInstallScript for the @os @profile
> + */
>  OsinfoInstallScript *osinfo_os_find_install_script(OsinfoOs *os, const gchar *profile)
>  {
>      g_return_val_if_fail(OSINFO_IS_OS(os), NULL);
> -- 
> 1.8.0.2
> 
> _______________________________________________
> virt-tools-list mailing list
> virt-tools-list at redhat.com
> https://www.redhat.com/mailman/listinfo/virt-tools-list
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/virt-tools-list/attachments/20121217/5e81eb16/attachment.sig>


More information about the virt-tools-list mailing list