[FFmpeg-devel] [PATCH] avcodec: remove leftover iff_byterun1 decoder
Andreas Cadhalpun
andreas.cadhalpun at googlemail.com
Mon Oct 12 00:26:46 CEST 2015
On 11.10.2015 23:56, Ronald S. Bultje wrote:
> On Sun, Oct 11, 2015 at 5:50 PM, Andreas Cadhalpun <
> andreas.cadhalpun at googlemail.com> wrote:
>> 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.
Thanks, pushed.
> 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 :)
Yes, I just mentioned it in case you want to write a patch for it. :)
Best regards,
Andreas
More information about the ffmpeg-devel
mailing list