Re: [dpdk-dev] [PATCH v3 6/6] cfgfile: add support for empty value string

2017-03-29 Thread Legacy, Allain
> -Original Message- > From: Dumitrescu, Cristian [mailto:cristian.dumitre...@intel.com] > Sent: Wednesday, March 29, 2017 5:31 AM <...> > > +#define CFG_FLAG_EMPTY_VALUES (1 << 1) > > /**@} */ > > > > I suggest we group all these flags into an (unnamed?) enum in rte_cfgfile.h > rather t

Re: [dpdk-dev] [PATCH v3 6/6] cfgfile: add support for empty value string

2017-03-29 Thread Dumitrescu, Cristian
> -Original Message- > From: Allain Legacy [mailto:allain.leg...@windriver.com] > Sent: Wednesday, March 29, 2017 1:48 AM > To: Dumitrescu, Cristian ; Richardson, Bruce > > Cc: dev@dpdk.org; yuanhan@linux.intel.com; > thomas.monja...@6wind.com > Subject: [PATCH v3 6/6] cfgfile: add s

[dpdk-dev] [PATCH v3 6/6] cfgfile: add support for empty value string

2017-03-28 Thread Allain Legacy
This commit adds support to the cfgfile library for parsing a key=value line that has no value string specified (e.g., "key="). This can be used to override a configuration attribute that has a default value or default list of values to set it back to an undefined value to disable functionality.