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 01FAE423F2; Mon, 16 Jan 2023 18:51:45 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id B807642D22; Mon, 16 Jan 2023 18:51:33 +0100 (CET) Received: from mail-pj1-f47.google.com (mail-pj1-f47.google.com [209.85.216.47]) by mails.dpdk.org (Postfix) with ESMTP id 78A2C40DFD for ; Mon, 16 Jan 2023 18:51:32 +0100 (CET) Received: by mail-pj1-f47.google.com with SMTP id z4-20020a17090a170400b00226d331390cso31727924pjd.5 for ; Mon, 16 Jan 2023 09:51:32 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20210112.gappssmtp.com; s=20210112; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=D1LXiex4yZHdRsIE0fd4Au3i4tYpZuFGgLXg+NdabeU=; b=mniN+zV3IyoQrtS1SgumRHMUcTIY90d4X83SXfR9NAYDWCT36/Okrz+A4tJUUI0Z5b p1c/VmEdDqkgmcBqryyf1NS6c5NQXOLrtuTJGf/YV+4OhklXfaiGiquyti7uo+3wK2tz 8v2XQHQ4Se8HWvDHFEXRJuKHOn6+Y4hQnPWV2pcWU2Xe0Ozgk2k3RpVzKvkIc94KmGFS zTXQ0b89jnhxq56yNCdW7Vosy23dMrEGASDCABXPtkOsJBXvjY2MmotvECOfhoSks9LL P9ja3aPwA4wRQRjOWHSvAuOa7egqfTRJZT1Q8hELLbAwxIbgxx/ru9+Gf+2xfzUNbfML mxfA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=D1LXiex4yZHdRsIE0fd4Au3i4tYpZuFGgLXg+NdabeU=; b=WQ/rfvMuYm8aYlnVXmAJsOfK5PTCnC7JGMpUvqQ0u3wMFQMxDURmQpVaqzRWyV08en noCWTBeZakpmX+gJvAWtZ6wBYpoV5dxBbdKVGfn47djlovRKAct9oeRQqIwGeM9UXoJv ZSYini9wdl35l3d5amk/JOs9lkJbmllvXvYT9brpMcZWraaigc1rVfepEoPKGEvRoKPo yHaoD3UUDkxeeJrA+7BJNNsFDX8Dxmh8c97sKaIGL6/oUTcP0NufXrXnk3055GlLnKQA t0r8fxbfLDhvRQBudH617IMnGIwYs+UA8nf2aeyhdsIVujngxlasY26ViTuh0Znx5Qpe V5vQ== X-Gm-Message-State: AFqh2kqEjgg/E1rYMp1qSxUp2VP0mmK+TSPj1U1fc8/dTroigR9ejgR6 IPPNFw2I8R4nbSs1XxmSCI7Sz6CAu42WQAnMO/Y= X-Google-Smtp-Source: AMrXdXsL/7ScReWJw77pgC7X6uJRljlVltszNVaLljgVCEaOaDBYyp6rBkWFziUQA+Nrdnemvnx+dw== X-Received: by 2002:a17:902:9696:b0:194:a52b:d3f8 with SMTP id n22-20020a170902969600b00194a52bd3f8mr425513plp.32.1673891491416; Mon, 16 Jan 2023 09:51:31 -0800 (PST) Received: from hermes.local (204-195-120-218.wavecable.com. [204.195.120.218]) by smtp.gmail.com with ESMTPSA id k15-20020a170902d58f00b001947c22185bsm5035795plh.184.2023.01.16.09.51.30 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 16 Jan 2023 09:51:31 -0800 (PST) From: Stephen Hemminger To: dev@dpdk.org Cc: Stephen Hemminger , Aman Singh , Yuying Zhang Subject: [PATCH v2 2/7] testpmd: fix whitespace Date: Mon, 16 Jan 2023 09:51:22 -0800 Message-Id: <20230116175127.87346-3-stephen@networkplumber.org> X-Mailer: git-send-email 2.39.0 In-Reply-To: <20230116175127.87346-1-stephen@networkplumber.org> References: <20230116172732.84976-1-stephen@networkplumber.org> <20230116175127.87346-1-stephen@networkplumber.org> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org The style standard is to use blank after keywords. I.e "if (" not "if(" Signed-off-by: Stephen Hemminger --- app/test-pmd/cmdline.c | 33 +++++++++++++++++---------------- app/test-pmd/parameters.c | 11 ++++++----- app/test-pmd/testpmd.c | 2 +- 3 files changed, 24 insertions(+), 22 deletions(-) diff --git a/app/test-pmd/cmdline.c b/app/test-pmd/cmdline.c index b32dc8bfd445..d2a52ac7b723 100644 --- a/app/test-pmd/cmdline.c +++ b/app/test-pmd/cmdline.c @@ -2728,13 +2728,14 @@ parse_reta_config(const char *str, unsigned long int_fld[_NUM_FLD]; char *str_fld[_NUM_FLD]; - while ((p = strchr(p0,'(')) != NULL) { + while ((p = strchr(p0, '(')) != NULL) { ++p; - if((p0 = strchr(p,')')) == NULL) + p0 = strchr(p, ')'); + if (p0 == NULL) return -1; size = p0 - p; - if(size >= sizeof(s)) + if (size >= sizeof(s)) return -1; snprintf(s, sizeof(s), "%.*s", size, p); @@ -3242,15 +3243,15 @@ cmd_config_thresh_parsed(void *parsed_result, if (!strcmp(res->name, "txpt")) tx_pthresh = res->value; - else if(!strcmp(res->name, "txht")) + else if (!strcmp(res->name, "txht")) tx_hthresh = res->value; - else if(!strcmp(res->name, "txwt")) + else if (!strcmp(res->name, "txwt")) tx_wthresh = res->value; - else if(!strcmp(res->name, "rxpt")) + else if (!strcmp(res->name, "rxpt")) rx_pthresh = res->value; - else if(!strcmp(res->name, "rxht")) + else if (!strcmp(res->name, "rxht")) rx_hthresh = res->value; - else if(!strcmp(res->name, "rxwt")) + else if (!strcmp(res->name, "rxwt")) rx_wthresh = res->value; else { fprintf(stderr, "Unknown parameter\n"); @@ -4084,8 +4085,8 @@ cmd_vlan_offload_parsed(void *parsed_result, len = strnlen(str, STR_TOKEN_SIZE); i = 0; /* Get port_id first */ - while(i < len){ - if(str[i] == ',') + while (i < len) { + if (str[i] == ',') break; i++; @@ -4093,7 +4094,7 @@ cmd_vlan_offload_parsed(void *parsed_result, str[i]='\0'; tmp = strtoul(str, NULL, 0); /* If port_id greater that what portid_t can represent, return */ - if(tmp >= RTE_MAX_ETHPORTS) + if (tmp >= RTE_MAX_ETHPORTS) return; port_id = (portid_t)tmp; @@ -4104,17 +4105,17 @@ cmd_vlan_offload_parsed(void *parsed_result, if (!strcmp(res->what, "strip")) rx_vlan_strip_set(port_id, on); - else if(!strcmp(res->what, "stripq")){ + else if (!strcmp(res->what, "stripq")) { uint16_t queue_id = 0; /* No queue_id, return */ - if(i + 1 >= len) { + if (i + 1 >= len) { fprintf(stderr, "must specify (port,queue_id)\n"); return; } tmp = strtoul(str + i + 1, NULL, 0); /* If queue_id greater that what 16-bits can represent, return */ - if(tmp > 0xffff) + if (tmp > 0xffff) return; queue_id = (uint16_t)tmp; @@ -7252,7 +7253,7 @@ static void cmd_mac_addr_parsed(void *parsed_result, ret = rte_eth_dev_mac_addr_remove(res->port_num, &res->address); /* check the return value and print it if is < 0 */ - if(ret < 0) + if (ret < 0) fprintf(stderr, "mac_addr_cmd error: (%s)\n", strerror(-ret)); } @@ -7825,7 +7826,7 @@ static void cmd_vf_mac_addr_parsed(void *parsed_result, res->vf_num); #endif - if(ret < 0) + if (ret < 0) fprintf(stderr, "vf_mac_addr_cmd error: (%s)\n", strerror(-ret)); } diff --git a/app/test-pmd/parameters.c b/app/test-pmd/parameters.c index d597c209ba5e..5f8f962bc93d 100644 --- a/app/test-pmd/parameters.c +++ b/app/test-pmd/parameters.c @@ -306,13 +306,14 @@ parse_portnuma_config(const char *q_arg) char *str_fld[_NUM_FLD]; /* reset from value set at definition */ - while ((p = strchr(p0,'(')) != NULL) { + while ((p = strchr(p0, '(')) != NULL) { ++p; - if((p0 = strchr(p,')')) == NULL) + p0 = strchr(p, ')'); + if (p0 == NULL) return -1; size = p0 - p; - if(size >= sizeof(s)) + if (size >= sizeof(s)) return -1; snprintf(s, sizeof(s), "%.*s", size, p); @@ -368,11 +369,11 @@ parse_ringnuma_config(const char *q_arg) /* reset from value set at definition */ while ((p = strchr(p0,'(')) != NULL) { ++p; - if((p0 = strchr(p,')')) == NULL) + if ((p0 = strchr(p,')')) == NULL) return -1; size = p0 - p; - if(size >= sizeof(s)) + if (size >= sizeof(s)) return -1; snprintf(s, sizeof(s), "%.*s", size, p); diff --git a/app/test-pmd/testpmd.c b/app/test-pmd/testpmd.c index 134d79a55547..c38e8fecc50a 100644 --- a/app/test-pmd/testpmd.c +++ b/app/test-pmd/testpmd.c @@ -2408,7 +2408,7 @@ start_packet_forwarding(int with_tx_first) test_done = 0; - if(!no_flush_rx) + if (!no_flush_rx) flush_fwd_rx_queues(); rxtx_config_display(); -- 2.39.0