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 vstinner
Recipients louielu, serhiy.storchaka, vstinner
Date 2017-11-21.23:38:18
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1511307498.52.0.213398074469.issue31324@psf.upfronthosting.co.za>
In-reply-to
Content
I merged my PR 4421 which is based on Serhiy's PR 4420. Thank you Serhiy for your reviews!

--

Serhiy Storchaka: "We could split patterns on two parts and create both matchers (...) I don't know whether it is worth to do."

My use bisect is to find a regression using the test.bisect tool. According to my tests, the fastest set().__contains__ matcher is always used, except of test_json. If you bisect a bug in test_json, it will work, but just be a little bit slower.

Honestly, I don't think that we can go further in term of optimization. In my tests, it seems like creating the matcher function and filter all tests take less than 1 second, so it's enough for my use case ;-)
History
Date User Action Args
2017-11-21 23:38:18vstinnersetrecipients: + vstinner, serhiy.storchaka, louielu
2017-11-21 23:38:18vstinnersetmessageid: <1511307498.52.0.213398074469.issue31324@psf.upfronthosting.co.za>
2017-11-21 23:38:18vstinnerlinkissue31324 messages
2017-11-21 23:38:18vstinnercreate