Re: [dpdk-dev] [PATCH] eal/option: fix option register duplicate detection

2018-12-19 Thread Thomas Monjalon
17/12/2018 11:19, David Marchand: > On Mon, Dec 17, 2018 at 10:26 AM Gaetan Rivet > wrote: > > > Missing brackets around the if means that the loop will end at its first > > iteration. > > > > Cc: sta...@dpdk.org > > > > Fixes: 2395332798d0 ("eal: add option register infrastructure") > > Signed-o

Re: [dpdk-dev] [PATCH] eal/option: fix option register duplicate detection

2018-12-17 Thread David Marchand
On Mon, Dec 17, 2018 at 10:26 AM Gaetan Rivet wrote: > Missing brackets around the if means that the loop will end at its first > iteration. > > Cc: sta...@dpdk.org > > Fixes: 2395332798d0 ("eal: add option register infrastructure") > Signed-off-by: Gaetan Rivet > --- > lib/librte_eal/common/rt

Re: [dpdk-dev] [PATCH] eal/option: fix option register duplicate detection

2018-12-17 Thread Burakov, Anatoly
On 17-Dec-18 9:25 AM, Gaetan Rivet wrote: Missing brackets around the if means that the loop will end at its first iteration. Cc: sta...@dpdk.org Fixes: 2395332798d0 ("eal: add option register infrastructure") Signed-off-by: Gaetan Rivet --- lib/librte_eal/common/rte_option.c | 3 ++- 1 fil

[dpdk-dev] [PATCH] eal/option: fix option register duplicate detection

2018-12-17 Thread Gaetan Rivet
Missing brackets around the if means that the loop will end at its first iteration. Cc: sta...@dpdk.org Fixes: 2395332798d0 ("eal: add option register infrastructure") Signed-off-by: Gaetan Rivet --- lib/librte_eal/common/rte_option.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) dif