Results 5 comments of Loay

@mamartins @btassone I will have a look today at the PR.

yeah I was about to wonder why did you close the main PR then I saw a replacement. I will try to rebase myself and see where it goes.

here is an update: The pr needs units tests which is a must before merging to detect any regression. The rebasing will take some time too. The code itself looks...