This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author serhiy.storchaka
Recipients ezio.melotti, joente, mrabarnett, serhiy.storchaka
Date 2015-11-05.10:31:24
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1446719484.97.0.0543862962304.issue25554@psf.upfronthosting.co.za>
In-reply-to
Content
Thank you for your report and patch Jeroen.

Indeed, there is a regression, and your patch fixes it. But I don't like the idea of using weakref. For now sre_parse has very little dependencies, but weakref depends on collections that depends on a number of modules. For now importing weakref works, but it is too easy to create a dependency loop in future.

Here is alternative patch that gets rid of references at all. The subpatterns list was added in the patch for issue9179 and is an implementation detail. We can replace it with a list of subpattern widths.
History
Date User Action Args
2015-11-05 10:31:25serhiy.storchakasetrecipients: + serhiy.storchaka, ezio.melotti, mrabarnett, joente
2015-11-05 10:31:24serhiy.storchakasetmessageid: <1446719484.97.0.0543862962304.issue25554@psf.upfronthosting.co.za>
2015-11-05 10:31:24serhiy.storchakalinkissue25554 messages
2015-11-05 10:31:24serhiy.storchakacreate