diff options
| author | Matthias Clasen <mclasen@redhat.com> | 2012-11-29 22:45:24 -0500 |
|---|---|---|
| committer | Matthias Clasen <mclasen@redhat.com> | 2013-01-03 17:59:26 -0500 |
| commit | f294563685f09736a3f184c1d6588d9d188b7179 (patch) | |
| tree | facff95d6316126c1c4e6d29ba237d32d2813f8b | |
| parent | b85098c819a7cebf944e227c98276638ca72a8ac (diff) | |
| download | gtk+-f294563685f09736a3f184c1d6588d9d188b7179.tar.gz | |
GtkExpander: Clarify a nested switch
This code confused Coverity into thinking we were falling through,
when we were not. Add a few explicit breaks to clear that up.
| -rw-r--r-- | gtk/gtkexpander.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/gtk/gtkexpander.c b/gtk/gtkexpander.c index 6a98910b06..d4253e8588 100644 --- a/gtk/gtkexpander.c +++ b/gtk/gtkexpander.c @@ -1251,6 +1251,7 @@ get_next_site (GtkExpander *expander, case GTK_DIR_RIGHT: return FOCUS_WIDGET; } + break; case FOCUS_WIDGET: switch (direction) { @@ -1264,8 +1265,8 @@ get_next_site (GtkExpander *expander, return FOCUS_LABEL; case GTK_DIR_RIGHT: return ltr ? FOCUS_LABEL : FOCUS_NONE; - break; } + break; case FOCUS_LABEL: switch (direction) { @@ -1279,8 +1280,8 @@ get_next_site (GtkExpander *expander, return FOCUS_CHILD; case GTK_DIR_RIGHT: return ltr ? FOCUS_CHILD : FOCUS_WIDGET; - break; } + break; case FOCUS_CHILD: switch (direction) { @@ -1293,6 +1294,7 @@ get_next_site (GtkExpander *expander, case GTK_DIR_RIGHT: return FOCUS_NONE; } + break; } g_assert_not_reached (); |