[FFmpeg-devel] [PATCH 1/5] avformat/ffmdec: Remove some st->codec uses which set encoder parameters

Andreas Cadhalpun andreas.cadhalpun at googlemail.com
Sat Dec 3 15:03:58 EET 2016


On 03.12.2016 13:23, Michael Niedermayer wrote:
> On Sat, Dec 03, 2016 at 01:14:19AM +0100, Andreas Cadhalpun wrote:
>> On 03.12.2016 00:52, Michael Niedermayer wrote:
>>> Modern ffserver should not need these
>>>
>>> Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
>>> ---
>>>  libavformat/ffmdec.c | 148 +++++++++++++++++++++++++--------------------------
>>>  1 file changed, 74 insertions(+), 74 deletions(-)
>>>
>>> diff --git a/libavformat/ffmdec.c b/libavformat/ffmdec.c
>>> index 960e793220..9192bff507 100644
>>> --- a/libavformat/ffmdec.c
>>> +++ b/libavformat/ffmdec.c
>>> @@ -405,51 +405,51 @@ static int ffm2_read_header(AVFormatContext *s)
>>>              ret = av_image_check_size(codec->width, codec->height, 0, s);
>>>              if (ret < 0)
>>>                  goto fail;
>>> -            codec->gop_size = avio_rb16(pb);
>>> +            avio_rb16(pb);
>>
>> Please add a comment like /* gop_size */ here and similar ones below.
>> Otherwise it'll be rather mysterious what's the point of all these avio_rb* calls.
> 
> done locally

Thanks, patch is OK then.

Best regards,
Andreas



More information about the ffmpeg-devel mailing list