[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v5 7/8] qom: add a object_property_add_enum help
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH v5 7/8] qom: add a object_property_add_enum helper method |
Date: |
Fri, 19 Jun 2015 10:12:42 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.7.0 |
On 05/27/2015 09:07 AM, Daniel P. Berrange wrote:
> A QOM property can be parsed as enum using the visit_type_enum()
> helper method, but this forces callers to use the more complex
> generic object_property_add() method when registering it. It
> also requires that users of that object have access to the
> string map when they want to read the property value.
>
> This patch introduces a specialized object_property_add_enum()
> method which simplifies the use of enum properties, so the
> setters/getters directly get passed the int value.
>
> typedef enum {
> /**
> + * object_property_add_enum:
> + * @obj: the object to add a property to
> + * @name: the name of the property
> + * @typename: the name of the enum data type
> + * @get: the getter or %NULL if the property is write-only.
> + * @set: the setter or %NULL if the property is read-only
> + * @errp: if an error occurs, a pointer to an area to store the error
> + *
> + * Add a enum property using getters/setters. This function will add a
s/a enum/an enum/
> +static void property_release_enum(Object *obj, const char *name,
> + void *opaque)
> +{
> + EnumProperty *prop = opaque;
> + g_free(prop);
Could also be written as g_free(opaque), but that's a little less
readable, so I don't see a point in changing it.
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
- Re: [Qemu-devel] [PATCH v5 7/8] qom: add a object_property_add_enum helper method,
Eric Blake <=