kvargs: make pointers in string arrays const
authorBruce Richardson <bruce.richardson@intel.com>
Thu, 12 Jan 2017 16:18:27 +0000 (16:18 +0000)
committerThomas Monjalon <thomas.monjalon@6wind.com>
Fri, 13 Jan 2017 18:28:26 +0000 (19:28 +0100)
commit0615355ffae4e3ef53cd27703061e9e87cd2f640
tree4b4cd8a86fd3fc7cef164322f091099be6d177ba
parentd74cbeca7596b3237fe5b823fb8a4bcfe8a7d979
kvargs: make pointers in string arrays const

Change the parameters of functions from const char *valid[] to
const char * const valid[]. This additional const is needed to
allow us to fix some checkpatch warnings, as well as being good
programming practice.

For the checkpatch warnings, if we have a set of command line
args that we want to check defined as:
static const char *args[] = { "arg1", "arg2", NULL };
kvlist = rte_kvargs_parse(params, args);

checkpatch will complain:
WARNING:STATIC_CONST_CHAR_ARRAY: static const char *
array should probably be static const char * const

Adding the additional const to the definition of the args
will then trigger a compiler error in the absence of this
change to the kvargs library, as we lose the const in the
call to kvargs_parse.

Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
Acked-by: Olivier Matz <olivier.matz@6wind.com>
lib/librte_kvargs/rte_kvargs.c
lib/librte_kvargs/rte_kvargs.h