rtspclientsink: clean switch statements
authorLuis de Bethencourt <luisbg@osg.samsung.com>
Mon, 8 Feb 2016 23:33:03 +0000 (23:33 +0000)
committerLuis de Bethencourt <luisbg@osg.samsung.com>
Mon, 8 Feb 2016 23:33:22 +0000 (23:33 +0000)
Coverity demands for fallthrough statements to be clearly commented,
to distinguish from accidental fall throughs. And it also needs all
cases to finish with a break, even if the break is never going to be
executed like in the case of a continue jump.

CID #1352039
CID #1352040

gst/rtsp-sink/gstrtspclientsink.c

index 3d5076f..8c378eb 100644 (file)
@@ -1949,6 +1949,7 @@ gst_rtsp_client_sink_loop_rx (GstRTSPClientSink * sink)
           goto server_eof;
         }
         continue;
+        break;
       case GST_RTSP_ENET:
         GST_DEBUG_OBJECT (sink, "An ethernet problem occured.");
       default:
@@ -4414,6 +4415,7 @@ gst_rtsp_client_sink_handle_message (GstBin * bin, GstMessage * message)
             gst_element_state_get_name (newstate),
             gst_element_state_get_name (pending), rtsp_client_sink->prerolled);
       }
+      /* fallthrough */
     }
     default:
     {