matroskamux: remove duplicate check
authorLuis de Bethencourt <luisbg@osg.samsung.com>
Wed, 18 Nov 2015 16:01:48 +0000 (16:01 +0000)
committerLuis de Bethencourt <luisbg@osg.samsung.com>
Wed, 18 Nov 2015 16:06:27 +0000 (16:06 +0000)
We want 1 or 2 streamheaders, the check  if (bufarr->len != 1 &&
bufarr->len != 2) is enough. Not need to check if bufarr->len is <= 0 or
> 255.

gst/matroska/matroska-mux.c

index d60b863..2f2d762 100644 (file)
@@ -1616,9 +1616,7 @@ opus_streamheader_to_codecdata (const GValue * streamheader,
     goto wrong_type;
 
   bufarr = g_value_peek_pointer (streamheader);
-  if (bufarr->len <= 0 || bufarr->len > 255)    /* one header, and count stored in a byte */
-    goto wrong_count;
-  if (bufarr->len != 1 && bufarr->len != 2)
+  if (bufarr->len != 1 && bufarr->len != 2)     /* one header, and count stored in a byte */
     goto wrong_count;
 
   /* Opus headers are not in-band */