Re: [libosinfo v3] Add 'installer-reboots' parameter to OsinfoMedia

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Oct 08, 2012 at 07:40:56PM +0300, Zeeshan Ali (Khattak) wrote:
> +
> +    /**
> +     * OsinfoMedia::installer-reboots:
> +     *
> +     * If media is an installer, this property indicates the number of reboots
> +     * the installer takes before installation is complete.
> +     *
> +     * This property is not applicable to media that has no installer. You can
> +     * use #osinfo_media_get_installer (or OsinfoMedia::installer) to check
> +     * that.
> +     *
> +     * Warning: Some media allow you to install from live sessions, in which
> +     * case number of reboots *alone* is not a reliable method for tracking
> +     * installation.
> +     */
> +    pspec = g_param_spec_int ("installer-reboots",
> +                              "InstallerReboots",
> +                              "Number of installer reboots",
> +                              G_MININT,

The actual minimum value is -1 (or 0 ?)

> +                              G_MAXINT,
> +                              1 /* default value */,
> +                              G_PARAM_READWRITE |
> +                              G_PARAM_CONSTRUCT | /* to set default value */

I don't think we need it to be set during construction, do we?

> +                              G_PARAM_STATIC_NAME |
> +                              G_PARAM_STATIC_NICK |
> +                              G_PARAM_STATIC_BLURB);

and this could be _STATIC_STRING


ACK

Christophe

Attachment: pgpPTsWSN7_hL.pgp
Description: PGP signature


[Index of Archives]     [Linux Virtualization]     [KVM Development]     [CentOS Virtualization]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]     [Video 4 Linux]

  Powered by Linux