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
Attachment:
pgp8CK_DuDFvC.pgp
Description: PGP signature
_______________________________________________ virt-tools-list mailing list virt-tools-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/virt-tools-list