[PATCH 1/1] sandbox: make SDL window resizable

Heinrich Schuchardt xypron.glpk at gmx.de
Fri Oct 9 23:43:01 CEST 2020


On 09.10.20 19:23, Simon Glass wrote:
> Hi Heinrich,
>
> On Thu, 8 Oct 2020 at 03:46, Heinrich Schuchardt <xypron.glpk at gmx.de> wrote:
>>
>> On 05.10.20 03:41, Simon Glass wrote:
>>> Hi Heinrich,
>>>
>>> On Mon, 28 Sep 2020 at 19:11, Heinrich Schuchardt <xypron.glpk at gmx.de> wrote:
>>>>
>>>> Without resizing the SDL window showed by
>>>>
>>>>     ./u-boot -D -l
>>>>
>>>> is not legible on a high resolution screen.
>>>>
>>>> Start with a maximized window and allow resizing.
>>>>
>>>> Signed-off-by: Heinrich Schuchardt <xypron.glpk at gmx.de>
>>>> ---
>>>>  arch/sandbox/cpu/sdl.c | 4 +++-
>>>>  1 file changed, 3 insertions(+), 1 deletion(-)
>>>>
>>>
>>> Have you troubled the --double_lcd option?
>>
>> That looks better.
>>
>> This option is not documented in doc/arch/sandbox.rst.
>>
>>>
>>> Does this have any effect on performance?
>>
>> With --double_lcd you are already doing a resize operation. I could not
>> observe any performance loss for a maximized window. The BitBlt
>> operations in X11 are well accelerated by current GPUs.
>>
>>>
>>> I actually don't like this as it makes the text hard to read, and full
>>> screen is annoying. I wonder if this should be a flag?
>>
>> I understand that you do not want the window maximized. But is anything
>> wrong about making the window resizable (SDL_WINDOW_RESIZABLE)?
>>
>> Shall I update the patch accordingly?
>
> I think that is OK...does it mean that the window has scroll bars, etc?

No, it will only have dragable edges and a maximize button.

Best regards

Heinrich


More information about the U-Boot mailing list