[Libosinfo] [PATCHv4 08/11] Add osinfo_install_config_new_for_script

Christophe Fergeau cfergeau at redhat.com
Fri Dec 21 09:53:06 UTC 2012


On Fri, Dec 21, 2012 at 02:21:57AM +0200, Zeeshan Ali (Khattak) wrote:
> > diff --git a/osinfo/osinfo_install_config.c b/osinfo/osinfo_install_config.c
> > index 4c42746..e61bfb6 100644
> > --- a/osinfo/osinfo_install_config.c
> > +++ b/osinfo/osinfo_install_config.c
> > @@ -189,9 +189,39 @@ osinfo_install_config_init (OsinfoInstallConfig *config)
> >   */
> >  OsinfoInstallConfig *osinfo_install_config_new(const gchar *id)
> >  {
> > -    return g_object_new(OSINFO_TYPE_INSTALL_CONFIG,
> > -                        "id", id,
> > -                        NULL);
> > +    return osinfo_install_config_new_for_script(id, NULL);
> > +}
> > +
> > +
> > +/**
> > + * osinfo_install_config_new_for_script:
> > + * @id: the unique identifier
> > + * @script: the #OsinfoInstallScript we are creating the configuration for
> > + *
> > + * Construct a new install configuration associated with @script.
> > + * OsinfoInstallConfig:config-params will contain the
> > + * #OsinfoInstallConfigParamList describing the parameters that can be set
> > + * on the config object when creating a configuration for @script. See
> > + * osinfo_install_config_new() for a description of the default values that
> > + * will be set on the newly created #OsinfoInstallConfig.
> > + *
> > + * Returns: (transfer full): an install configuration
> > + */
> > +OsinfoInstallConfig *osinfo_install_config_new_for_script(const gchar *id,
> > +                                                          OsinfoInstallScript *script)
> > +{
> > +    OsinfoInstallConfigParamList *params = NULL;
> > +    OsinfoInstallConfig *config;
> > +
> > +    if (script != NULL)
> > +        params = osinfo_install_script_get_config_params(script);
> > +
> 
> I don't think it makes sense to have script argument nullable here but
> no strong feelings. If you disagree, you want to add gir annotation in
> the doc comment. Either way, you can just change it before pushing.

osinfo_install_config_new is calling it with a NULL script. Given that
without that call, it's a one-liner, I'll change the code to forbid NULL
scripts.

Christophe

> 
> -- 
> Regards,
> 
> Zeeshan Ali (Khattak)
> FSF member#5124
-------------- 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/libosinfo/attachments/20121221/258902a9/attachment.sig>


More information about the Libosinfo mailing list