[vlc-devel] [PATCH 2/3] video_output/mmal: Add mmal based video output plugin

[Julian Scheel]

Am 05/04/14 22:04, schrieb Rémi Denis-Courmont:
> Le samedi 5 avril 2014, 20:15:49 Julian Scheel a écrit :
>>>> +    vc_tv_unregister_callback(tvservice_cb);
>>>
>>> Is that reference counted per callback? If not, it will crash on multiple
>>> video tracks.
>>
>> vc_tv callbacks are tracked as a list in the vcos implementation so
>> multiple callbacks can be registered and unregistered independently. So
>> I don't see a problem here.
>
> You have:
>
> +    vc_tv_register_callback(tvservice_cb, vd);
>
> but:
>
> +    vc_tv_unregister_callback(tvservice_cb);
>
> How do you know which 'vd' to remove from the list if there are more than one?

Ah yes, you're of course right about that one. Will think about a way to 
deal with this.