[MPlayer-cvslog] r35809 - in trunk: gui/util/bitmap.c libmpcodecs/ad_ffmpeg.c libmpcodecs/ad_spdif.c libmpcodecs/vd_ffmpeg.c libmpcodecs/vf_mcdeint.c libmpcodecs/vf_screenshot.c libmpcodecs/vf_uspp.c libmpcodecs/vf...

[Alexander Strasser]

Diego Biurrun wrote:
> On Tue, Jan 22, 2013 at 09:03:48AM +0100, Reimar Döffinger wrote:
> > On 22 Jan 2013, at 05:01, compn <tempn at twmi.rr.com> wrote:
> > > On Tue, 22 Jan 2013 00:22:06 +0100, Diego Biurrun wrote:
> > >> On Mon, Jan 21, 2013 at 06:17:11PM -0500, compn wrote:
> > >>> On Mon, 21 Jan 2013 20:44:53 +0100 (CET), diego wrote:
> > >>>> Author: diego
> > >>>> Date: Mon Jan 21 20:44:53 2013
> > >>>> New Revision: 35809
> > >>>> 
> > >>>> Log:
> > >>>> Replace CODEC_IDs their modern AV_-prefixed counterparts.
> > >>> 
> > >>> this means you cant compile mplayer with an older ffmpeg anymore.
> > >> 
> > >> How old?  It sure works in trunk ...
> > > 
> > > from before AV_CODEC_ID was added ?
> > 
> > I think that's long ago, I'd expect it to have many issues anyway.
> > I didn't how long ago it actually is though...
> 
> Quoting from APIchanges:
> 
>   2012-08-07 - 104e10f - lavc 54.25 - avcodec.h
>     Rename CodecID to AVCodecID and all CODEC_ID_* to AV_CODEC_ID_*.
> 
> In any case attempts to support more than HEAD versions of libav*
> sure is news to me ...

  IMHO we do not attempt to needlessly break older versions than HEAD.
The problem here is of a quite different nature AFAICT.

  Anyway, OK for me if it works with FFmpeg HEAD.

  Alexander