Hi Atul, that patch looks quite neat! I really like this kind of refactoring of yours where you split a more complex function into smaller parts and call these smaller parts instead. The code is so much readable this way. I might have an idea how using the functor would not be necessary but will have to try it out… and let you know.
The patch is pushed in revno 94 of your public branch.
Regards, Peter
