Here is a patch for discussion. It is similar to the patch sent last
week, with the exception that I have added two condition checks that
ensure the behaviour is the same with this patch as without. Net
reduction in code size is 4 lines, but it also moves all the config file
parsing to one loop.
My belief is that either one of the two condition checks are not
necessary. IMO, the condition check prior to the config parsing loop
where it checks for file existence can be removed. We will then rely on
the check after config parsing which tests for config_opts['root'] to
ensure a proper config is given.
Patch is for discussion. I can send an updated version after decisions
Show replies by date