[FFmpeg-devel] [PATCH] Make RTP work with IPv6 enabled v.2

[Michael Niedermayer]

Hi

On Sun, Oct 28, 2007 at 01:36:16PM -0400, Ronald S. Bultje wrote:
> Hi,
> 
> On 10/28/07, Michael Niedermayer <michaelni at gmx.at> wrote:
> >
> > On Sun, Oct 28, 2007 at 12:18:54PM -0400, Ronald S. Bultje wrote:
> > > On 10/28/07, Ronald S. Bultje <rsbultje at gmail.com> wrote:
> > > > 3 - ffmpeg-udp-connect_and_send.patch - replaces sendto() by connect()
> > and
> > > > send(), which makes it work for MacOSX, where for some reason sendto()
> > fails
> > > > with EINVAL (undocumented) for no obvious reason, where connect() and
> > send()
> > > > with the same parameters works just fine. recvfrom() stays the way it
> > is
> > > > because the man page suggests so.
> > > >
> > >
> > > Attached is a better version.
> > [...]
> > > +printf("Set remote: %s\n", uri);
> > >  #ifdef CONFIG_IPV6
> > > -    return udp_ipv6_set_remote_url(h, uri);
> > > +    int ret = udp_ipv6_set_remote_url(h, uri);
> >
> > hardly
> > besides that you break stdout output, the int ret will break gcc 2.95
> 
> 
> Oops, that was left from debugging... Better version attached.
[...]
> +        perror("connect");

hmm shouldnt that be av_log() ?

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

Opposition brings concord. Out of discord comes the fairest harmony.
-- Heraclitus
-------------- 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/20071028/5b489d7c/attachment.pgp>