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 mrabarnett
Recipients brian.curtin, ezio.melotti, mrabarnett, serhiy.storchaka, techmaurice, vstinner
Date 2013-01-24.03:53:01
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1358999581.3.0.808903557849.issue13169@psf.upfronthosting.co.za>
In-reply-to
Content
IMHO, I don't think that MAXREPEAT should be defined in sre_constants.py _and_ SRE_MAXREPEAT defined in sre_constants.h. (In the latter case, why is it in decimal?)

I think that it should be defined in one place, namely sre_constants.h, perhaps as:

#define SRE_MAXREPEAT ~(SRE_CODE)0

and then imported into sre_constants.py.

That'll reduce the chance of an inadvertent mismatch, and it's the C code that's imposing the limit to the number of repeats, not the Python code.
History
Date User Action Args
2013-01-24 03:53:01mrabarnettsetrecipients: + mrabarnett, vstinner, ezio.melotti, brian.curtin, techmaurice, serhiy.storchaka
2013-01-24 03:53:01mrabarnettsetmessageid: <1358999581.3.0.808903557849.issue13169@psf.upfronthosting.co.za>
2013-01-24 03:53:01mrabarnettlinkissue13169 messages
2013-01-24 03:53:01mrabarnettcreate