[vlc-devel] [PATCH 11/15] yuv: use the source visible area to shift pixels

Rémi Denis-Courmont remi at remlab.net
Thu Sep 3 16:25:54 CEST 2020


Le torstaina 3. syyskuuta 2020, 9.26.55 EEST Steve Lhomme a écrit :
> On 2020-09-03 7:52, Steve Lhomme wrote:
> > The values are always the same as in the output format.
> 
> In fact they are not, because of video_format_ApplyRotation(). So this
> patch is wrong.
> 
> Dynamic cropping/AR is not handled and pretends to the core that it
> does, so it's probably broken. It should probably return an error when
> that happens and handle RESET_PICTURES. Then a conversion filter to
> match the original format will be added.

There are no reasons to use a conversion filter here. We're just writing pixel 
values and meta to a file.

-- 
Реми Дёни-Курмон
http://www.remlab.net/





More information about the vlc-devel mailing list