[FFmpeg-devel] [PATCH] avcodec: remove leftover iff_byterun1 decoder
Ronald S. Bultje
rsbultje at gmail.com
Sun Oct 11 23:56:34 CEST 2015
Howdy,
On Sun, Oct 11, 2015 at 5:50 PM, Andreas Cadhalpun <
andreas.cadhalpun at googlemail.com> wrote:
> Hi Ronald,
>
> On 11.10.2015 23:39, Ronald S. Bultje wrote:
> > On Sun, Oct 11, 2015 at 5:17 PM, Andreas Cadhalpun <
> > andreas.cadhalpun at googlemail.com> wrote:
> >> --- a/libavcodec/avcodec.h
> >> +++ b/libavcodec/avcodec.h
> >> @@ -241,7 +241,7 @@ enum AVCodecID {
> >> AV_CODEC_ID_ANM,
> >> AV_CODEC_ID_BINKVIDEO,
> >> AV_CODEC_ID_IFF_ILBM,
> >> - AV_CODEC_ID_IFF_BYTERUN1,
> >> +#define AV_CODEC_ID_IFF_BYTERUN1 AV_CODEC_ID_IFF_ILBM
> >> AV_CODEC_ID_KGV1,
> >> AV_CODEC_ID_YOP,
> >> AV_CODEC_ID_VP8,
> >>
> >
> > Should this go under a version define so it disappears in 3 years?
>
> It shouldn't as long as Libav still uses this codec ID.
>
Hm, ... ok so it's one of these political bits. OK, fine, I'll drop that
objection. Rest of patch is OK.
AV_CODEC_ID_H265 on the other hand could disappear in such a way.
That's unrelated to this patch though, so no need to hold this one up for
that :)
Ronald
More information about the ffmpeg-devel
mailing list