Hi Johan,
On Mon, Feb 4, 2013 at 2:30 AM, Johan Hedberg <johan.hedberg@xxxxxxxxx> wrote:
> Hi Luiz,
>
> On Sun, Feb 03, 2013, Luiz Augusto von Dentz wrote:
>> --- a/profiles/audio/player.c
>> +++ b/profiles/audio/player.c
>> @@ -67,7 +67,7 @@ struct media_player {
>> char *name; /* Player name */
>> char *type; /* Player type */
>> char *subtype; /* Player subtype */
>> - uint64_t features[2]; /* Player features */
>> + uint8_t *features; /* Player features */
>
> Why not just have this as features[16] here as well so you don't need to
> do g_memdup and g_free?
I was planning to have a NULL check if the features are valid.
--
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
