From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 5A2E246801 for ; Tue, 27 May 2025 11:22:50 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 5402B40144; Tue, 27 May 2025 11:22:50 +0200 (CEST) Received: from dpdk.org (dpdk.org [92.243.24.197]) by mails.dpdk.org (Postfix) with ESMTP id D873040144 for ; Tue, 27 May 2025 11:22:48 +0200 (CEST) Received: by dpdk.org (Postfix, from userid 65534) id C1306125267; Tue, 27 May 2025 11:22:10 +0200 (CEST) Subject: |WARNING| pw153744 [PATCH 3/3] argparse: use enums to remove max-value defines in lists In-Reply-To: <20250527092113.903910-4-bruce.richardson@intel.com> References: <20250527092113.903910-4-bruce.richardson@intel.com> To: test-report@dpdk.org From: checkpatch@dpdk.org Cc: Bruce Richardson Message-Id: <20250527092210.C1306125267@dpdk.org> Date: Tue, 27 May 2025 11:22:10 +0200 (CEST) X-BeenThere: test-report@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: automatic DPDK test reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: test-report-bounces@dpdk.org Test-Label: checkpatch Test-Status: WARNING http://dpdk.org/patch/153744 _coding style issues_ WARNING:LONG_LINE: line length of 129 exceeds 100 columns #107: FILE: app/test/test_argparse.c:82: + { "--abc", "-a", "abc argument", (void *)1, (void *)1, RTE_ARGPARSE_VALUE_NONE, RTE_ARGPARSE_VALUE_TYPE_NONE }, \ WARNING:LONG_LINE: line length of 129 exceeds 100 columns #108: FILE: app/test/test_argparse.c:83: + { "--xyz", "-x", "xyz argument", (void *)1, (void *)2, RTE_ARGPARSE_VALUE_NONE, RTE_ARGPARSE_VALUE_TYPE_NONE }, \ WARNING:LONG_LINE: line length of 101 exceeds 100 columns #606: FILE: app/test/test_argparse.c:845: + ret = rte_argparse_parse_type(bool_numeric_invalid, RTE_ARGPARSE_VALUE_TYPE_BOOL, &val_bool); ERROR:SWITCH_CASE_INDENT_LEVEL: switch and case should be at the same indent #753: FILE: lib/argparse/rte_argparse.c:36: + switch (arg->value_required) { + case RTE_ARGPARSE_VALUE_NONE: + case RTE_ARGPARSE_VALUE_OPTIONAL: + case RTE_ARGPARSE_VALUE_REQUIRED: ERROR:SWITCH_CASE_INDENT_LEVEL: switch and case should be at the same indent #770: FILE: lib/argparse/rte_argparse.c:49: + switch (arg->value_type) { + case RTE_ARGPARSE_VALUE_TYPE_NONE: + case RTE_ARGPARSE_VALUE_TYPE_INT: + case RTE_ARGPARSE_VALUE_TYPE_U8: + case RTE_ARGPARSE_VALUE_TYPE_U16: + case RTE_ARGPARSE_VALUE_TYPE_U32: + case RTE_ARGPARSE_VALUE_TYPE_U64: + case RTE_ARGPARSE_VALUE_TYPE_STR: + case RTE_ARGPARSE_VALUE_TYPE_BOOL: WARNING:LONG_LINE: line length of 103 exceeds 100 columns #965: FILE: lib/argparse/rte_argparse.c:670: + ARGPARSE_LOG(ERR, "argument %s doesn't have value!", arg_name); total: 2 errors, 4 warnings, 954 lines checked