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 | | |