nilix/openbox
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

it would seem this cannot return NULL now

Browse source 
It will get the primary monitor if there is not monitor under the pointer. But
assert so it's clear something went wrong if this does happen. Note that there
was previously no check for the return value even though the comment claimed
there should be.
This commit is contained in:
Dana Jansens 2011-10-14 18:52:36 -04:00
parent b1039d6142
commit 332280daf9
1 changed files with 3 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
openbox/place.c
View file

@ -451,9 +451,10 @@ static gboolean place_per_app_setting(ObClient *client, gint *x, gint *y,
ob_debug("placing by per-app settings");
/* Find which head the pointer is on */
if (settings->monitor == 0)
/* this can return NULL */
if (settings->monitor == 0) {
screen = pick_pointer_head(client);
g_assert(screen);
}
else {
guint m = settings->monitor;
if (m < 1 || m > screen_num_monitors)