From 1d57f7b08590255e57dd5931a1993507b4f116c2 Mon Sep 17 00:00:00 2001 From: Dana Jansens Date: Fri, 12 Feb 2010 13:44:10 -0500 Subject: using g_slice_new() instead of g_new() part 1 --- openbox/place.c | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) (limited to 'openbox/place.c') diff --git a/openbox/place.c b/openbox/place.c index c396e8fa..cf4d3048 100644 --- a/openbox/place.c +++ b/openbox/place.c @@ -150,7 +150,7 @@ static gboolean place_random(ObClient *client, gint *x, gint *y) else *y = areas[i]->y; for (i = 0; i < screen_num_monitors; ++i) - g_free(areas[i]); + g_slice_free(Rect, areas[i]); g_free(areas); return TRUE; @@ -339,7 +339,7 @@ static gboolean place_nooverlap(ObClient *c, gint *x, gint *y) } for (i = 0; i < screen_num_monitors; ++i) - g_free(areas[i]); + g_slice_free(Rect, areas[i]); g_free(areas); return ret; } @@ -364,6 +364,8 @@ static gboolean place_under_mouse(ObClient *client, gint *x, gint *y) *y = py - client->area.height / 2 - client->frame->size.top; *y = MIN(MAX(*y, t), b); + g_slice_free(Rect, area); + return TRUE; } @@ -394,7 +396,7 @@ static gboolean place_per_app_setting(ObClient *client, gint *x, gint *y, /* don't free the first one, it's being set as "screen" */ for (i = 1; i < screen_num_monitors; ++i) - g_free(areas[i]); + g_slice_free(Rect, areas[i]); g_free(areas); } @@ -414,7 +416,7 @@ static gboolean place_per_app_setting(ObClient *client, gint *x, gint *y, else *y = screen->y + settings->position.y.pos; - g_free(screen); + g_slice_free(Rect, screen); return TRUE; } @@ -460,7 +462,7 @@ static gboolean place_transient_splash(ObClient *client, gint *x, gint *y) *y = (areas[0]->height - client->frame->area.height) / 2 + areas[0]->y; for (i = 0; i < screen_num_monitors; ++i) - g_free(areas[i]); + g_slice_free(Rect, areas[i]); g_free(areas); return TRUE; } -- cgit v1.2.3