From d1b0cf4625c1efa309d696c9c6100b242b63d3d8 Mon Sep 17 00:00:00 2001 From: Niklas Jang Date: Sat, 12 Oct 2019 15:41:03 +0900 Subject: [PATCH] [CodeClean] remove redundant check for LLVM compatibility As bdata->mode is defined as bounding_box_modes type, it is never lower than zero. Because find_key_strv()'s return value is definded as 'The count of replaced. (Set NULL if it is unnecessary)', it does not affect. And not latest clang complains about this redundance.(LLVM) Signed-off-by: Niklas Jang --- ext/nnstreamer/tensor_decoder/tensordec-boundingbox.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ext/nnstreamer/tensor_decoder/tensordec-boundingbox.c b/ext/nnstreamer/tensor_decoder/tensordec-boundingbox.c index bee9822..3b18f1a 100644 --- a/ext/nnstreamer/tensor_decoder/tensordec-boundingbox.c +++ b/ext/nnstreamer/tensor_decoder/tensordec-boundingbox.c @@ -379,7 +379,7 @@ bb_setOption (void **pdata, int opNum, const char *param) bounding_box_modes previous = bdata->mode; bdata->mode = find_key_strv (bb_modes, param); - if (NULL == param || *param == '\0' || bdata->mode < 0) { + if (NULL == param || *param == '\0') { GST_ERROR ("Please set the valid mode at option1"); return FALSE; } -- 2.7.4