From e7fbbf47e5ec30cba1ef55e0af6601c1b04e004f Mon Sep 17 00:00:00 2001 From: Uli Schlachter Date: Sat, 10 Oct 2015 21:13:40 +0200 Subject: [PATCH] Fix client_apply_size_hints() The bit that indicates that the base size is set is XCB_ICCCM_SIZE_HINT_BASE_SIZE. However, instead this code checked XCB_ICCCM_SIZE_HINT_P_SIZE which is set to indicate how the initial window position is chosen. So we were checking the complete wrong bit. Whoops... Fixes: https://github.com/awesomeWM/awesome/issues/456 Signed-off-by: Uli Schlachter --- objects/client.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/objects/client.c b/objects/client.c index 0c180d129..e3dc350bb 100644 --- a/objects/client.c +++ b/objects/client.c @@ -624,7 +624,7 @@ client_apply_size_hints(client_t *c, area_t geometry) /* Size hints are applied to the window without any decoration */ client_remove_titlebar_geometry(c, &geometry); - if(c->size_hints.flags & XCB_ICCCM_SIZE_HINT_P_SIZE) + if(c->size_hints.flags & XCB_ICCCM_SIZE_HINT_BASE_SIZE) { basew = c->size_hints.base_width; baseh = c->size_hints.base_height; @@ -643,7 +643,7 @@ client_apply_size_hints(client_t *c, area_t geometry) minw = c->size_hints.min_width; minh = c->size_hints.min_height; } - else if(c->size_hints.flags & XCB_ICCCM_SIZE_HINT_P_SIZE) + else if(c->size_hints.flags & XCB_ICCCM_SIZE_HINT_BASE_SIZE) { /* min size is substituted with base size if not specified */ minw = c->size_hints.base_width;