Re: [FFmpeg-devel] [PATCH 05/11] ffserver: allow skip setting defaults

2014-11-23 Thread Reynaldo H. Verdejo Pinochet
Hi On 11/21/2014 09:13 PM, Lukasz Marek wrote: [...] @@ -1032,6 +1059,14 @@ static int ffserver_parse_config_stream(FFServerConfig *config, const char *cmd, } else if (!av_strcasecmp(cmd, File) || !av_strcasecmp(cmd, ReadOnlyFile)) { ffserver_get_arg(stream-feed_filename,

Re: [FFmpeg-devel] [PATCH 05/11] ffserver: allow skip setting defaults

2014-11-21 Thread Lukasz Marek
On 21.11.2014 16:31, Reynaldo H. Verdejo Pinochet wrote: Hi On 11/20/2014 10:09 PM, Lukasz Marek wrote: On 18.11.2014 23:25, Reynaldo H. Verdejo Pinochet wrote: [..] I do think undefined behavior should be avoided if possible without too much hassle, so if we go with the former I would

Re: [FFmpeg-devel] [PATCH 05/11] ffserver: allow skip setting defaults

2014-11-20 Thread Lukasz Marek
On 18.11.2014 23:25, Reynaldo H. Verdejo Pinochet wrote: Hi On 11/18/2014 06:54 PM, Nicolas George wrote: L'octidi 28 brumaire, an CCXXIII, Reynaldo H. Verdejo Pinochet a écrit : I think I commented about this before but having yesvar novar options seems redundant. Having var = yes or no if

Re: [FFmpeg-devel] [PATCH 05/11] ffserver: allow skip setting defaults

2014-11-18 Thread Reynaldo H. Verdejo Pinochet
Hi On 11/16/2014 10:46 PM, Lukasz Marek wrote: [..] diff --git a/doc/ffserver.texi b/doc/ffserver.texi index b7c5b6a..d3ff13e 100644 --- a/doc/ffserver.texi +++ b/doc/ffserver.texi @@ -408,6 +408,11 @@ ignored, and the log is written to standard output. Set no-daemon mode. This option is

Re: [FFmpeg-devel] [PATCH 05/11] ffserver: allow skip setting defaults

2014-11-18 Thread Nicolas George
L'octidi 28 brumaire, an CCXXIII, Reynaldo H. Verdejo Pinochet a écrit : I think I commented about this before but having yesvar novar options seems redundant. Having var = yes or no if absent (or the other way around depending on the intended default) seems less cumbersome and should

Re: [FFmpeg-devel] [PATCH 05/11] ffserver: allow skip setting defaults

2014-11-18 Thread Reynaldo H. Verdejo Pinochet
Hi On 11/18/2014 06:54 PM, Nicolas George wrote: L'octidi 28 brumaire, an CCXXIII, Reynaldo H. Verdejo Pinochet a écrit : I think I commented about this before but having yesvar novar options seems redundant. Having var = yes or no if absent (or the other way around depending on the