[FFmpeg-devel] [PATCH] Don't overwrite previously setup dimensions for all codecs

Li, Zhong zhong.li at intel.com
Thu Jan 11 07:34:07 EET 2018


> > 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 <michaelni at 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?


More information about the ffmpeg-devel mailing list