[vlc-devel] [PATCH 2/2] libvlc media list player: add getter player instance

Felix Paul Kühne fkuehne at videolan.org
Thu Sep 10 18:13:46 CEST 2015


> On 10 Sep 2015, at 18:10, Rémi Denis-Courmont <remi at remlab.net> wrote:
> 
> Subject does not parse

Indeed :D

>> + * \return p_mi media player instance
> 
> ??

*cough*
> 
>> + * \note the caller is responsible for releasing the player instance
> 
> Typography.

OK.

>> +    if (p_mlp->p_mi == NULL)
>> +        return NULL;
> 
> How do you trigger this?

Useless sanity check indeed, will drop.

Thanks for the quick review!

Felix



More information about the vlc-devel mailing list