[MPlayer-cvslog] r21984 - in trunk: libmpcodecs/ae_lavc.c libmpdemux/demux_lavf.c libmpdemux/muxer_lavf.c

[Reimar Doeffinger]

Hello,
On Mon, Jan 22, 2007 at 12:18:13AM +0100, Diego Biurrun wrote:
> On Sun, Jan 21, 2007 at 04:44:58PM +0100, reimar wrote:
> > 
> > Modified:
> >    trunk/libmpcodecs/ae_lavc.c
> >    trunk/libmpdemux/demux_lavf.c
> >    trunk/libmpdemux/muxer_lavf.c
> > 
> > Log:
> > Simplify by using av_codec_get_id and include riff.h only in demux_lavf.c
> 
> /**
>  * @file riff.h
>  * internal header for RIFF based (de)muxers
>  * do NOT include this in end user applications
>  */

So? Same is true for bswap.h, even if it does not say as clearly. I
leave it to someone who cares to move the needed functionality to
official headers, I don't see using FFmpeg internals as an issue to us,
MPlayer breaks far more often due to build system changes than due to
changes in some headers we are not supposed to include.

Greetings,
Reimar Doeffinger