Andrei Kulakov <[email protected]> added the comment:
Irit: I assume you mean r' \r?\n', that's a great idea, it's much faster than
adding a separate replacement step.
Latest version I came up with is this:
if re.search(r' \r?\n', text):
text = re.sub(r' \r?\n', ' ', text)
if re.search(r'\r?\n ', text):
text = re.sub(r'\r?\n ', ' ', text)
This optimizes the case when there's no newlines, which is likely the most
common case for small fragments of text, but it may be the less common case for
larger fragments where performance is more important; so I'm not sure if it's
worth it.
Timings:
# sub() has to run
2904 (~/opensource/cpython) % ./python.exe -mtimeit 'import textwrap'
'textwrap.wrap("abc foo\n bar baz", 5)' ----VICMD----
5000 loops, best of 5: 67.6 usec per loop
# search() runs; but sub() does NOT because there's no adjacent space
2906 (~/opensource/cpython) % ./python.exe -mtimeit 'import textwrap'
'textwrap.wrap("abc foo\nbar baz", 5)' ----VICMD----
5000 loops, best of 5: 60.3 usec per loop
----------
_______________________________________
Python tracker <[email protected]>
<https://bugs.python.org/issue32397>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe:
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
