[FFmpeg-devel] [PATCH] lavfi/hue: add process_command callback

Stefano Sabatini stefasab at gmail.com
Mon Aug 27 18:01:19 CEST 2012


On date Monday 2012-08-27 17:33:25 +0200, Jérémy TRAN encoded:
> 2012/8/27 Stefano Sabatini <stefasab at gmail.com>:
> > This is duplicating code in init. Best thing would be to implement
> > "reinit=..." which takes the same arguments as init(), and reuses the
> > same code.
> 
> Yes you’re right.
> But if, for example, the saturation is not set when sending a reinit
> command, it will go back to its default value.
> Is that fine ? Should we modify the init function or move things in
> the config callback and call that function ?

Ideally, change only the parameters which are specified
explicitly. First init call will assume the default parameters.

Note: we should start to document the supported commands, maybe in a
dedicated @subsection.
-- 
FFmpeg = Fundamentalist and Fast Merciless Programmable Eretic Goblin


More information about the ffmpeg-devel mailing list