macos: remove assertion from external API
authorChristian Hergert <christian@hergert.me>
Tue, 15 Feb 2022 20:12:45 +0000 (12:12 -0800)
committerChristian Hergert <christian@hergert.me>
Tue, 15 Feb 2022 20:38:07 +0000 (12:38 -0800)
We only should be asserting in static functions. Furthermore, this function
did not need to have GDK_BEGIN_MACOS_ALLOC_POOL as nothing is being
allocated there which would cause pooling to get used.

gdk/macos/gdkmacosmonitor.c

index 413b8f8c253b543e5d2694c27a12ce2cce216a39..6df1da0edc0ccafcff22a007a5aadfdb9d8c1cef 100644 (file)
@@ -56,13 +56,11 @@ void
 gdk_macos_monitor_get_workarea (GdkMonitor   *monitor,
                                 GdkRectangle *geometry)
 {
-  GDK_BEGIN_MACOS_ALLOC_POOL;
-
   GdkMacosMonitor *self = (GdkMacosMonitor *)monitor;
   int x,  y;
 
-  g_assert (GDK_IS_MACOS_MONITOR (self));
-  g_assert (geometry != NULL);
+  g_return_if_fail (GDK_IS_MACOS_MONITOR (self));
+  g_return_if_fail (geometry != NULL);
 
   x = self->workarea.origin.x;
   y = self->workarea.origin.y + self->workarea.size.height;
@@ -75,8 +73,6 @@ gdk_macos_monitor_get_workarea (GdkMonitor   *monitor,
   geometry->y = y;
   geometry->width = self->workarea.size.width;
   geometry->height = self->workarea.size.height;
-
-  GDK_END_MACOS_ALLOC_POOL;
 }
 
 static void