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

[Martin Storsjö]

On Wed, 15 Jul 2009, Martin Storsj? wrote:

> 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.

Here's another update to the patch series; I noticed that not all 
HAVE_SIGNAL_H checks in ffmpeg.c were needed.

// 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: 511 bytes
Desc: 
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090717/5240d4d9/attachment.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-Check-for-signal.h.patch
Type: text/x-diff
Size: 601 bytes
Desc: 
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090717/5240d4d9/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: 936 bytes
Desc: 
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090717/5240d4d9/attachment-0002.patch>