Re: [FFmpeg-devel] [PATCH] ffserver_config: improve AVOption handing

2014-11-02 Thread Lukasz Marek
On 02.11.2014 22:37, Reynaldo H. Verdejo Pinochet wrote: If you are always returning 0 then this func should probably be declared static void instead? I left it as int for future, but probably no reason for that, so changed. [..] diff --git a/ffserver_config.h b/ffserver_config.h index f29f07

Re: [FFmpeg-devel] [PATCH] ffserver_config: improve AVOption handing

2014-11-02 Thread Reynaldo H. Verdejo Pinochet
Hi On 11/01/2014 03:33 PM, Lukasz Marek wrote: > [..] > @@ -517,7 +538,6 @@ static int ffserver_parse_config_feed(FFServerConfig > *config, const char *cmd, c > static int ffserver_apply_stream_config(AVCodecContext *enc, const > AVDictionary *conf, AVDictionary **opts) > { > AVDictionar

[FFmpeg-devel] [PATCH] ffserver_config: improve AVOption handing

2014-11-01 Thread Lukasz Marek
AVOption are checked right after found in config file. It allows to report exact line in config file. Options provided more than once are threated as errors. It also fixes flag options handing. Flags may occur more than once in config file. Signed-off-by: Lukasz Marek --- ffserver_config.c | 4