Re: [PATCH] Fix response for GetCurrentPlayerApplicationSettingValue

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Lucas,

On Mon, Aug 8, 2011 at 5:48 PM, Lucas De Marchi
<lucas.demarchi@xxxxxxxxxxxxxx> wrote:
> According to AVRCP spec there's no player id in response.
> ---
>  parser/avrcp.c |    7 +------
>  1 files changed, 1 insertions(+), 6 deletions(-)
>
> diff --git a/parser/avrcp.c b/parser/avrcp.c
> index 485bb55..f8e4443 100644
> --- a/parser/avrcp.c
> +++ b/parser/avrcp.c
> @@ -612,7 +612,7 @@ response:
>        printf("ValueCount: 0x%02x\n", num);
>
>        for (; num > 0; num--) {
> -               uint8_t attr, player, value;
> +               uint8_t attr, value;
>
>                p_indent(level, frm);
>
> @@ -621,11 +621,6 @@ response:
>
>                p_indent(level, frm);
>
> -               player = get_u8(frm);
> -               printf("Player: 0x%02x\n", player);
> -
> -               p_indent(level, frm);
> -
>                value = get_u8(frm);
>                printf("ValueID: 0x%02x (%s)\n", value,
>                                                value2str(attr, value));
> --
> 1.7.6

Nice catch, I guess I got confused because of the page break.



-- 
Luiz Augusto von Dentz
--
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


[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux