[Ffmpeg-devel] [PATCH] remove mingw32 ifdefs in file.c

Aurelien Jacobs aurel
Tue Dec 19 13:36:03 CET 2006


On Tue, 19 Dec 2006 02:10:28 -0200 (BRDT)
ramiro at lisha.ufsc.br wrote:

> Hello,
> 
> This Windows support was added in revision 78. Probably still meant for
> MSVC++.
> 
> Regression tests pass. Except for ffserver tests, which shouldn't even be
> run, since ffserver wasn't built.
> 
> Is <sys/ioctl.h> really necessary? It compiled fine for me without
> including it on linux too.

I checked this whole file, and I can't see anything which require ioctl.h
so IMO, remove this include. (And AFAICT you could also remove the include
sys/time.h but that is a separate issue)

Aurel




More information about the ffmpeg-devel mailing list