On Thu, 11 Jan 2018 05:34:07 +0000
"Li, Zhong" <zhong...@intel.com> wrote:
> > > IIUC, This commit was merged from Libav, checking the commit log of
> > cf7d3846fc865df47bd64f7d4d10bbedf9e3a17b:
> >
> > Yeah, but...
> >
> > > Merge commit '9de9b828ef005dec37052548c195a6b4f18fc701'
> > >
> > > * commit '9de9b828ef005dec37052548c195a6b4f18fc701':
> > > lavc: don't overwrite display dimensions with coded dimensions.
> > > lavc: extend / update the [coded_]{width,height} doxy
> > >
> > > Conflicts:
> > > libavcodec/avcodec.h
> > > libavcodec/utils.c
> > >
> > > The change to the w/h handling is not merged as it breaks lowres
> > >
> > > Merged-by: Michael Niedermayer <michae...@gmx.at>
> >
> > This says this specific part was skipped. Apparently it broke lowres, an
> > obscure feature that probably nobody uses and that Libav removed.
>
> Yup... 9de9b82 was not merged as a whole.
> I guess I am not the only one person who is confused why w/h are overwritten
> by coded_w/coded_h here
> (https://ffmpeg.org/pipermail/ffmpeg-devel/2013-March/140142.html )
> For "lowres", is it still broken for H264/VP6F/DXV by current code?
I don't know (from the mail above this might be more an API problem?),
but I'd prefer your patch over the current situation as well.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
