From: Luis de Bethencourt <luisbg@osg.samsung.com>
Date: Wed, 18 Nov 2015 16:01:48 +0000 (+0000)
Subject: matroskamux: remove duplicate check
X-Git-Tag: 1.10.4~742
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=40aa27b788c0b13250312d3f876406b27229a91a;p=platform%2Fupstream%2Fgst-plugins-good.git

matroskamux: remove duplicate check

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.
---

diff --git a/gst/matroska/matroska-mux.c b/gst/matroska/matroska-mux.c
index d60b863..2f2d762 100644
--- a/gst/matroska/matroska-mux.c
+++ b/gst/matroska/matroska-mux.c
@@ -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 */