Re: [dpdk-dev] [PATCH v4 3/6] cfgfile: add support for configurable comment character

2017-03-31 Thread Legacy, Allain
> -Original Message- > From: Thomas Monjalon [mailto:thomas.monja...@6wind.com] > Sent: Friday, March 31, 2017 6:08 AM <...> > > + * @return > > + * Handle to configuration file on success, NULL otherwise > > + * @param > > This @param should be removed. Ok. I will remove it. > > >

Re: [dpdk-dev] [PATCH v4 3/6] cfgfile: add support for configurable comment character

2017-03-31 Thread Thomas Monjalon
Hi, 2017-03-30 14:54, Allain Legacy: > /** > + * Open config file with specified optional parameters. > + * > + * @param filename > + * Config file name > + * @param flags > + * Config file flags > + * @param params > + * Additional configuration attributes. Must be configured with desired

[dpdk-dev] [PATCH v4 3/6] cfgfile: add support for configurable comment character

2017-03-30 Thread Allain Legacy
The current cfgfile comment character is hardcoded to ';'. This commit a new API to allow the user to specify which comment character to use while parsing the file. This is to ease adoption by applications that have an existing configuration file which may use a different comment character. For