[FFmpeg-devel] [PATCH] Add coded_width/height to AVCodecParameters

wm4 nfxjfg at googlemail.com
Sun Apr 3 14:41:11 CEST 2016


On Sun, 3 Apr 2016 14:31:06 +0200
Hendrik Leppkes <h.leppkes at gmail.com> wrote:

> On Sun, Apr 3, 2016 at 2:07 PM, Michael Niedermayer
> <michael at niedermayer.cc> wrote:
> > Without this or some other change moving data back and forth corrupts it
> > as the cropped & possibly low res resolution modified w/h will be taken
> > a the pre lowres resolution
> > also without 2 sets theres no place to communicate the cropped dimensions
> >
> > can be reproduced with
> > ./ffplay_g -vlowres 2 tickets/162/avid.avi
> >  
> 
> lowres is supposed to be applied in the decoder, not the demuxer, so I
> don't see how this is required here.
> AVCodecParameters is supposed to contain container information, not
> pre-interpreted info based on some decoding options.

I have to agree.

Which codecs supposedly require coded_width and width both as part of
the container info?


More information about the ffmpeg-devel mailing list