[FFmpeg-cvslog] r25413 - in trunk: configure doc/general.texi libavformat/Makefile libavformat/allformats.c libavformat/sapenc.c

[Dave Yeo]

On 10/09/10 02:22 am, Martin Storsj? wrote:
> On Fri, 8 Oct 2010, Dave Yeo wrote:
>
>> On 10/08/10 05:05 am, mstorsjo wrote:
>>> Author: mstorsjo
>>> Date: Fri Oct  8 14:05:04 2010
>>> New Revision: 25413
>>>
>>> Log:
>>> Add a SAP (Session Announcement Protocol, RFC 2974) muxer
>>>
>>> Added:
>>>      trunk/libavformat/sapenc.c
>>
>> Hi, this broke OS/2
>> (http://fate.ffmpeg.org/x86.os2.444/20101008143916/compile). Perhaps a fix
>> like this?
>
> I applied the part with os_support.h, but the rest doesn't seem right -
> why check for the multicast structs instead of just
> HAVE_STRUCT_SOCKADDR_IN6 (which is available in many more configurations
> than just the ones with ipv6 multicast structs, too). I committed a fix
> checking for that define yesterday already. Please tell me if that still
> doesn't compile properly. (According to fate, socklen_t seems to be the
> only issue left.)
>

Sorry, my mistake. Thanks for the fixes, compile works fine now.
Dave