[MPlayer-cvslog] r25065 - trunk/stream/stream_dvd.c

Ivo ivop at euronet.nl
Sun Nov 18 20:38:59 CET 2007


On Sunday 18 November 2007 20:06, Ivo wrote:
> On Saturday 17 November 2007 19:20, reimar wrote:
> > Author: reimar
> > Date: Sat Nov 17 19:20:04 2007
> > New Revision: 25065
> >
> > Log:
> > Use AV_WB32 instead of manual bit-fiddling when setting DVD speed
>
> This broke compilation for me:
[..]
> After further investigation, I found out that libavutil/bswap.h includes
> <byteswap.h> conditionally and the condition is met in my ffmpeg tree
> (which compiles fine) but not in my mplayer tree. The condition is
> HAVE_BYTESWAP_H. And byteswap.h probably gets included indirectly anyway,
> which causes bswap_16 to be a macro (defines to __bswap_16 which is in
> bits/byteswap.h an contains the above code that bails out).
>
> I think checking for byteswap.h in configure and setting HAVE_BYTESWAP_H
> accordingly should suffice, shouldn't it?

Here's a patch. OK to apply?

--Ivo
-------------- next part --------------
A non-text attachment was scrubbed...
Name: byteswap.patch
Type: text/x-diff
Size: 776 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-cvslog/attachments/20071118/23f3978b/attachment.patch>


More information about the MPlayer-cvslog mailing list