we-like-parsers / cpython

Here we work on integrating pegen into CPython; use branch 'pegen'
https://github.com/gvanrossum/pegen
Other
1 stars 0 forks source link

Speed-up f-string tokenization by reducing strlen() calls #203

Closed isidentical closed 1 year ago

isidentical commented 1 year ago

This took a bit of longer than expected since multiline f-strings did not work with basic arithmetic but I think the current form should work (would love if you folks can also double-check it).

pablogsal commented 1 year ago

Do we have tests that fail if we introduce some bug here?

isidentical commented 1 year ago

We don't but I've manually checked test_fstrings numbers (it was 87 failures before, 87 failures now).