Re: [dpdk-dev] [PATCH 1/4] kvargs: support list value

2018-10-10 Thread Remy Horton
On 09/10/2018 16:11, Stephen Hemminger wrote: On Tue, 09 Oct 2018 16:31:24 +0200 [..] This is getting complex enough that doing a real parser maybe necessary. I thought the same thing back in April with the port representor patchsets. Quickly run out of things to use as delimiters otherwis

Re: [dpdk-dev] [PATCH 1/4] kvargs: support list value

2018-10-09 Thread Thomas Monjalon
09/10/2018 17:11, Stephen Hemminger: > On Tue, 09 Oct 2018 16:31:24 +0200 > Thomas Monjalon wrote: > > > 09/10/2018 16:14, Gaëtan Rivet: > > > Hi Thomas, > > > > > > On Tue, Oct 09, 2018 at 04:18:55AM +0200, Thomas Monjalon wrote: > > > > If a value contains a comma, rte_kvargs_tokenize() will

Re: [dpdk-dev] [PATCH 1/4] kvargs: support list value

2018-10-09 Thread Stephen Hemminger
On Tue, 09 Oct 2018 16:31:24 +0200 Thomas Monjalon wrote: > 09/10/2018 16:14, Gaëtan Rivet: > > Hi Thomas, > > > > On Tue, Oct 09, 2018 at 04:18:55AM +0200, Thomas Monjalon wrote: > > > If a value contains a comma, rte_kvargs_tokenize() will split here. > > > In order to support list syntax [a

Re: [dpdk-dev] [PATCH 1/4] kvargs: support list value

2018-10-09 Thread Thomas Monjalon
09/10/2018 16:14, Gaëtan Rivet: > Hi Thomas, > > On Tue, Oct 09, 2018 at 04:18:55AM +0200, Thomas Monjalon wrote: > > If a value contains a comma, rte_kvargs_tokenize() will split here. > > In order to support list syntax [a,b] as value, an extra parsing of > > the square brackets is added. > > >

Re: [dpdk-dev] [PATCH 1/4] kvargs: support list value

2018-10-09 Thread Gaëtan Rivet
Hi Thomas, On Tue, Oct 09, 2018 at 04:18:55AM +0200, Thomas Monjalon wrote: > If a value contains a comma, rte_kvargs_tokenize() will split here. > In order to support list syntax [a,b] as value, an extra parsing of > the square brackets is added. > Nice, I was actually planning to do this. I t

[dpdk-dev] [PATCH 1/4] kvargs: support list value

2018-10-08 Thread Thomas Monjalon
If a value contains a comma, rte_kvargs_tokenize() will split here. In order to support list syntax [a,b] as value, an extra parsing of the square brackets is added. Signed-off-by: Thomas Monjalon --- lib/librte_kvargs/rte_kvargs.c | 10 ++ test/test/test_kvargs.c| 17 +++