30 Jul
2014
30 Jul
'14
2:23 p.m.
Le 30/07/2014 15:48, Serhiy Storchaka a écrit :
Ignoring tests and comments my patch adds/removes/modifies about 200 lines, and David's patch -- about 150 lines of code. But it's __sizeof__ looks not correct, correcting it requires changing about 50 lines. In sum the complexity of both patches is about equal.
I meant that David's approach is conceptually simpler, which makes it easier to review. Regardless, there is no exclusive-OR here: if you can improve over the current version, there's no reason not to consider it/
I didn't look at David's patch too close yet. But my patch includes optimization for end-of-line scanning.
Ahah, unrelated stuff :-)