diff options
author | David King <amigadave@amigadave.com> | 2015-01-29 13:08:22 +0000 |
---|---|---|
committer | David King <amigadave@amigadave.com> | 2015-02-16 21:37:33 +0000 |
commit | 5c98c8d5ba6f59a6e51b45839efdaf227eb890d7 (patch) | |
tree | 57752266c449e7a11b8ef885a828968ee0bf7404 | |
parent | 8ff1b22ad51e8237838a27e26a908b60726a2f24 (diff) |
Avoid warnings from -Wswitch-enum
Check switch statements where several cases are intentionally not
handled by using an if instead.
-rw-r--r-- | libcheese/cheese-camera.c | 22 | ||||
-rw-r--r-- | libcheese/cheese-widget.c | 10 |
2 files changed, 10 insertions, 22 deletions
diff --git a/libcheese/cheese-camera.c b/libcheese/cheese-camera.c index a98ef16f..49cd837f 100644 --- a/libcheese/cheese-camera.c +++ b/libcheese/cheese-camera.c @@ -190,10 +190,11 @@ static void cheese_camera_bus_message_cb (GstBus *bus, GstMessage *message, CheeseCamera *camera) { CheeseCameraPrivate *priv = cheese_camera_get_instance_private (camera); + GstMessageType type; - switch (GST_MESSAGE_TYPE (message)) - { - case GST_MESSAGE_WARNING: + type = GST_MESSAGE_TYPE (message); + + if (type == GST_MESSAGE_WARNING) { GError *err = NULL; gchar *debug = NULL; @@ -207,9 +208,8 @@ cheese_camera_bus_message_cb (GstBus *bus, GstMessage *message, CheeseCamera *ca } g_free (debug); - break; } - case GST_MESSAGE_ERROR: + else if (type == GST_MESSAGE_ERROR) { GError *err = NULL; gchar *debug = NULL; @@ -226,9 +226,8 @@ cheese_camera_bus_message_cb (GstBus *bus, GstMessage *message, CheeseCamera *ca g_signal_emit (camera, camera_signals[STATE_FLAGS_CHANGED], 0, GST_STATE_NULL); g_free (debug); - break; } - case GST_MESSAGE_STATE_CHANGED: + else if (type == GST_MESSAGE_STATE_CHANGED) { if (strcmp (GST_MESSAGE_SRC_NAME (message), "camerabin") == 0) { @@ -241,9 +240,8 @@ cheese_camera_bus_message_cb (GstBus *bus, GstMessage *message, CheeseCamera *ca priv->effect_pipeline_is_playing); } } - break; } - case GST_MESSAGE_ELEMENT: + else if (type == GST_MESSAGE_ELEMENT) { const GstStructure *structure; GstSample *sample; @@ -286,13 +284,7 @@ cheese_camera_bus_message_cb (GstBus *bus, GstMessage *message, CheeseCamera *ca priv->is_recording = FALSE; } } - break; } - default: - { - break; - } - } } /* diff --git a/libcheese/cheese-widget.c b/libcheese/cheese-widget.c index 23619d6a..586824dd 100644 --- a/libcheese/cheese-widget.c +++ b/libcheese/cheese-widget.c @@ -283,14 +283,10 @@ static void webcam_state_changed (CheeseCamera *camera, GstState state, CheeseWidget *widget) { - switch (state) + if (state == GST_STATE_NULL) { - case GST_STATE_NULL: - cheese_widget_set_problem_page (widget, "error"); - break; - default: - /* TODO: Handle other cases. */ - break; + cheese_widget_set_problem_page (widget, "error"); + /* TODO: Handle other cases. */ } } |