[gimp] Revert "app: handle gtk_accel_map_change_entry () failure..."



commit 4a9a0723f5a4e8cbd5e5a5e5978d01df9bad70a0
Author: Jehan <jehan girinstud io>
Date:   Fri Nov 25 06:05:32 2016 +0100

    Revert "app: handle gtk_accel_map_change_entry () failure..."
    
    This reverts commit eab4929a782ef63a06284515f8e518592152ee47.
    
    Oups it would seem gtk_accel_map_change_entry() could return FALSE even
    when the expected shortcut is correctly set (my guess is that it was
    already the same shortcut, so indeed no "change" happened, though it is
    not a failure either; yet I haven't checked if that is the actual
    reason).
    Let's just revert this. It's not always a good thing to be too thorough!
    Sorry for this!

 app/actions/windows-actions.c |   10 +++-------
 1 files changed, 3 insertions(+), 7 deletions(-)
---
diff --git a/app/actions/windows-actions.c b/app/actions/windows-actions.c
index eea4f63..a92292c 100644
--- a/app/actions/windows-actions.c
+++ b/app/actions/windows-actions.c
@@ -431,13 +431,9 @@ windows_actions_update_display_accels (GimpActionGroup *group)
           else
             accel_key = GDK_KEY_0;
 
-          if (! gtk_accel_map_change_entry (accel_path,
-                                            accel_key, GDK_MOD1_MASK,
-                                            TRUE))
-            {
-              g_warning ("%s: failed to set accelerator for %s",
-                         G_STRFUNC, accel_path);
-            }
+          gtk_accel_map_change_entry (accel_path,
+                                      accel_key, GDK_MOD1_MASK,
+                                      TRUE);
         }
     }
 }


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