Thomas Wouters added the comment:
I prefer the current method, as it's more obviously walking in two
strides across the same array. I also dislike hiding the final memmove()
of the tail bit inside the loop. As for which is more obvious, I would
submit neither is obvious, as it took me quite a bit of brainsweat to
figure out how either version was supposed to work after not looking at
the code for months :)
Committed revision 57619.
----------
assignee: -> twouters
resolution: -> fixed
status: open -> closed
_____________________________________
Tracker <report@bugs.python.org>
<http://bugs.python.org/issue1617687>
_____________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com
I prefer the current method, as it's more obviously walking in two
strides across the same array. I also dislike hiding the final memmove()
of the tail bit inside the loop. As for which is more obvious, I would
submit neither is obvious, as it took me quite a bit of brainsweat to
figure out how either version was supposed to work after not looking at
the code for months :)
Committed revision 57619.
----------
assignee: -> twouters
resolution: -> fixed
status: open -> closed
_____________________________________
Tracker <report@bugs.python.org>
<http://bugs.python.org/issue1617687>
_____________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com