video-converter: bits variable always set
authorLuis de Bethencourt <luis.bg@samsung.com>
Tue, 10 Feb 2015 15:25:04 +0000 (15:25 +0000)
committerLuis de Bethencourt <luis.bg@samsung.com>
Tue, 10 Feb 2015 15:25:04 +0000 (15:25 +0000)
In function gst_video_scaler_vertical() the bits variable is always
set to either 8 or 16 in every possible format. No need to initialize it.

If the format isn't valid it goes to no_func, so there is no need to
handle the case of bits not being 8 or 16.

CID #1268401

gst-libs/gst/video/video-scaler.c

index 5d3f685a77f9e342355982dbc83e2333daed9f8b..94eb02b346eed4324cbb7079d7768b55689c85f6 100644 (file)
@@ -1256,7 +1256,7 @@ void
 gst_video_scaler_vertical (GstVideoScaler * scale, GstVideoFormat format,
     gpointer src_lines[], gpointer dest, guint dest_offset, guint width)
 {
-  gint n_elems, bits = 0;
+  gint n_elems, bits;
   GstVideoScalerVFunc func;
 
   g_return_if_fail (scale != NULL);
@@ -1340,8 +1340,7 @@ gst_video_scaler_vertical (GstVideoScaler * scale, GstVideoFormat format,
         func = video_scale_v_ntap_u16;
         break;
     }
-  } else
-    goto no_func;
+  }
 
   if (scale->tmpwidth < width)
     realloc_tmplines (scale, n_elems, width);