[dpdk-dev] [PATCH 1/1] test_table: added ACL table test to the list
Gajdzica, MaciejX T
maciejx.t.gajdzica at intel.com
Wed Aug 12 14:58:22 CEST 2015
> -----Original Message-----
> From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com]
> Sent: Wednesday, August 12, 2015 2:53 PM
> To: Gajdzica, MaciejX T
> Cc: dev at dpdk.org
> Subject: Re: [dpdk-dev] [PATCH 1/1] test_table: added ACL table test to the list
>
> 2015-08-12 14:41, Maciej Gajdzica:
> > test_table_ACL wasn't invoked when running table_autotest. Added this
> > test to table_tests array to make it run with other table tests.
> >
> > Signed-off-by: Maciej Gajdzica <maciejx.t.gajdzica at intel.com>
> [...]
> > --- a/app/test/test_table_tables.h
> > +++ b/app/test/test_table_tables.h
> > @@ -36,7 +36,7 @@ int test_table_lpm(void); int
> > test_table_lpm_ipv6(void); int test_table_array(void); #ifdef
> > RTE_LIBRTE_ACL -int test_table_acl(void);
> > +int test_table_ACL(void);
>
> Why renaming it uppercase?
> Function names should be lowercase.
>
> It is lowercase also in app/test/test_table_combined.h
Yes, but in app/test/test_table_acl.c it is uppercase, and there is no definition for lowercase function. I can change name in test_table_acl.c to lowercase.
Best Regards
Maciek
More information about the dev
mailing list