From 10d0ed65aa9b6f753c839d797f1d34ed8c7ca6de Mon Sep 17 00:00:00 2001
From: Alessandro Bono <alessandro.bono369@gmail.com>
Date: Tue, 17 Sep 2024 21:27:59 +0200
Subject: [PATCH] prefs: Invert if statement condition

Mimic what we already do for the locate-pointer. There is
no need for an empty "then" branch.

Part-of: <https://gitlab.gnome.org/GNOME/mutter/-/merge_requests/4050>
---
 src/core/prefs.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/src/core/prefs.c b/src/core/prefs.c
index 78d48629c..fcbd5de56 100644
--- a/src/core/prefs.c
+++ b/src/core/prefs.c
@@ -1476,9 +1476,7 @@ overlay_key_handler (GVariant *value,
   *result = NULL; /* ignored */
   string_value = g_variant_get_string (value, NULL);
 
-  if (string_value && meta_parse_accelerator (string_value, &combo))
-    ;
-  else
+  if (!string_value || !meta_parse_accelerator (string_value, &combo))
     {
       meta_topic (META_DEBUG_KEYBINDINGS,
                   "Failed to parse value for overlay-key");