On 10/25/2016 07:15 PM, Luis Machado wrote:
Before i go ahead and adjust this even more, what's your plan and ETA
for the above?
I'll try to post this today.
This is disturbing more code as we try to consolidade
slightly different functions into a single one in order to make things a
bit more clean. But i'm afraid this is besides the point of the original
patch itself?
I just want to understand what's the end goal, because the scope seems
to be changing slightly with each iteration. :-)
No, the scope has not changed at all. Your original version duplicated
a large chunk of code, and then the attempt to refactor things did it
incorrectly. Still the same scope, but the patch as is, is buggy.
Between accepted duplicated code, and fixing the patch, I take the latter.