[FFmpeg-devel] [RFC] Split libavformat

[Michael Niedermayer]

On Tue, Nov 27, 2007 at 04:17:59PM -0500, Ronald S. Bultje wrote:
> Hi,
> 
> On Nov 27, 2007 12:34 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
> 
> > > resolve_host[_ipv6]() isn't an os_support function and would move to
> > > network.c (new).
> >
> > it seems you dont get it, lavf WILL NOT provide an externally vissible
> > resolve_host() no matter what API or implementation, ffserver seems to
> > need
> > it though <http://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-devel>
> 
> 
> But not in ossupport also, right? If provided, it is a lib for functions
> missing on some OSes but available on (and thus not newly implemented for)
> others. This definition does not fit ff_socket_nonblock() or resolve_host().

what was it that the standard functions where lacking?
for ff_socket_nonblock() something like the following could be used:

int av_fcntl(int fd, int cmd, long arg){
#ifdef HAVE_WINSOCK2_H
    if(cmd == F_SETFL){
        int enable= !!(arg & O_NONBLOCK);
        ioctlsocket(socket, FIONBIO, &enable);
    }
#endif
    return fcntl(fd, cmd, arg);
}

also i have significantly fewer objections to adding a
av_nonstd_socket_nonblock() or av_nonstd_resolve_host()
to libossupport than libavformat, that is if the need for such additions
has been demonstrated, so far though iam not convinced that the standard
functions are insufficient

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I have often repented speaking, but never of holding my tongue.
-- Xenocrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20071127/0323a549/attachment.pgp>