[MPlayer-dev-eng] [PATCH] libvo: set PBaseSize in SizeHints

Bert Wesarg bert.wesarg at googlemail.com
Fri Dec 5 19:39:36 CET 2008


On Fri, Dec 5, 2008 at 19:37, Bert Wesarg <bert.wesarg at googlemail.com> wrote:
> On Fri, Dec 5, 2008 at 19:27, Reimar Döffinger
> <Reimar.Doeffinger at stud.uni-karlsruhe.de> wrote:
>> On Fri, Dec 05, 2008 at 07:12:19PM +0100, Bert Wesarg wrote:
>>> from ICCCM 4.1.2.3:
>>>
>>>   width = base_width + (i × width_inc)
>>>   height = base_height + (j × height_inc)
>>>
>>>   Window managers are encouraged to use i and j instead of width and height
>>>   in reporting window sizes to users. If a base size is not provided, the
>>>   minimum size is to be used in its place and vice versa.
>>>
>>> But mplayer sets the PMinSize size to 4x4 in x11_common.c and no
>>> PBaseSize, so the WM takes the PMinSize as PBaseSize and reports a
>>> size minus 4x4. Which is just plain wrong. The WM behaves conform to
>>> the ICCCM so mplayer needs to fix this.
>>
>> "plain wrong"? As I read it you don't like that your window manager
>> tells you the window size relative to the minimum and would prefer
>> absolute values, and like this you make MPlayer force your window
>> manager to do that. But that does not make the previous behaviour wrong.
>> In addition, the obvious question is if what you suggest makes most
>> sense, because what I would consider the "base size" is the unscaled
>> (but aspect-adjusted) size of the video, why not set that instead of 0?
> You mean, say for a 640x480 video, a base size of 640x480? I think
> because the relation of min size and base size that would mean you
> can't make the window smaler, but I'm not sure.
Actually, if your WM let you resize your window to a smaller size that
the base size, you would get negative sizes, I don't think you want
this either.

>
> Bert
>>
>> Greetings,
>> Reimar Döffinger
>> _______________________________________________
>> MPlayer-dev-eng mailing list
>> MPlayer-dev-eng at mplayerhq.hu
>> https://lists.mplayerhq.hu/mailman/listinfo/mplayer-dev-eng
>>
>


More information about the MPlayer-dev-eng mailing list