Terry J. Reedy <[email protected]> added the comment:
I decided to commit the current patch (i12510c.test.diff + NEWS changes) as is.
Another change is needed to correctly delete the first arg of all methods, not
just instance methods with the first arg named 'self'. (In the current new
classmethod test, I just allowed non-removal of 'cls'. There should also be a
test with a variant of 'self'.)
In _self_pat = re.compile('self\,?\s*'), 'self' in the pattern should be
replaced by '(xxx', where 'xxx' is the pattern that matches an identifier. (And
add '(' to the replacement pattern.) If anyone can quickly tell me what that
is, that would be helpful.
----------
_______________________________________
Python tracker <[email protected]>
<http://bugs.python.org/issue12510>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe:
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
