[MPlayer-dev-eng] [PATCH] Check for windows.h

Erik Auerswald auerswal at unix-ag.uni-kl.de
Tue Jan 8 18:49:57 CET 2013


Hi,

On 01/08/2013 06:01 PM, Josh Finch wrote:
> Checking for the headers doesnt have to be in the "if mingw32" statement, just end the if statement after checking for w32threads.

The attached patch does just this. Thus it fixes the following -Wundef 
warnings on non-Windows systems:

libdvdcss/libdvdcss.c:123:5: warning: "HAVE_DIRECT_H" is not defined 
[-Wundef]
libdvdcss/libdvdcss.c:241:5: warning: "HAVE_DIRECT_H" is not defined 
[-Wundef]
libavformat/cache.c:36:5: warning: "HAVE_IO_H" is not defined [-Wundef]
libavformat/file.c:26:5: warning: "HAVE_IO_H" is not defined [-Wundef]
libavutil/file.c:28:5: warning: "HAVE_IO_H" is not defined [-Wundef]
libavutil/log.c:32:5: warning: "HAVE_IO_H" is not defined [-Wundef]
libavutil/time.c:32:5: warning: "HAVE_WINDOWS_H" is not defined [-Wundef]

On Windows systems, windows.h will be included in 
ffmpeg/libavutil/time.c if it is detected.

This patch replaces both other patches sent by me today.

After applying this change, the OS/2 thread checks should be moved up 
above the direct.h, io.h and windows.h checks to improve maintainability.

Thanks,
Erik
-------------- next part --------------
A non-text attachment was scrubbed...
Name: mplayer-configure-always_check_windows_direct_h_io_headers.patch
Type: text/x-diff
Size: 967 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20130108/c55abeb6/attachment.bin>


More information about the MPlayer-dev-eng mailing list