[virt-tools-list] [libosinfo] Consistent properties enum naming

Daniel P. Berrange berrange at redhat.com
Thu Oct 20 19:25:05 UTC 2011


On Thu, Oct 20, 2011 at 10:16:13PM +0300, Zeeshan Ali (Khattak) wrote:
> From: "Zeeshan Ali (Khattak)" <zeeshanak at gnome.org>
> 
> ---
>  osinfo/osinfo_devicelink.c |    2 +-
>  osinfo/osinfo_entity.c     |   12 ++++++------
>  osinfo/osinfo_os.c         |   10 +++++-----
>  3 files changed, 12 insertions(+), 12 deletions(-)
> 
> diff --git a/osinfo/osinfo_devicelink.c b/osinfo/osinfo_devicelink.c
> index a967ed8..ae197e6 100644
> --- a/osinfo/osinfo_devicelink.c
> +++ b/osinfo/osinfo_devicelink.c
> @@ -43,7 +43,7 @@ struct _OsinfoDeviceLinkPrivate
>      OsinfoDevice *target;
>  };
>  
> -enum OSI_ENTITY_PROPERTIES {
> +enum {
>      PROP_0,
>  
>      PROP_TARGET
> diff --git a/osinfo/osinfo_entity.c b/osinfo/osinfo_entity.c
> index 01eaded..a9af782 100644
> --- a/osinfo/osinfo_entity.c
> +++ b/osinfo/osinfo_entity.c
> @@ -51,10 +51,10 @@ struct _OsinfoEntityPrivate
>  
>  static void osinfo_entity_finalize (GObject *object);
>  
> -enum OSI_ENTITY_PROPERTIES {
> -    OSI_ENTITY_PROP_0,
> +enum {
> +    PROP_0,
>  
> -    OSI_ENTITY_ID,
> +    PROP_ID,
>  };
>  
>  static void
> @@ -67,7 +67,7 @@ osinfo_entity_set_property (GObject      *object,
>  
>      switch (property_id)
>          {
> -        case OSI_ENTITY_ID:
> +        case PROP_ID:
>              g_free(entity->priv->id);
>              entity->priv->id = g_value_dup_string (value);
>              break;
> @@ -88,7 +88,7 @@ osinfo_entity_get_property (GObject    *object,
>  
>      switch (property_id)
>          {
> -        case OSI_ENTITY_ID:
> +        case PROP_ID:
>              g_value_set_string (value, entity->priv->id);
>              break;
>          default:
> @@ -138,7 +138,7 @@ osinfo_entity_class_init (OsinfoEntityClass *klass)
>                                   G_PARAM_STATIC_NICK |
>                                   G_PARAM_STATIC_BLURB);
>      g_object_class_install_property (g_klass,
> -                                     OSI_ENTITY_ID,
> +                                     PROP_ID,
>                                       pspec);
>  
>      g_klass->finalize = osinfo_entity_finalize;
> diff --git a/osinfo/osinfo_os.c b/osinfo/osinfo_os.c
> index bc013aa..8bb9252 100644
> --- a/osinfo/osinfo_os.c
> +++ b/osinfo/osinfo_os.c
> @@ -55,10 +55,10 @@ struct _OsinfoOsDeviceLink {
>      gchar *driver;
>  };
>  
> -enum OSI_OS_PROPERTIES {
> -    OSI_OS_PROP_0,
> +enum {
> +    PROP_0,
>  
> -    OSI_OS_FAMILY,
> +    PROP_FAMILY,
>  };
>  
>  static void osinfo_os_finalize (GObject *object);
> @@ -78,7 +78,7 @@ osinfo_os_get_property (GObject    *object,
>  
>      switch (property_id)
>          {
> -        case OSI_OS_FAMILY:
> +        case PROP_FAMILY:
>              g_value_set_string (value,
>                                  osinfo_entity_get_param_value (entity,
>                                                                 "family"));
> @@ -129,7 +129,7 @@ osinfo_os_class_init (OsinfoOsClass *klass)
>                                   G_PARAM_STATIC_NICK |
>                                   G_PARAM_STATIC_BLURB);
>      g_object_class_install_property (g_klass,
> -                                     OSI_OS_FAMILY,
> +                                     PROP_FAMILY,
>                                       pspec);
>  }

ACK


Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|




More information about the virt-tools-list mailing list