mirror of
https://gitlab.freedesktop.org/gstreamer/gstreamer.git
synced 2024-11-23 18:21:04 +00:00
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
This commit is contained in:
parent
97ed6a2668
commit
efca04f23f
6 changed files with 0 additions and 16 deletions
|
@ -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;
|
||||
|
|
|
@ -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");
|
||||
|
|
|
@ -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;
|
||||
|
|
|
@ -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 ();
|
||||
}
|
||||
|
|
|
@ -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;
|
||||
|
|
|
@ -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);
|
||||
|
|
Loading…
Reference in a new issue