[PATCH] Add override for system internal property
Patrick Stewart
patstew at gmail.com
Wed Mar 9 19:42:01 PST 2011
On 9 March 2011 18:56, Martin Pitt <martin.pitt at ubuntu.com> wrote:
> Hello Patrick,
>
> (I can't help but asking: you didn't happen to play the captain in a
> famous SciFi TV series, did you? :-) )
I was once assimilated by a hyper advanced cybernetic race, and now
I'm reduced to submitting minor patches to random software projects on
the internet :(
> Patrick Stewart [2011-02-14 18:20 +0000]:
>> + if (g_udev_device_has_property (device->priv->d, "UDISKS_FORCE_SYSTEM_INTERNAL"))
>
> This indeed come up quite a lot, and seems like a good idea. Would you
> mind if we renamed that just to "UDISKS_SYSTEM_INTERNAL", to be a bit
> better aligned with the other properties, and remove the rather
> redundant "force"?
>
>> + {
>> + if (g_udev_device_get_property_as_boolean (device->priv->d, "UDISKS_FORCE_SYSTEM_INTERNAL"))
>> + {
>> + is_system_internal = TRUE;
>> + }
>> + else
>> + {
>> + is_system_internal = FALSE;
>> + }
>
> This can just be
>
> is_system_internal = g_udev_device_get_property_as_boolean (device->priv->d, "UDISKS_FORCE_SYSTEM_INTERNAL");
Duh! Oh well, feel free to change the property to whatever you want,
I'll be happy as long as it's in the next udisks in one form or
another. Thanks.
> Thanks!
>
> Martin
>
> --
> Martin Pitt | http://www.piware.de
> Ubuntu Developer (www.ubuntu.com) | Debian Developer (www.debian.org)
>
More information about the devkit-devel
mailing list