Message43251
Logged In: YES
user_id=7887
Greg, thank you very much for the patch. I've included the
fixes without introducing the reorganization mentioned, for
the sake of stability. Also, the second fix mentioned in
your report don't fix the mentioned problem anymore, because
of the change introduced by patch #720991. The new fix
wasn't complicated though, and is included as well.
About the reorganization, I don't have a strong opinion
about it, and I'm also not sure about the real impact the
new "mark reset" code will cause in the module. IMO we need
to test it further.
Andrew, that explains why the second fix greg mentions
doesn't work anymore. Thanks for mentioning it.
The changes were applied as:
Modules/_sre.c: 2.91
Lib/test/re_tests.py: 1.35
Lib/test/test_sre.py: 1.41
|
|
Date |
User |
Action |
Args |
2007-08-23 15:21:50 | admin | link | issue712900 messages |
2007-08-23 15:21:50 | admin | create | |
|