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
