diff options
| author | Matthias Clasen <mclasen@redhat.com> | 2012-11-29 22:06:18 -0500 |
|---|---|---|
| committer | Matthias Clasen <mclasen@redhat.com> | 2013-01-03 17:58:45 -0500 |
| commit | 5d551918805f0c938bce36c912c9b41c3af6e1a6 (patch) | |
| tree | b55ee9434233d2d2161c7b91e6cef7b4e94cc86d | |
| parent | 35079629a5392245f2dc9faf983fc0f80a87e4e9 (diff) | |
| download | gtk+-5d551918805f0c938bce36c912c9b41c3af6e1a6.tar.gz | |
Don't check the same crossing mode twice
This was broken since commit b2aaa94 in 2008. Its commit message
clearly states that the intention was to check for GTK_GRAB,
GTK_UNGRAB and STATE_CHANGED. Lets do that, then.
This was found by Coverity.
| -rw-r--r-- | gtk/gtkmenushell.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/gtk/gtkmenushell.c b/gtk/gtkmenushell.c index bc4f5f75ed..81fe3a8e0b 100644 --- a/gtk/gtkmenushell.c +++ b/gtk/gtkmenushell.c @@ -1107,7 +1107,7 @@ gtk_menu_shell_leave_notify (GtkWidget *widget, GdkEventCrossing *event) { if (event->mode == GDK_CROSSING_GTK_GRAB || - event->mode == GDK_CROSSING_GTK_GRAB || + event->mode == GDK_CROSSING_GTK_UNGRAB || event->mode == GDK_CROSSING_STATE_CHANGED) return TRUE; |