[gimp] Revert "app: handle gtk_accel_map_change_entry () failure..."
- From: Jehan Pagès <jehanp src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gimp] Revert "app: handle gtk_accel_map_change_entry () failure..."
- Date: Fri, 25 Nov 2016 05:15:00 +0000 (UTC)
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]