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 DDA21428B1; Tue, 4 Apr 2023 17:59:08 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id BAFE440EF0; Tue, 4 Apr 2023 17:59:08 +0200 (CEST) Received: from linux.microsoft.com (linux.microsoft.com [13.77.154.182]) by mails.dpdk.org (Postfix) with ESMTP id 8F85540A7E for ; Tue, 4 Apr 2023 17:59:07 +0200 (CEST) Received: by linux.microsoft.com (Postfix, from userid 1086) id DF337210DD98; Tue, 4 Apr 2023 08:59:06 -0700 (PDT) DKIM-Filter: OpenDKIM Filter v2.11.0 linux.microsoft.com DF337210DD98 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.microsoft.com; s=default; t=1680623946; bh=sO5bQQhxwwRScLMXfshwtuAke7aP06fkorGAWxYsCSQ=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=ifUfH/CEkFMPg2SSSS13FUEvWZkuNBallOISZ0JD8JPO7px1MnGtnLr9irI4u5yl7 Iq+j3tzf7Ch0E4vh/yeO0mbomeyc09lYmblWDdBbVmRGQWTn/fncgF2P1kiFZNQAZd 12siVthYXner2PMuvtQrmy7ImPnRIARcPNn261BY= Date: Tue, 4 Apr 2023 08:59:06 -0700 From: Tyler Retzlaff To: Konstantin Ananyev Cc: "dev@dpdk.org" , "ciara.power@intel.com" , "bruce.richardson@intel.com" , "david.marchand@redhat.com" , "thomas@monjalon.net" Subject: Re: [PATCH v3] telemetry: use portable syntax to initialize array Message-ID: <20230404155906.GE23247@linuxonhyperv3.guj3yctzbm1etfxqx2vob5hsef.xx.internal.cloudapp.net> References: <1680539424-20255-1-git-send-email-roretzla@linux.microsoft.com> <1680548365-16525-1-git-send-email-roretzla@linux.microsoft.com> <1680548365-16525-2-git-send-email-roretzla@linux.microsoft.com> <65480e4a272445779710dcc4c6f21dfa@huawei.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <65480e4a272445779710dcc4c6f21dfa@huawei.com> User-Agent: Mutt/1.5.21 (2010-09-15) 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 On Tue, Apr 04, 2023 at 09:01:50AM +0000, Konstantin Ananyev wrote: > > > > -----Original Message----- > > From: Tyler Retzlaff > > Sent: Monday, April 3, 2023 7:59 PM > > To: dev@dpdk.org > > Cc: ciara.power@intel.com; bruce.richardson@intel.com; david.marchand@redhat.com; thomas@monjalon.net; Tyler Retzlaff > > > > Subject: [PATCH v3] telemetry: use portable syntax to initialize array > > > > Use of ranges in designated initialization are a non-standard gcc > > extension. Use loops to initialize permitted characters on first use. > > > > Signed-off-by: Tyler Retzlaff > > --- > > lib/telemetry/telemetry_data.c | 22 +++++++++++++++------- > > 1 file changed, 15 insertions(+), 7 deletions(-) > > > > diff --git a/lib/telemetry/telemetry_data.c b/lib/telemetry/telemetry_data.c > > index 2bac2de..562b387 100644 > > --- a/lib/telemetry/telemetry_data.c > > +++ b/lib/telemetry/telemetry_data.c > > @@ -152,13 +152,21 @@ > > static bool > > valid_name(const char *name) > > { > > - char allowed[128] = { > > - ['0' ... '9'] = 1, > > - ['A' ... 'Z'] = 1, > > - ['a' ... 'z'] = 1, > > - ['_'] = 1, > > - ['/'] = 1, > > - }; > > + int index; > > + static bool initialized; > > + static char allowed[128]; > > + > > + if (!initialized) { > > + for (index = '0'; index <= '9'; index++) > > + allowed[index] = 1; > > + for (index = 'A'; index <= 'Z'; index++) > > + allowed[index] = 1; > > + for (index = 'a'; index <= 'z'; index++) > > + allowed[index] = 1; > > + allowed[(int)'_'] = allowed[(int)'/'] = 1; > > + initialized = true; > > + } > > + > > while (*name != '\0') { > > if ((size_t)*name >= RTE_DIM(allowed) || allowed[(int)*name] == 0) > > return false; > > Wonder isn't the same as: > while (*name != 0) > if (!isascii(name[0] || (!isalnum(name[0]) && name[0] != '_' && name[0] != '/')) > return false; > > Or MSVC doesn't support these macros? it's standard C msvc supports it. Bruce acceptable to take Konstantin's suggestion here? > > > > > -- > > 1.8.3.1