From f76d1a25140c0d475e300ae2ba93c8d3e55e2272 Mon Sep 17 00:00:00 2001 From: Luis de Bethencourt Date: Thu, 30 Oct 2014 14:27:38 +0000 Subject: [PATCH] mpegts: remove storage of never used values Both _parse_atsc_mgt() and _parse_atsc_vct () change the value of the variable data just before returning. The new value is never used since data is a pointer declared at the beginning of the function and going out of scope just after the new value is stored. https://bugzilla.gnome.org/show_bug.cgi?id=739404 --- gst-libs/gst/mpegts/gst-atsc-section.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/gst-libs/gst/mpegts/gst-atsc-section.c b/gst-libs/gst/mpegts/gst-atsc-section.c index 459a323e0f..7532548950 100644 --- a/gst-libs/gst/mpegts/gst-atsc-section.c +++ b/gst-libs/gst/mpegts/gst-atsc-section.c @@ -193,7 +193,6 @@ _parse_atsc_vct (GstMpegtsSection * section) gst_mpegts_parse_descriptors (data, descriptors_loop_length); if (vct->descriptors == NULL) goto error; - data += descriptors_loop_length; return (gpointer) vct; @@ -361,7 +360,6 @@ _parse_atsc_mgt (GstMpegtsSection * section) } mgt->descriptors = gst_mpegts_parse_descriptors (data, descriptors_loop_length); - data += descriptors_loop_length; return (gpointer) mgt;