Hi, On Wed, Aug 11, 2010 at 4:27 PM, Johan Hedberg <johan.hedberg@xxxxxxxxx> wrote: > Hi Luiz, > > On Wed, Aug 11, 2010, Luiz Augusto von Dentz wrote: >> --- a/src/dbus-hci.c >> +++ b/src/dbus-hci.c >> @@ -746,8 +746,8 @@ int hcid_dbus_link_key_notify(bdaddr_t *local, bdaddr_t *peer, >> else if (!bonding && old_key_type == 0xff) >> hcid_dbus_bonding_process_complete(local, peer, 0); >> >> - if (temporary) >> - device_set_temporary(device, TRUE); >> + if (device_is_temporary(device) != temporary) >> + device_set_temporary(device, temporary); >> >> return 0; > > I'd prefer if you just call device_set_temporary(device, temporary) > directly. That'd save you the if-check and a call to device_is_temporary. Fair enough, I will change it and send a new patch in a few minutes. -- Luiz Augusto von Dentz Computer Engineer -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
