Message165058
Thank you for the 'positive lookbehind assertion' for anchoring the match after '('. That is what I was missing. '[^,]' is too broad as, in particular, it matches the ')' in 'a(self)'. We don't want the ')' removed. The tests passed with a faulty re because the method test was faulty in that it only tested that something is removed, but not that the removal is correct. I fixed the tests also to really test proper removal. |
|
Date |
User |
Action |
Args |
2012-07-09 04:17:32 | terry.reedy | set | recipients:
+ terry.reedy, ned.deily, roger.serwy, python-dev, Roy.Fox |
2012-07-09 04:17:32 | terry.reedy | set | messageid: <1341807452.43.0.635732107393.issue12510@psf.upfronthosting.co.za> |
2012-07-09 04:17:31 | terry.reedy | link | issue12510 messages |
2012-07-09 04:17:31 | terry.reedy | create | |
|