[mutter/gnome-41] stack: Add prerequisites to some stack API



commit 447d82de1876a62f7b75e506cb358b252a085466
Author: Carlos Garnacho <carlosg gnome org>
Date:   Wed Dec 1 12:22:12 2021 +0100

    stack: Add prerequisites to some stack API
    
    Passing a NULL workspace does not make sense, since it silently
    returns no windows. Mandate that a workspace is explicitly requested,
    and while at it check the other arguments as well.
    
    Part-of: <https://gitlab.gnome.org/GNOME/mutter/-/merge_requests/2124>
    (cherry picked from commit fa31c97aec925056ef04162b2c172a7f5e74af68)

 src/core/stack.c | 8 ++++++++
 1 file changed, 8 insertions(+)
---
diff --git a/src/core/stack.c b/src/core/stack.c
index 95223e48e4..50e9e7e04f 100644
--- a/src/core/stack.c
+++ b/src/core/stack.c
@@ -1091,6 +1091,10 @@ meta_stack_get_default_focus_window_at_point (MetaStack     *stack,
                                               int            root_x,
                                               int            root_y)
 {
+  g_return_val_if_fail (META_IS_STACK (stack), NULL);
+  g_return_val_if_fail (META_IS_WORKSPACE (workspace), NULL);
+  g_return_val_if_fail (!not_this_one || META_IS_WINDOW (not_this_one), NULL);
+
   return get_default_focus_window (stack, workspace, not_this_one,
                                    TRUE, root_x, root_y);
 }
@@ -1100,6 +1104,10 @@ meta_stack_get_default_focus_window (MetaStack     *stack,
                                      MetaWorkspace *workspace,
                                      MetaWindow    *not_this_one)
 {
+  g_return_val_if_fail (META_IS_STACK (stack), NULL);
+  g_return_val_if_fail (META_IS_WORKSPACE (workspace), NULL);
+  g_return_val_if_fail (!not_this_one || META_IS_WINDOW (not_this_one), NULL);
+
   return get_default_focus_window (stack, workspace, not_this_one,
                                    FALSE, 0, 0);
 }


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]