mpeg: fix unused-but-set warning
authorMatthew Waters <matthew@centricular.com>
Mon, 28 Mar 2022 07:12:43 +0000 (18:12 +1100)
committerGStreamer Marge Bot <gitlab-merge-bot@gstreamer-foundation.org>
Mon, 28 Mar 2022 10:30:23 +0000 (10:30 +0000)
../gst-libs/gst/mpegts/gst-dvb-section.c:206:9: error: variable 'i' set but not used [-Werror,-Wunused-but-set-variable]
  guint i = 0, allocated_events = 12;
        ^
../gst-libs/gst/mpegts/gst-dvb-section.c:365:9: error: variable 'i' set but not used [-Werror,-Wunused-but-set-variable]
  guint i = 0, allocated_streams = 12;
        ^
../gst-libs/gst/mpegts/gst-dvb-section.c:543:9: error: variable 'i' set but not used [-Werror,-Wunused-but-set-variable]
  guint i = 0, allocated_streams = 12;
        ^
../gst-libs/gst/mpegts/gst-dvb-section.c:885:9: error: variable 'i' set but not used [-Werror,-Wunused-but-set-variable]
  guint i = 0, allocated_services = 8;
        ^
../gst-libs/gst/mpegts/gst-dvb-section.c:1316:9: error: variable 'i' set but not used [-Werror,-Wunused-but-set-variable]
  guint i = 0, allocated_services = 8;
        ^

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2046>

subprojects/gst-plugins-bad/gst-libs/gst/mpegts/gst-dvb-section.c

index d41723f..9beca81 100644 (file)
@@ -203,7 +203,7 @@ static gpointer
 _parse_eit (GstMpegtsSection * section)
 {
   GstMpegtsEIT *eit = NULL;
-  guint i = 0, allocated_events = 12;
+  guint allocated_events = 12;
   guint8 *data, *end, *duration_ptr;
   guint16 descriptors_loop_length;
 
@@ -267,8 +267,6 @@ _parse_eit (GstMpegtsSection * section)
     if (event->descriptors == NULL)
       goto error;
     data += descriptors_loop_length;
-
-    i += 1;
   }
 
   if (data != end - 4) {
@@ -362,7 +360,7 @@ static gpointer
 _parse_bat (GstMpegtsSection * section)
 {
   GstMpegtsBAT *bat = NULL;
-  guint i = 0, allocated_streams = 12;
+  guint allocated_streams = 12;
   guint8 *data, *end, *entry_begin;
   guint16 descriptors_loop_length, transport_stream_loop_length;
 
@@ -444,7 +442,6 @@ _parse_bat (GstMpegtsSection * section)
 
     data += descriptors_loop_length;
 
-    i += 1;
     transport_stream_loop_length -= data - entry_begin;
   }
 
@@ -540,7 +537,7 @@ static gpointer
 _parse_nit (GstMpegtsSection * section)
 {
   GstMpegtsNIT *nit = NULL;
-  guint i = 0, allocated_streams = 12;
+  guint allocated_streams = 12;
   guint8 *data, *end, *entry_begin;
   guint16 descriptors_loop_length, transport_stream_loop_length;
 
@@ -625,7 +622,6 @@ _parse_nit (GstMpegtsSection * section)
 
     data += descriptors_loop_length;
 
-    i += 1;
     transport_stream_loop_length -= data - entry_begin;
   }
 
@@ -882,7 +878,7 @@ static gpointer
 _parse_sdt (GstMpegtsSection * section)
 {
   GstMpegtsSDT *sdt = NULL;
-  guint i = 0, allocated_services = 8;
+  guint allocated_services = 8;
   guint8 *data, *end, *entry_begin;
   guint tmp;
   guint sdt_info_length;
@@ -954,7 +950,6 @@ _parse_sdt (GstMpegtsSection * section)
     data += descriptors_loop_length;
 
     sdt_info_length -= data - entry_begin;
-    i += 1;
   }
 
   if (data != end - 4) {
@@ -1313,7 +1308,7 @@ static gpointer
 _parse_sit (GstMpegtsSection * section)
 {
   GstMpegtsSIT *sit = NULL;
-  guint i = 0, allocated_services = 8;
+  guint allocated_services = 8;
   guint8 *data, *end, *entry_begin;
   guint sit_info_length;
   guint descriptors_loop_length;
@@ -1373,7 +1368,6 @@ _parse_sit (GstMpegtsSection * section)
     data += descriptors_loop_length;
 
     sit_info_length -= data - entry_begin;
-    i += 1;
   }
 
   if (data != end - 4) {