[FFmpeg-devel] libossupport status

Rich Felker dalias
Sun Dec 23 11:55:22 CET 2007


On Sat, Dec 22, 2007 at 11:42:28PM +0100, Vadim Lebedev wrote:
> I agree,
> But it seems that original message in this thread suggest different 
> approach:  The patch for example proposes
> to remove #include "os_support.h" and
> to replace:
> 
>     ff_socket_nonblock(server_fd, 1);
> by:
>     fcntl(server_fd, F_SETFL, fcntl(server_fd, F_GETFL) | O_NONBLOCK);
> 
> 
> 
> This will no work on win32 (there is no F_SETFL on WIN32)

Then add it. Submit patches to mingw for their broken crap, and then
require latest mingw...

Anyway WTF is the point of this? O_NONBLOCK is useless; just do a
select before attempting to read/write... Or are you worried about
running into the Linux bug in select? :)

> The best way IMO will be to keep evrywhere
> 
> #include "os_support.h" 

No this is absolutely disgusting and unnecessary. The idea of "loss"
is to be a no-op for non-broken operating systems, not to pollute
everywhere with windows-workaround crap.

Rich




More information about the ffmpeg-devel mailing list