summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--src/core/boxes.c8
-rw-r--r--src/core/testboxes.c92
-rw-r--r--src/core/workspace.c16
-rw-r--r--src/include/boxes.h9
4 files changed, 55 insertions, 70 deletions
diff --git a/src/core/boxes.c b/src/core/boxes.c
index c9ce916a..e05924e6 100644
--- a/src/core/boxes.c
+++ b/src/core/boxes.c
@@ -764,12 +764,6 @@ meta_rectangle_expand_to_avoiding_struts (MetaRectangle *rect,
} /* end loop over struts */
} /* end meta_rectangle_expand_to_avoiding_struts */
-void
-meta_rectangle_free_list_and_elements (GList *filled_list)
-{
- g_list_free_full (filled_list, g_free);
-}
-
gboolean
meta_rectangle_could_fit_in_region (const GList *spanning_rects,
const MetaRectangle *rect)
@@ -1783,7 +1777,7 @@ meta_rectangle_find_onscreen_edges (const MetaRectangle *basic_rect,
ret = g_list_sort (ret, meta_rectangle_edge_cmp);
/* Free the fixed struts list */
- meta_rectangle_free_list_and_elements (fixed_strut_rects);
+ g_list_free_full (fixed_strut_rects, g_free);
return ret;
}
diff --git a/src/core/testboxes.c b/src/core/testboxes.c
index dfd65c69..07aa571e 100644
--- a/src/core/testboxes.c
+++ b/src/core/testboxes.c
@@ -357,7 +357,7 @@ get_xinerama_edges (int which_xinerama_set, int which_strut_set)
ret = meta_rectangle_find_nonintersected_xinerama_edges (&screenrect, xins, struts);
free_strut_list (struts);
- meta_rectangle_free_list_and_elements (xins);
+ g_list_free_full (xins, g_free);
return ret;
}
@@ -598,8 +598,8 @@ test_regions_okay ()
tmp = NULL;
tmp = g_list_prepend (tmp, new_meta_rect (0, 0, 1600, 1200));
verify_lists_are_equal (region, tmp);
- meta_rectangle_free_list_and_elements (tmp);
- meta_rectangle_free_list_and_elements (region);
+ g_list_free_full (tmp, g_free);
+ g_list_free_full (region, g_free);
/*************************************************************/
/* Make sure test region 1 has the right spanning rectangles */
@@ -609,8 +609,8 @@ test_regions_okay ()
tmp = g_list_prepend (tmp, new_meta_rect (0, 20, 400, 1180));
tmp = g_list_prepend (tmp, new_meta_rect (0, 20, 1600, 1140));
verify_lists_are_equal (region, tmp);
- meta_rectangle_free_list_and_elements (tmp);
- meta_rectangle_free_list_and_elements (region);
+ g_list_free_full (tmp, g_free);
+ g_list_free_full (region, g_free);
/*************************************************************/
/* Make sure test region 2 has the right spanning rectangles */
@@ -623,8 +623,8 @@ test_regions_okay ()
tmp = g_list_prepend (tmp, new_meta_rect ( 0, 20, 800, 1130));
tmp = g_list_prepend (tmp, new_meta_rect ( 0, 20, 1600, 1080));
verify_lists_are_equal (region, tmp);
- meta_rectangle_free_list_and_elements (tmp);
- meta_rectangle_free_list_and_elements (region);
+ g_list_free_full (tmp, g_free);
+ g_list_free_full (region, g_free);
/*************************************************************/
/* Make sure test region 3 has the right spanning rectangles */
@@ -649,8 +649,8 @@ test_regions_okay ()
printf ("%s vs. %s\n", region_list, tmp_list);
#endif
verify_lists_are_equal (region, tmp);
- meta_rectangle_free_list_and_elements (tmp);
- meta_rectangle_free_list_and_elements (region);
+ g_list_free_full (tmp, g_free);
+ g_list_free_full (region, g_free);
/*************************************************************/
/* Make sure test region 4 has the right spanning rectangles */
@@ -659,8 +659,8 @@ test_regions_okay ()
tmp = NULL;
tmp = g_list_prepend (tmp, new_meta_rect ( 800, 20, 800, 1180));
verify_lists_are_equal (region, tmp);
- meta_rectangle_free_list_and_elements (tmp);
- meta_rectangle_free_list_and_elements (region);
+ g_list_free_full (tmp, g_free);
+ g_list_free_full (region, g_free);
/*************************************************************/
/* Make sure test region 5 has the right spanning rectangles */
@@ -692,7 +692,7 @@ test_region_fitting ()
g_assert (meta_rectangle_contained_in_region (region, &rect) == FALSE ||
meta_rectangle_could_fit_in_region (region, &rect) == TRUE);
}
- meta_rectangle_free_list_and_elements (region);
+ g_list_free_full (region, g_free);
/* Do some manual tests too */
region = get_screen_region (1);
@@ -709,14 +709,14 @@ test_region_fitting ()
g_assert (meta_rectangle_could_fit_in_region (region, &rect));
g_assert (!meta_rectangle_contained_in_region (region, &rect));
- meta_rectangle_free_list_and_elements (region);
+ g_list_free_full (region, g_free);
region = get_screen_region (2);
rect = meta_rect (1000, 50, 600, 1100);
g_assert (meta_rectangle_could_fit_in_region (region, &rect));
g_assert (!meta_rectangle_contained_in_region (region, &rect));
- meta_rectangle_free_list_and_elements (region);
+ g_list_free_full (region, g_free);
printf ("%s passed.\n", G_STRFUNC);
}
@@ -746,7 +746,7 @@ test_clamping_to_region ()
g_assert (meta_rectangle_could_fit_in_region (region, &rect) == TRUE);
g_assert (rect.x == temp.x && rect.y == temp.y);
}
- meta_rectangle_free_list_and_elements (region);
+ g_list_free_full (region, g_free);
/* Do some manual tests too */
region = get_screen_region (1);
@@ -812,7 +812,7 @@ test_clamping_to_region ()
&min_size);
g_assert (rect.width == 100 && rect.height == 999999);
- meta_rectangle_free_list_and_elements (region);
+ g_list_free_full (region, g_free);
printf ("%s passed.\n", G_STRFUNC);
}
@@ -856,7 +856,7 @@ test_clipping_to_region ()
g_assert (meta_rectangle_contained_in_region (region, &rect) == TRUE);
}
}
- meta_rectangle_free_list_and_elements (region);
+ g_list_free_full (region, g_free);
/* Do some manual tests too */
region = get_screen_region (2);
@@ -895,7 +895,7 @@ test_clipping_to_region ()
&rect);
g_assert (meta_rectangle_equal (&rect, &temp));
- meta_rectangle_free_list_and_elements (region);
+ g_list_free_full (region, g_free);
printf ("%s passed.\n", G_STRFUNC);
}
@@ -918,7 +918,7 @@ test_shoving_into_region ()
g_assert (meta_rectangle_contained_in_region (region, &rect));
}
}
- meta_rectangle_free_list_and_elements (region);
+ g_list_free_full (region, g_free);
/* Do some manual tests too */
region = get_screen_region (2);
@@ -965,7 +965,7 @@ test_shoving_into_region ()
&rect);
g_assert (meta_rectangle_equal (&rect, &temp));
- meta_rectangle_free_list_and_elements (region);
+ g_list_free_full (region, g_free);
printf ("%s passed.\n", G_STRFUNC);
}
@@ -1038,8 +1038,8 @@ test_find_onscreen_edges ()
tmp = g_list_prepend (tmp, new_screen_edge (1600, 0, 0, 1200, right));
tmp = g_list_prepend (tmp, new_screen_edge ( 0, 0, 0, 1200, left));
verify_edge_lists_are_equal (edges, tmp);
- meta_rectangle_free_list_and_elements (tmp);
- meta_rectangle_free_list_and_elements (edges);
+ g_list_free_full (tmp, g_free);
+ g_list_free_full (edges, g_free);
/*************************************************/
/* Make sure test region 1 has the correct edges */
@@ -1053,8 +1053,8 @@ test_find_onscreen_edges ()
tmp = g_list_prepend (tmp, new_screen_edge ( 400, 1160, 0, 40, right));
tmp = g_list_prepend (tmp, new_screen_edge ( 0, 20, 0, 1180, left));
verify_edge_lists_are_equal (edges, tmp);
- meta_rectangle_free_list_and_elements (tmp);
- meta_rectangle_free_list_and_elements (edges);
+ g_list_free_full (tmp, g_free);
+ g_list_free_full (edges, g_free);
/*************************************************/
/* Make sure test region 2 has the correct edges */
@@ -1074,8 +1074,8 @@ test_find_onscreen_edges ()
tmp = g_list_prepend (tmp, new_screen_edge ( 450, 1150, 0, 50, left));
tmp = g_list_prepend (tmp, new_screen_edge ( 0, 20, 0, 1180, left));
verify_edge_lists_are_equal (edges, tmp);
- meta_rectangle_free_list_and_elements (tmp);
- meta_rectangle_free_list_and_elements (edges);
+ g_list_free_full (tmp, g_free);
+ g_list_free_full (edges, g_free);
/*************************************************/
/* Make sure test region 3 has the correct edges */
@@ -1109,8 +1109,8 @@ test_find_onscreen_edges ()
#endif
verify_edge_lists_are_equal (edges, tmp);
- meta_rectangle_free_list_and_elements (tmp);
- meta_rectangle_free_list_and_elements (edges);
+ g_list_free_full (tmp, g_free);
+ g_list_free_full (edges, g_free);
/*************************************************/
/* Make sure test region 4 has the correct edges */
@@ -1122,8 +1122,8 @@ test_find_onscreen_edges ()
tmp = g_list_prepend (tmp, new_screen_edge (1600, 20, 0, 1180, right));
tmp = g_list_prepend (tmp, new_screen_edge ( 800, 20, 0, 1180, left));
verify_edge_lists_are_equal (edges, tmp);
- meta_rectangle_free_list_and_elements (tmp);
- meta_rectangle_free_list_and_elements (edges);
+ g_list_free_full (tmp, g_free);
+ g_list_free_full (edges, g_free);
/*************************************************/
/* Make sure test region 5 has the correct edges */
@@ -1131,8 +1131,8 @@ test_find_onscreen_edges ()
edges = get_screen_edges (5);
tmp = NULL;
verify_edge_lists_are_equal (edges, tmp);
- meta_rectangle_free_list_and_elements (tmp);
- meta_rectangle_free_list_and_elements (edges);
+ g_list_free_full (tmp, g_free);
+ g_list_free_full (edges, g_free);
/*************************************************/
/* Make sure test region 6 has the correct edges */
@@ -1144,8 +1144,8 @@ test_find_onscreen_edges ()
tmp = g_list_prepend (tmp, new_screen_edge (1600, 40, 0, 1160, right));
tmp = g_list_prepend (tmp, new_screen_edge ( 0, 40, 0, 1160, left));
verify_edge_lists_are_equal (edges, tmp);
- meta_rectangle_free_list_and_elements (tmp);
- meta_rectangle_free_list_and_elements (edges);
+ g_list_free_full (tmp, g_free);
+ g_list_free_full (edges, g_free);
printf ("%s passed.\n", G_STRFUNC);
}
@@ -1167,8 +1167,8 @@ test_find_nonintersected_xinerama_edges ()
edges = get_xinerama_edges (0, 0);
tmp = NULL;
verify_edge_lists_are_equal (edges, tmp);
- meta_rectangle_free_list_and_elements (tmp);
- meta_rectangle_free_list_and_elements (edges);
+ g_list_free_full (tmp, g_free);
+ g_list_free_full (edges, g_free);
/*************************************************************************/
/* Make sure test xinerama set 2 for with region 1 has the correct edges */
@@ -1178,8 +1178,8 @@ test_find_nonintersected_xinerama_edges ()
tmp = g_list_prepend (tmp, new_xinerama_edge ( 0, 600, 1600, 0, bottom));
tmp = g_list_prepend (tmp, new_xinerama_edge ( 0, 600, 1600, 0, top));
verify_edge_lists_are_equal (edges, tmp);
- meta_rectangle_free_list_and_elements (tmp);
- meta_rectangle_free_list_and_elements (edges);
+ g_list_free_full (tmp, g_free);
+ g_list_free_full (edges, g_free);
/*************************************************************************/
/* Make sure test xinerama set 1 for with region 2 has the correct edges */
@@ -1197,8 +1197,8 @@ test_find_nonintersected_xinerama_edges ()
big_buffer1, big_buffer2);
#endif
verify_edge_lists_are_equal (edges, tmp);
- meta_rectangle_free_list_and_elements (tmp);
- meta_rectangle_free_list_and_elements (edges);
+ g_list_free_full (tmp, g_free);
+ g_list_free_full (edges, g_free);
/*************************************************************************/
/* Make sure test xinerama set 3 for with region 3 has the correct edges */
@@ -1212,8 +1212,8 @@ test_find_nonintersected_xinerama_edges ()
tmp = g_list_prepend (tmp, new_xinerama_edge ( 800, 675, 0, 425, right));
tmp = g_list_prepend (tmp, new_xinerama_edge ( 800, 675, 0, 525, left));
verify_edge_lists_are_equal (edges, tmp);
- meta_rectangle_free_list_and_elements (tmp);
- meta_rectangle_free_list_and_elements (edges);
+ g_list_free_full (tmp, g_free);
+ g_list_free_full (edges, g_free);
/*************************************************************************/
/* Make sure test xinerama set 3 for with region 4 has the correct edges */
@@ -1224,8 +1224,8 @@ test_find_nonintersected_xinerama_edges ()
tmp = g_list_prepend (tmp, new_xinerama_edge ( 800, 600, 800, 0, top));
tmp = g_list_prepend (tmp, new_xinerama_edge ( 800, 600, 0, 600, right));
verify_edge_lists_are_equal (edges, tmp);
- meta_rectangle_free_list_and_elements (tmp);
- meta_rectangle_free_list_and_elements (edges);
+ g_list_free_full (tmp, g_free);
+ g_list_free_full (edges, g_free);
/*************************************************************************/
/* Make sure test xinerama set 3 for with region 5has the correct edges */
@@ -1233,8 +1233,8 @@ test_find_nonintersected_xinerama_edges ()
edges = get_xinerama_edges (3, 5);
tmp = NULL;
verify_edge_lists_are_equal (edges, tmp);
- meta_rectangle_free_list_and_elements (tmp);
- meta_rectangle_free_list_and_elements (edges);
+ g_list_free_full (tmp, g_free);
+ g_list_free_full (edges, g_free);
printf ("%s passed.\n", G_STRFUNC);
}
diff --git a/src/core/workspace.c b/src/core/workspace.c
index dffb1ebf..98c33e20 100644
--- a/src/core/workspace.c
+++ b/src/core/workspace.c
@@ -156,11 +156,11 @@ meta_workspace_free (MetaWorkspace *workspace)
{
workspace_free_struts (workspace);
for (i = 0; i < screen->n_xinerama_infos; i++)
- meta_rectangle_free_list_and_elements (workspace->xinerama_region[i]);
+ g_list_free_full (workspace->xinerama_region[i], g_free);
g_free (workspace->xinerama_region);
- meta_rectangle_free_list_and_elements (workspace->screen_region);
- meta_rectangle_free_list_and_elements (workspace->screen_edges);
- meta_rectangle_free_list_and_elements (workspace->xinerama_edges);
+ g_list_free_full (workspace->screen_region, g_free);
+ g_list_free_full (workspace->screen_edges, g_free);
+ g_list_free_full (workspace->xinerama_edges, g_free);
}
g_free (workspace);
@@ -560,11 +560,11 @@ meta_workspace_invalidate_work_area (MetaWorkspace *workspace)
workspace_free_struts (workspace);
for (i = 0; i < workspace->screen->n_xinerama_infos; i++)
- meta_rectangle_free_list_and_elements (workspace->xinerama_region[i]);
+ g_list_free_full (workspace->xinerama_region[i], g_free);
g_free (workspace->xinerama_region);
- meta_rectangle_free_list_and_elements (workspace->screen_region);
- meta_rectangle_free_list_and_elements (workspace->screen_edges);
- meta_rectangle_free_list_and_elements (workspace->xinerama_edges);
+ g_list_free_full (workspace->screen_region, g_free);
+ g_list_free_full (workspace->screen_edges, g_free);
+ g_list_free_full (workspace->xinerama_edges, g_free);
workspace->xinerama_region = NULL;
workspace->screen_region = NULL;
workspace->screen_edges = NULL;
diff --git a/src/include/boxes.h b/src/include/boxes.h
index cf49fe62..87cc68f2 100644
--- a/src/include/boxes.h
+++ b/src/include/boxes.h
@@ -188,15 +188,6 @@ void meta_rectangle_expand_to_avoiding_struts (
const MetaDirection direction,
const GSList *all_struts);
-/* Free the list created by
- * meta_rectangle_get_minimal_spanning_set_for_region()
- * or
- * meta_rectangle_find_onscreen_edges ()
- * or
- * meta_rectangle_find_nonintersected_xinerama_edges()
- */
-void meta_rectangle_free_list_and_elements (GList *filled_list);
-
/* could_fit_in_region determines whether one of the spanning_rects is
* big enough to contain rect. contained_in_region checks whether one
* actually contains it.