[PATCH 03/17] shell: Add missing cases to switch statements for animations and fading

Philip Withnall philip at tecnocode.co.uk
Mon Nov 25 10:01:32 PST 2013


From: Philip Withnall <philip.withnall at collabora.co.uk>

This fixes two GCC warnings when compiling with -Wswitch-enum
-Wswitch-default, and makes it clearer that those cases have been
thought about explicitly when writing the code, rather than just being
forgotten.
---
 src/shell.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/src/shell.c b/src/shell.c
index 1a35d54..17a64ae 100644
--- a/src/shell.c
+++ b/src/shell.c
@@ -3973,6 +3973,8 @@ shell_fade_done(struct weston_view_animation *animation, void *data)
 	case FADE_OUT:
 		lock(shell);
 		break;
+	default:
+		break;
 	}
 }
 
@@ -4374,6 +4376,7 @@ map(struct desktop_shell *shell, struct shell_surface *shsurf,
 		case ANIMATION_ZOOM:
 			weston_zoom_run(shsurf->view, 0.5, 1.0, NULL, NULL);
 			break;
+		case ANIMATION_NONE:
 		default:
 			break;
 		}
-- 
1.8.3.1



More information about the wayland-devel mailing list