[FFmpeg-devel] [PATCH] Check for signal.h, conditionally compile parts of ffmpeg.c

Martin Storsjö martin
Tue Jul 14 23:04:49 CEST 2009


On Tue, 14 Jul 2009, M?ns Rullg?rd wrote:

> There is no need for this to variable to be sig_atomic_t in the first
> place.  The worst that could possibly happen is the encoding loop
> doing one extra iteration, and that only if an 'int' cannot be read in
> a single instruction, which is exceedingly unlikely to be the case on
> any platform we'll ever support.

Ok, updated the patch series to start by changing this to an int.

// Martin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Change-received_sigterm-from-sig_atomic_t-into-int-t.patch
Type: text/x-diff
Size: 398 bytes
Desc: 
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090715/6219b392/attachment.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-Check-for-signal.h.patch
Type: text/x-diff
Size: 558 bytes
Desc: 
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090715/6219b392/attachment-0001.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0003-Check-for-HAVE_SIGNAL_H-in-ffmpeg.c.patch
Type: text/x-diff
Size: 1143 bytes
Desc: 
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090715/6219b392/attachment-0002.patch>



More information about the ffmpeg-devel mailing list