[vlc-devel] [PATCH 2/5] input: add DEMUX_VOUT_CHANGED

Rémi Denis-Courmont remi at remlab.net
Fri Jul 13 18:32:42 CEST 2018


Le perjantaina 13. heinäkuuta 2018, 12.34.38 EEST Thomas Guillem a écrit :
> This control allow to propagate vout events from the decoder to the demux.

To me, it feels backward to have the demuxer track the video output, 
regardless if it's done with an input thread event or a demuxer control.

Mouse events should be dispatched like key events already are, e.g.:

const vlc_mouse_t *state;

if (demux_Control(demux, DEMUX_NAV_MOUSE, state) != VLC_SUCCESS)
    /* Generic mouse processing, fullscreen toggle or whatever */;
else
    /* Mouse event eaten by demuxer. Drop. */;

And otherwise, the demuxer really should live the video output(s) alone IMO.

-- 
レミ・デニ-クールモン
http://www.remlab.net/





More information about the vlc-devel mailing list