From 73ac0c4e502449481f03b7ec7b6282503dfaac6f Mon Sep 17 00:00:00 2001 From: "U. Artie Eoff" Date: Thu, 19 Sep 2019 12:09:20 -0700 Subject: [PATCH] gst: encode: h264_fei: remove useless comparison The expression "len >= 0" is always true since "len" is an unsigned type. And it is clear that the writers intention was not to write "len > 0" since we handle len == 0 in the ensuing "if (len < 3)" conditional block. --- gst/vaapi/gstvaapiencode_h264_fei.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gst/vaapi/gstvaapiencode_h264_fei.c b/gst/vaapi/gstvaapiencode_h264_fei.c index 0b1b74a..c09eaea 100644 --- a/gst/vaapi/gstvaapiencode_h264_fei.c +++ b/gst/vaapi/gstvaapiencode_h264_fei.c @@ -241,7 +241,7 @@ _h264_byte_stream_next_nal (guint8 * buffer, guint32 len, guint32 * nal_size) guint32 flag = 0xFFFFFFFF; guint32 nal_start_len = 0; - g_assert (len >= 0 && buffer && nal_size); + g_assert (buffer && nal_size); if (len < 3) { *nal_size = len; nal_start = (len ? buffer : NULL); -- 2.7.4