Re: [PATCH 1/2] docs: Fix couple of warnings

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

 



On Mon, Jan 16, 2017 at 9:45 AM, Christophe Fergeau <cfergeau@xxxxxxxxxx> wrote:
> On Mon, Jan 16, 2017 at 09:06:50AM +0100, Michal Privoznik wrote:
>> Firstly, two functions
>> (osinfo_install_script_has_config_param_{name,list}) don't
>> document their arguments.  Secondly, gtk-doc requires enum
>> documentation to be in one block with no empty lines in it.
>> Then at last, private enums are marked as <private> not <skip>.
>
> Looks good to me..
> Acked-by: Christophe Fergeau <cfergeau@xxxxxxxxxx>

I've noticed that the 2 patches of this series ended up never being pushed.
Just did it now.

>
>>
>> Signed-off-by: Michal Privoznik <mprivozn@xxxxxxxxxx>
>> ---
>>  osinfo/osinfo_install_script.c  | 2 ++
>>  osinfo/osinfo_install_script.h  | 4 ----
>>  osinfo/osinfo_product_private.h | 2 +-
>>  3 files changed, 3 insertions(+), 5 deletions(-)
>>
>> diff --git a/osinfo/osinfo_install_script.c b/osinfo/osinfo_install_script.c
>> index 9206101..8fcde0f 100644
>> --- a/osinfo/osinfo_install_script.c
>> +++ b/osinfo/osinfo_install_script.c
>> @@ -283,6 +283,7 @@ gboolean osinfo_install_script_has_config_param_name(const OsinfoInstallScript *
>>
>>  /**
>>   * osinfo_install_script_get_config_param_list:
>> + * @script: the install script
>>   *
>>   * Get the list of valid config parameters for @script.
>>   *
>> @@ -297,6 +298,7 @@ GList *osinfo_install_script_get_config_param_list(const OsinfoInstallScript *sc
>>
>>  /**
>>   * osinfo_install_script_get_config_params:
>> + * @script: the install script
>>   *
>>   * Get the list of valid config parameters for @script.
>>   *
>> diff --git a/osinfo/osinfo_install_script.h b/osinfo/osinfo_install_script.h
>> index c494192..61bf0e4 100644
>> --- a/osinfo/osinfo_install_script.h
>> +++ b/osinfo/osinfo_install_script.h
>> @@ -93,17 +93,14 @@ typedef enum {
>>
>>  /**
>>   * OsinfoDeviceDriverSigningReq:
>> - *
>>   * @OSINFO_DEVICE_DRIVER_SIGNING_REQ_NONE: Script do not require device drivers
>>   * to be signed.
>> - *
>>   * @OSINFO_DEVICE_DRIVER_SIGNING_REQ_STRICT: Script must only be given signed
>>   * device drivers. Some scripts will allow overriding this requirement through
>>   * #osinfo_install_config_set_driver_signing function. You can query if a
>>   * script supports this by checking if
>>   * #OSINFO_INSTALL_CONFIG_PROP_DRIVER_SIGNING configuration parameter is used
>>   * by the script in question (or other scripts in the same profile).
>> - *
>>   * @OSINFO_DEVICE_DRIVER_SIGNING_REQ_WARN: A warning will be issued by OS
>>   * installer if device drivers are not signed and most probably require user
>>   * input (and thus breaking unattended installation). See
>> @@ -118,7 +115,6 @@ typedef enum {
>>
>>  /**
>>   * OsinfoInstallScriptInjectionMethod:
>> - *
>>   * @OSINFO_INSTALL_SCRIPT_INJECTION_METHOD_CDROM: Support injection of the
>>   * installation script trough a CD-ROM.
>>   * @OSINFO_INSTALL_SCRIPT_INJECTION_METHOD_DISK: Support injection of the
>> diff --git a/osinfo/osinfo_product_private.h b/osinfo/osinfo_product_private.h
>> index c4ecdd5..49cf0a2 100644
>> --- a/osinfo/osinfo_product_private.h
>> +++ b/osinfo/osinfo_product_private.h
>> @@ -29,7 +29,7 @@
>>
>>  typedef void (*OsinfoProductForeach)(OsinfoProduct *product, gpointer user_data);
>>
>> -typedef enum { /*< skip >*/
>> +typedef enum { /*< private >*/
>>      OSINFO_PRODUCT_FOREACH_FLAG_DERIVES_FROM = 1 << 0,
>>      OSINFO_PRODUCT_FOREACH_FLAG_UPGRADES = 1 << 1,
>>      OSINFO_PRODUCT_FOREACH_FLAG_CLONES = 1 << 2,
>> --
>> 2.11.0
>>
>> _______________________________________________
>> Libosinfo mailing list
>> Libosinfo@xxxxxxxxxx
>> https://www.redhat.com/mailman/listinfo/libosinfo
>
> _______________________________________________
> Libosinfo mailing list
> Libosinfo@xxxxxxxxxx
> https://www.redhat.com/mailman/listinfo/libosinfo
>



-- 
Fabiano Fidêncio

_______________________________________________
Libosinfo mailing list
Libosinfo@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libosinfo




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Fedora Users]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]

  Powered by Linux