RE: [ndctl PATCH v13 2/5] ndctl, monitor: add main ndctl monitor configuration file
> Hi,
>
> I have a future cleanup request. I've just come across, ciniparser
> [1]. It would be great if we could drop the open coded implementation and just
use that library.
> We use ccan modules for other portions of ndctl.
>
> [1]: https://ccodearchive.net/info/ciniparser.html
Hi Dan,
It sounds good.
I will study on it a bit and then make a cleanup patch.
Thank you very much.
Qi
Hi,
I have finished importing the ccan/ciniparser module and refactoring the
read_config_file() in my local branch.
But thinking more about that more than one configuration file might be added into ndctl in
future,
I want to add a common function parse_config_file() into ndctl/util/ to replace
read_config_file().
If you agree, I will try to make a patch.
Thank you very much.
QI