Hi

Thanks for the patch!

Some comments below.

On Wed, May 16, 2018 at 10:36 PM <michaelbuc...@gmail.com> wrote:

> From: Michael Buch <michaelbuc...@gmail.com>

> The termianl window is created without lower/upper bounds for window
width/height. On resizing the terminal to a size of 1x1 or below we crash.
This patch sets the minimum window height and width to prevent resizing
past this size thus handling the scenario similar to other terminals.

It would be nice to have this commit message wrapped at around 80 chars.

There is also a typo: s/termianl/terminal/


> ---
>   x.c | 6 ++++--
>   1 file changed, 4 insertions(+), 2 deletions(-)

> diff --git a/x.c b/x.c
> index c0bd890..00ea444 100644
> --- a/x.c
> +++ b/x.c
> @@ -788,15 +788,17 @@ xhints(void)

>          sizeh = XAllocSizeHints();

> -       sizeh->flags = PSize | PResizeInc | PBaseSize;
> +       sizeh->flags = PSize | PResizeInc | PBaseSize | PMinSize;
>          sizeh->height = win.h;
>          sizeh->width = win.w;
>          sizeh->height_inc = win.ch;
>          sizeh->width_inc = win.cw;
>          sizeh->base_height = 2 * borderpx;
>          sizeh->base_width = 2 * borderpx;
> +       sizeh->min_height = 2 * win.ch;
> +       sizeh->min_width = 2 * win.cw;
>          if (xw.isfixed) {
> -               sizeh->flags |= PMaxSize | PMinSize;
> +               sizeh->flags |= PMaxSize;

I think in the isfixed case we don't want the size to change at all. So we
should keep both the PMaxSize and PMinSize flags set.

Other than that the patch looks good to me!


Cheers,

Silvan

>                  sizeh->min_width = sizeh->max_width = win.w;
>                  sizeh->min_height = sizeh->max_height = win.h;
>          }
> --
> 2.17.0

Reply via email to