[vlc-devel] [PATCH 01/39] video_output: pass the video format to vout_SizeWindow()

Rémi Denis-Courmont remi at remlab.net
Mon Oct 7 13:09:55 CEST 2019


Hello,

This contradicts both the definition of zoom, auto-scale and other parameters, and contradicts earlier discussion about window resize policy here.

So nack.

Le 7 octobre 2019 10:33:58 GMT+03:00, Steve Lhomme <robux4 at ycbcr.xyz> a écrit :
>On 2019-10-03 20:42, Rémi Denis-Courmont wrote:
>> Le keskiviikkona 2. lokakuuta 2019, 17.23.16 EEST Steve Lhomme a
>écrit :
>>> This will be needed when the GetDevice call will create a window
>without
>>> setting the sys->original which is only related to the display
>module.
>> 
>> sys->original is the "input" format to the video output. I don't see
>what else
>> the vout window can use as the basis for size computation. In fact,
>anything
>> else would violate the existing definitions for the concerned
>configuration
>> parameters, AFAICT.
>
>The patch comment says "the sys->original which is only related to the 
>display module". In other words it's not useful to the window creation.
>
>The sys->original will only be set/used with the display module, not
>the 
>window module.
>_______________________________________________
>vlc-devel mailing list
>To unsubscribe or modify your subscription options:
>https://mailman.videolan.org/listinfo/vlc-devel

-- 
Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20191007/82438bc7/attachment.html>


More information about the vlc-devel mailing list