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

2014-11-16 Thread Lukasz Marek
Signed-off-by: Lukasz Marek --- doc/ffserver.texi | 10 ++ ffserver.c| 1 + ffserver_config.c | 30 ++ ffserver_config.h | 2 ++ 4 files changed, 43 insertions(+) diff --git a/doc/ffserver.texi b/doc/ffserver.texi index b7c5b6a..d3ff13e 100644 --- a/

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 optio

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 sim

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 th

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-21 Thread Reynaldo H. Verdejo Pinochet
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 >> appreciate doc entries specifying the options preceden

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 appreci

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_fi