[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