[FFmpeg-devel] [PATCH 5/5] avformat/ffmdec: remove last use of st->codec

[Andreas Cadhalpun]

On 03.12.2016 13:14, Michael Niedermayer wrote:
> On Sat, Dec 03, 2016 at 12:51:12PM +0100, Andreas Cadhalpun wrote:
>> On 03.12.2016 00:52, Michael Niedermayer wrote:
>>> @@ -364,14 +364,19 @@ static int ffm2_read_header(AVFormatContext *s)
>>>              }
>>>              codecpar->bit_rate = avio_rb32(pb);
>>>              if (codecpar->bit_rate < 0) {
>>> -                av_log(codec, AV_LOG_ERROR, "Invalid bit rate %"PRId64"\n", codecpar->bit_rate);
>>> +                av_log(s, AV_LOG_ERROR, "Invalid bit rate %"PRId64"\n", codecpar->bit_rate);
>>>                  ret = AVERROR_INVALIDDATA;
>>>                  goto fail;
>>>              }
>>> -            codec->flags = avio_rb32(pb);
>>> +            flags = avio_rb32(pb);
>>> +#if FF_API_LAVF_AVCTX
>>> +FF_DISABLE_DEPRECATION_WARNINGS
>>> +            st->codec->flags = flags;
>>> +FF_ENABLE_DEPRECATION_WARNINGS
>>> +#endif
>>
>> What's the point of this? It doesn't seem to be needed by ffserver.
> 
> not by the current one but by the one a few commits ago

I see, commit 311107. Patch is OK.

Best regards,
Andreas