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

Ramiro Polla ramiro.polla
Tue Jul 14 19:50:40 CEST 2009


On Mon, Jul 13, 2009 at 7:11 PM, Martin Storsj?<martin at martin.st> wrote:
> The attached two patches first check for signal.h in configure, and then
> leave out non-critical parts of ffmpeg.c if it isn't found.
>
> signal.h isn't available on mingw32ce (mingw32 for WinCE). These patches
> reduce the amount of ugly hacking needed to compile ffmpeg.c on that
> platform.
>
> The double definition of received_sigterm isn't really pretty, but IMHO
> looks better than ifdeffing all the places where it's used. Another
> solution would be to add "typedef sig_atomic_t int;" somewhere if signal.h
> isn't available, and move the variable definition out of the ifdef block.

I think redefining a POSIX type is even worse.

Mans, what do you think about this patch?

Ramiro Polla



More information about the ffmpeg-devel mailing list