On Sat, Sep 21, 2013 at 6:12 AM, Tanu Kaskinen <tanu.kaskinen at linux.intel.com> wrote: > On Wed, 2013-09-18 at 16:17 -0500, jprvita at gmail.com wrote: >> static void parse_interfaces_and_properties(pa_bluetooth_discovery *y, DBusMessageIter *dict_i) { >> DBusMessageIter element_i; >> const char *path; >> @@ -415,7 +474,8 @@ static void parse_interfaces_and_properties(pa_bluetooth_discovery *y, DBusMessa >> >> pa_log_debug("Adapter %s found", path); >> >> - /* TODO: parse adapter properties and register endpoints */ >> + parse_adapter_properties(a, &iface_i, false); > > If parsing fails, or if the Address property is missing, the adapter > should be marked as invalid. > We don't need to add a adapter_info_valid field, just not registering as an endpoint with that adapter should be enough. -- Jo?o Paulo Rechi Vita http://about.me/jprvita