7 Aug
2013
7 Aug
'13
12:57 p.m.
On 22/07/13 16:46, Andrew Gregory wrote:
This creates a generic ini parser than can be reused for hooks or other configuration files. I took a very different approach from the previous patchset by utilizing a callback that processes the options as the file is parsed. This should have a much lower memory footprint and is closer to how we were already parsing config files.
Parsing differences: * Include directives can be outside a section. They can include their own sections so it made little sense to require that they be in one already. * Errors in included files are now fatal.
These patches look fine to me and it sets us up for parsing ini based hook configuration files. Any other comments before I commit these? Allan