[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