[FFmpeg-devel] [PATCH 1/3] ffmpeg: set tty mode and signals after parsing options.

[Michael Niedermayer]

On Wed, Mar 16, 2016 at 11:01:02PM +0100, Nicolas George wrote:
> Le septidi 27 ventôse, an CCXXIV, Michael Niedermayer a écrit :
> > iam not saying its necessary (or that its not), just that its done
> > currently and would not be done anymore in the future.
> 
> But what? What is done now and not with this patch. I do now know what
> cleanup you are referring to.

after the patch a ctrl-c simply kills the process
before teh patch a crtl-c will cause the input open to error out
and all subsequent code of the affected input and other inputs
to execute.

One difference here is that with multiple inputs before the patch
you will get an error message telling you which was stuck

ffmpeg -i http://somipthattimesout -i test.avi -r 30 -f mpeg

Trailing options were found on the commandline.
http://somipthattimesout: Interrupted system call
Exiting normally, received signal 2.

vs.

Trailing options were found on the commandline.
^C



[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

If you think the mosad wants you dead since a long time then you are either
wrong or dead since a long time.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160316/2a7fe18a/attachment.sig>