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

[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?

-- 
Rémi Denis-Courmont
http://www.remlab.net/