mpegts: remove storage of never used values
authorLuis de Bethencourt <luis.bg@samsung.com>
Thu, 30 Oct 2014 14:27:38 +0000 (14:27 +0000)
committerLuis de Bethencourt <luis.bg@samsung.com>
Thu, 30 Oct 2014 23:19:59 +0000 (23:19 +0000)
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

index 459a323..7532548 100644 (file)
@@ -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;