[vlc-devel] [PATCH] Subtitles fps support

[Laurent Aimar]

On Mon, May 02, 2011 at 01:42:00PM +0200, Laurent Aimar wrote:
> Hi,
> On Sat, Apr 30, 2011 at 05:27:18PM +0300, Yuval Tze wrote:
> > add DEMUX_SET_FPS demux command
> > handle fps in the subtitle demux
> > add INPUT_CONTROL_SET_SUB_FPS event
> > monitor sub-fps variable changes
> 
>  I don't think DEMUX_SET_FPS/INPUT_CONTROL_SET_SUB_FPS are needed at all. You
> can simply use variable inheritence. It won't allow dynamic changes, but I am
> not sure if it is usefull.
 If dynamic reconfiguration is needed, then implementing it at the input slave
layer (independantly of the type) would be more generic, but I haven't thought
weither or not it would be complicated to do.

-- 
fenrir