Re: [PATCH BlueZ v0 0/6] Exploit btd_service's userdata pointer

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