[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