[FFmpeg-devel] [PATCH 1/2] ffmpeg: don't reconfigure terminal if we're not taking input from stdin

2016-09-10 Thread Rodger Combs
--- ffmpeg.c | 4 +--- ffmpeg_opt.c | 3 +++ 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/ffmpeg.c b/ffmpeg.c index d858407..111d844 100644 --- a/ffmpeg.c +++ b/ffmpeg.c @@ -366,7 +366,7 @@ static BOOL WINAPI CtrlHandler(DWORD fdwCtrlType) void term_init(void) { #if

Re: [FFmpeg-devel] [PATCH 1/2] ffmpeg: don't reconfigure terminal if we're not taking input from stdin

2016-09-10 Thread Nicolas George
Le quintidi 25 fructidor, an CCXXIV, Rodger Combs a écrit : > --- > ffmpeg.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/ffmpeg.c b/ffmpeg.c > index d858407..08a7a3d 100644 > --- a/ffmpeg.c > +++ b/ffmpeg.c > @@ -366,7 +366,7 @@ static BOOL WINAPI

[FFmpeg-devel] [PATCH 1/2] ffmpeg: don't reconfigure terminal if we're not taking input from stdin

2016-09-10 Thread Rodger Combs
--- ffmpeg.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/ffmpeg.c b/ffmpeg.c index d858407..08a7a3d 100644 --- a/ffmpeg.c +++ b/ffmpeg.c @@ -366,7 +366,7 @@ static BOOL WINAPI CtrlHandler(DWORD fdwCtrlType) void term_init(void) { #if HAVE_TERMIOS_H -

Re: [FFmpeg-devel] [PATCH 1/2] ffmpeg: don't reconfigure terminal if we're not taking input from stdin

2016-09-10 Thread Nicolas George
Le tridi 23 fructidor, an CCXXIV, Rodger Combs a écrit : > Agreed in principle; I did it this way because I'm not entirely sure if > there would be negative consequences to parsing options (and thus opening > I/O) before setting up signal handlers. If you think that's not an issue, > I'll make the

Re: [FFmpeg-devel] [PATCH 1/2] ffmpeg: don't reconfigure terminal if we're not taking input from stdin

2016-09-08 Thread Rodger Combs
> On Sep 8, 2016, at 07:49, Nicolas George wrote: > > Le duodi 22 fructidor, an CCXXIV, Rodger Combs a écrit : >> --- >> ffmpeg.c | 6 +- >> 1 file changed, 5 insertions(+), 1 deletion(-) >> >> diff --git a/ffmpeg.c b/ffmpeg.c >> index d858407..1d793fe 100644 >> ---

Re: [FFmpeg-devel] [PATCH 1/2] ffmpeg: don't reconfigure terminal if we're not taking input from stdin

2016-09-08 Thread Nicolas George
Le duodi 22 fructidor, an CCXXIV, Rodger Combs a écrit : > --- > ffmpeg.c | 6 +- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git a/ffmpeg.c b/ffmpeg.c > index d858407..1d793fe 100644 > --- a/ffmpeg.c > +++ b/ffmpeg.c > @@ -366,7 +366,7 @@ static BOOL WINAPI CtrlHandler(DWORD

Re: [FFmpeg-devel] [PATCH 1/2] ffmpeg: don't reconfigure terminal if we're not taking input from stdin

2016-09-08 Thread Michael Niedermayer
On Wed, Sep 07, 2016 at 07:17:59PM -0500, Rodger Combs wrote: > --- > ffmpeg.c | 6 +- > 1 file changed, 5 insertions(+), 1 deletion(-) ill leave these 2 patches for nicolas to review ... thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Asymptotically

[FFmpeg-devel] [PATCH 1/2] ffmpeg: don't reconfigure terminal if we're not taking input from stdin

2016-09-07 Thread Rodger Combs
--- ffmpeg.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/ffmpeg.c b/ffmpeg.c index d858407..1d793fe 100644 --- a/ffmpeg.c +++ b/ffmpeg.c @@ -366,7 +366,7 @@ static BOOL WINAPI CtrlHandler(DWORD fdwCtrlType) void term_init(void) { #if HAVE_TERMIOS_H -