diff options
author | Luis de Bethencourt <luis.bg@samsung.com> | 2014-09-25 16:04:33 +0100 |
---|---|---|
committer | Luis de Bethencourt <luis.bg@samsung.com> | 2014-09-25 16:41:15 +0100 |
commit | efca04f23fc4dc52036fd61186e8acb7d49ebe81 (patch) | |
tree | 18e9938c4becafdee5c7453ca7ba2e15c8e8b08d | |
parent | 97ed6a26684e40b99458ca11c39a8145c8a2b5eb (diff) |
style: remove unnecessary break statements
Code flow never arrives to these break statements due to the gotos preceding
them. So the breaks are unnecessary, as discussed in bug 736942.
https://bugzilla.gnome.org/show_bug.cgi?id=737370
-rw-r--r-- | gst/dvdspu/gstdvdspu.c | 1 | ||||
-rw-r--r-- | gst/jp2kdecimator/jp2kcodestream.c | 10 | ||||
-rw-r--r-- | gst/jpegformat/gstjifmux.c | 1 | ||||
-rw-r--r-- | gst/nuvdemux/gstnuvdemux.c | 1 | ||||
-rw-r--r-- | gst/videoparsers/gsth265parse.c | 2 | ||||
-rw-r--r-- | gst/videoparsers/gstmpeg4videoparse.c | 1 |
6 files changed, 0 insertions, 16 deletions
diff --git a/gst/dvdspu/gstdvdspu.c b/gst/dvdspu/gstdvdspu.c index 2c270fd9b..57447266e 100644 --- a/gst/dvdspu/gstdvdspu.c +++ b/gst/dvdspu/gstdvdspu.c @@ -1170,7 +1170,6 @@ gst_dvd_spu_subpic_event (GstPad * pad, GstObject * parent, GstEvent * event) * video might still continue, though */ gst_event_unref (event); goto done; - break; default: res = gst_pad_event_default (pad, parent, event); break; diff --git a/gst/jp2kdecimator/jp2kcodestream.c b/gst/jp2kdecimator/jp2kcodestream.c index 943d83734..af2c0471e 100644 --- a/gst/jp2kdecimator/jp2kcodestream.c +++ b/gst/jp2kdecimator/jp2kcodestream.c @@ -1152,22 +1152,18 @@ parse_tile (GstJP2kDecimator * self, GstByteReader * reader, GST_ERROR_OBJECT (self, "COC marker not supported yet"); ret = GST_FLOW_ERROR; goto done; - break; case MARKER_POC: GST_ERROR_OBJECT (self, "POC marker not supported yet"); ret = GST_FLOW_ERROR; goto done; - break; case MARKER_RGN: GST_ERROR_OBJECT (self, "RGN marker not supported yet"); ret = GST_FLOW_ERROR; goto done; - break; case MARKER_PPT: GST_ERROR_OBJECT (self, "PPT marker not supported yet"); ret = GST_FLOW_ERROR; goto done; - break; case MARKER_PLT:{ PacketLengthTilePart *plt = g_slice_new (PacketLengthTilePart); @@ -1521,32 +1517,26 @@ parse_main_header (GstJP2kDecimator * self, GstByteReader * reader, GST_ERROR_OBJECT (self, "POC marker not supported yet"); ret = GST_FLOW_ERROR; goto done; - break; case MARKER_COC: GST_ERROR_OBJECT (self, "COC marker not supported yet"); ret = GST_FLOW_ERROR; goto done; - break; case MARKER_RGN: GST_ERROR_OBJECT (self, "RGN marker not supported yet"); ret = GST_FLOW_ERROR; goto done; - break; case MARKER_TLM: GST_ERROR_OBJECT (self, "TLM marker not supported yet"); ret = GST_FLOW_ERROR; goto done; - break; case MARKER_PLM: GST_ERROR_OBJECT (self, "PLM marker not supported yet"); ret = GST_FLOW_ERROR; goto done; - break; case MARKER_PPM: GST_ERROR_OBJECT (self, "PPM marker not supported yet"); ret = GST_FLOW_ERROR; goto done; - break; case MARKER_QCD: if (header->qcd.data != NULL) { GST_ERROR_OBJECT (self, "Multiple QCD markers"); diff --git a/gst/jpegformat/gstjifmux.c b/gst/jpegformat/gstjifmux.c index 28638b733..719067a3b 100644 --- a/gst/jpegformat/gstjifmux.c +++ b/gst/jpegformat/gstjifmux.c @@ -312,7 +312,6 @@ gst_jif_mux_parse_image (GstJifMux * self, GstBuffer * buf) m = gst_jif_mux_new_marker (marker, 0, NULL, FALSE); self->priv->markers = g_list_prepend (self->priv->markers, m); goto done; - break; default: if (!gst_byte_reader_get_uint16_be (&reader, &size)) goto error; diff --git a/gst/nuvdemux/gstnuvdemux.c b/gst/nuvdemux/gstnuvdemux.c index 5bd29a5ad..ce4e98dad 100644 --- a/gst/nuvdemux/gstnuvdemux.c +++ b/gst/nuvdemux/gstnuvdemux.c @@ -717,7 +717,6 @@ gst_nuv_demux_play (GstPad * pad) break; case GST_NUV_DEMUX_INVALID_DATA: goto pause; - break; default: g_assert_not_reached (); } diff --git a/gst/videoparsers/gsth265parse.c b/gst/videoparsers/gsth265parse.c index caf461691..fa4ad50e6 100644 --- a/gst/videoparsers/gsth265parse.c +++ b/gst/videoparsers/gsth265parse.c @@ -866,7 +866,6 @@ gst_h265_parse_handle_frame (GstBaseParse * parse, case GST_H265_PARSER_NO_NAL: *skipsize = size - 3; goto skip; - break; default: g_assert_not_reached (); break; @@ -930,7 +929,6 @@ gst_h265_parse_handle_frame (GstBaseParse * parse, nalu.offset = nalu.sc_offset; goto end; } - break; default: g_assert_not_reached (); break; diff --git a/gst/videoparsers/gstmpeg4videoparse.c b/gst/videoparsers/gstmpeg4videoparse.c index 1afaa17c8..a16936306 100644 --- a/gst/videoparsers/gstmpeg4videoparse.c +++ b/gst/videoparsers/gstmpeg4videoparse.c @@ -509,7 +509,6 @@ next: mp4vparse->last_sc = size - 3; } goto out; - break; default: /* decide whether this startcode ends a frame */ ret = gst_mpeg4vparse_process_sc (mp4vparse, &packet, size); |