[vlc-devel] [PATCH 1/2] input: refactor Control for the next commit

Thomas Guillem thomas at gllm.fr
Tue Sep 4 09:47:59 CEST 2018


On Mon, Sep 3, 2018, at 18:11, Rémi Denis-Courmont wrote:
> Le lundi 3 septembre 2018, 18:56:57 EEST Thomas Guillem a écrit :
> > ---
> >  src/input/input.c | 52 ++++++++++++++++++++++++++---------------------
> >  1 file changed, 29 insertions(+), 23 deletions(-)
> 
> Adding demux_t helpers to vlc_demux.h would be more conventional  ?

Does this mean the following:
 - Add 2 new controls: DEMUX_JUMP_POSITION and DEMUX_JUMP_TIME
 - Implement these controls in demux_vaControlHelper()
 - Modify all demux modules to forward this control to the helper demux_vaControlHelper()

Cons: have to modify all modules

Pros: demux modules can implement JUMP controls themselves, simpler input Control implementation.

> 
> -- 
> Rémi Denis-Courmont
> 
> 
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel


More information about the vlc-devel mailing list