[virt-tools-list] [libosinfo v3 2/3] docs: Fix typos

Christophe Fergeau cfergeau at redhat.com
Tue Dec 18 09:46:53 UTC 2012


Hey,


On Tue, Dec 18, 2012 at 01:20:19AM -0200, Fabiano Fidêncio wrote:
> - a -> an
> - add a missing word (elements)
> - references: @param: an Osinfo* -> @param: an #Osinfo*

Looks good to me now, thanks for doing that. 2 comments below.

> @@ -171,7 +171,7 @@ osinfo_avatar_format_init (OsinfoAvatarFormat *avatar)
>  /**
>   * osinfo_avatar_format_new:
>   *
> - * Construct a new user avatar file to a #OsinfoInstallScript.
> + * Construct a new user avatar file to an #OsinfoInstallScript.

Shouldn't it be:
Construct a new user avatar file *for* an #OsinfoInstallScript.


> --- a/osinfo/osinfo_install_script.c
> +++ b/osinfo/osinfo_install_script.c
> @@ -507,7 +507,7 @@ osinfo_install_script_set_avatar_format(OsinfoInstallScript *script,
>  }
>  
>  /**
> - * osinfo_install_script_get_avatar_format
> + * osinfo_install_script_get_avatar_format:


This change is unrelated to all the other ones, so it would be nicer to
have it separate, but I can live with it staying in this commit.

Christophe
-------------- 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/20121218/1cc4a101/attachment.sig>


More information about the virt-tools-list mailing list