[dpdk-dev] [PATCH v4 1/4] lib/librte_flow_classify: remove table id parameter from apis

[Singh, Jasvinder]

> -----Original Message-----
> From: Thomas Monjalon [mailto:thomas at monjalon.net]
> Sent: Wednesday, January 10, 2018 10:53 AM
> To: Singh, Jasvinder <jasvinder.singh at intel.com>
> Cc: dev at dpdk.org; Iremonger, Bernard <bernard.iremonger at intel.com>;
> Kovacevic, Marko <marko.kovacevic at intel.com>
> Subject: Re: [dpdk-dev] [PATCH v4 1/4] lib/librte_flow_classify: remove table
> id parameter from apis
> 
> 10/01/2018 10:54, Singh, Jasvinder:
> > From: Thomas Monjalon [mailto:thomas at monjalon.net]
> > > 19/12/2017 15:29, Jasvinder Singh:
> > > > This patch removes table id parameter from all the flow classify
> > > > apis to reduce the complexity alongwith some code cleanup.
> > > >
> > > > The validate api is exposed as public api to allow user to
> > > > validate the flow before adding it to the classifier.
> > >
> > > This patch does not compile alone.
> > > Should we merge all patches together?
> >
> > No, It should compile alone. On my system it doesn't show any error and
> compilation goes successfully. Could you send me error log, etc ?
> 
> test/test/test_flow_classify.c:48:10: fatal error: too many arguments to
>       function call, expected 6, have 7
>                         NULL, NULL);
>                               ^~~~
> 
> I guess it is expected that test and example does not compile after patch 1.
> That's why I suggest to merge all 4 patches.

Ok, I will send revised patch with all merged into one.

Thanks,
Jasvinder