Hi Mikel, On Mon, May 27, 2013, Mikel Astiz wrote: > With the adoption of btd_service, profile implementations can benefit > from the provided userdata pointer which can replace the previously > necessary lists to associate services with custom private types. > > The proposed change not only removes boilerplate code but also data > redundancy, presumably making the codebase more robust. > > This patchset addresses the first examples which looked like more > obvious to refactor. Note that some of the profiles were not tested > due to lack of hardware. > > Patch 2/6 fixes an issue that I came across during this work. > > Mikel Astiz (6): > network: Remove duplicated search > network: Fix missing NULL check for given UUID > network: Remove find_connection() > deviceinfo: Use btd_service userdata pointer > scanparam: Use btd_service userdata pointer > input: Use btd_service userdata pointer > > profiles/deviceinfo/deviceinfo.c | 46 ++++++---------------------------------- > profiles/input/device.c | 37 +++++++++----------------------- > profiles/network/connection.c | 38 ++++++++++++--------------------- > profiles/scanparam/scan.c | 36 ++++++------------------------- > 4 files changed, 35 insertions(+), 122 deletions(-) Sorry for the delay in processing this set. It looks fine to me and all patches in it have now been pushed upstream. Johan -- 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