Re: [PATCH BlueZ 1/4] AVRCP: move MediaPlayer to adapter object

[Andrei Emeltchenko <andrei.emeltchenko.news@gmail.com>]

Hi guys,

On Thu, Oct 06, 2011 at 11:16:47AM +0300, Johan Hedberg wrote:
> Hi Luiz,
> 
> On Wed, Oct 05, 2011, Luiz Augusto von Dentz wrote:
> > This move the MediaPlayer registration to adapter object on Media
> > interface so we can track players properly.
> > ---
> >  audio/avrcp.c     |  840 +++++++++++------------------------------------------
> >  audio/avrcp.h     |   67 ++++-
> >  audio/device.c    |    3 -
> >  audio/device.h    |    2 -
> >  audio/manager.c   |    7 -
> >  audio/media.c     |  784 +++++++++++++++++++++++++++++++++++++++++++++++++
> >  doc/media-api.txt |  163 +++++++++++
> >  7 files changed, 1179 insertions(+), 687 deletions(-)
> 
> All four patches have been applied. Thanks.

Can it cause this:

$ LANG=en make
make --no-print-directory all-am
  CC     audio/media.o
cc1: warnings being treated as errors
audio/media.c: In function 'get_setting':
audio/media.c:1106:44: error: cast to pointer from integer of different
size
make[1]: *** [audio/media.o] Error 1
make: *** [all] Error 2

in the line below:
        <------8<-------------------------------------------------------------
        |  value = g_hash_table_lookup(mp->settings, GUINT_TO_POINTER(attr));
        <------8<-------------------------------------------------------------

Is patch below OK?

@@ -1103,7 +1103,8 @@ static int get_setting(uint8_t attr, void
*user_data)
 
        DBG("%s", attr_to_str(attr));
 
-       value = g_hash_table_lookup(mp->settings, GUINT_TO_POINTER(attr));
+       value = g_hash_table_lookup(mp->settings,
+                                       GUINT_TO_POINTER((uint32_t) attr));
        if (!value)
                return -EINVAL;
 

Best regards 
Andrei Emeltchenko