[libvirt] Re: [augeas-devel] PATCH: Add a augeas lens for libvirtd.conf

> While I can see why it is useful to gather these entries logically, it's not > very optimised for the parser. Eventually, all this will be compiled into a > huge regexp, so it's more efficient to regroup entries by type and feed the > functions with regexps instead of strings, like Is performance really that much of a problem ? The libvirt config file only has 20 or so different settings, and while we'll add more I can't see it getting very much larger. Changing these settings is also not something that would be done on a frequent basis / performance critical path. I find it more readable to group them by functional area unless there is a serious real world performance issue.