On Wed, 2012-07-25 at 16:29 +0200, Mikel Astiz wrote: > +static int parse_device_property(pa_bluetooth_discovery *y, pa_bluetooth_device *d, DBusMessageIter *i) { > + const char *key; > + DBusMessageIter variant_i; > + > + pa_assert(y); > + pa_assert(d); > + > + key = check_variant_property(i); > + if (key == NULL) > + return -1; No tabs, please! (There's a tab used also in the other checks.) Otherwise, looks good, thanks! I've pushed this now. -- Tanu