> > 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 _______________________________________________ Linux-nvdimm mailing list [email protected] https://lists.01.org/mailman/listinfo/linux-nvdimm
