GH-130328: Speedup pasting in legacy console on Windows#133728

chris-eibl · 2025-05-09T06:57:33Z

Issue: Pasting in new REPL is slow #130328

chris-eibl · 2025-05-09T07:00:25Z

Lib/_pyrepl/windows_console.py

+recs, rec_count=self._read_input_bulk(1024)
 foriinrange(rec_count):
 rec=recs[i]
+# In case of a legacy console, we do not only receive a keydown


Actually, there is a lot of duplicated logic from get_event now. Maybe refactor that, so the logic can be reused here?

chris-eibl · 2025-05-09T07:02:07Z

Lib/_pyrepl/windows_console.py

 and there is no event pending, otherwise waits for the
 completion of an event."""

+ifnotblockandnotself.wait(timeout=0):


I have moved blocking and waiting logic into get_event like in unix_console.py. IMHO that shouldn't be part of reading and we're in perfect sync with Unix now.

chris-eibl · 2025-05-09T07:05:23Z

Lib/_pyrepl/commands.py

 data=""
 start=time.time()
 whiledonenotindata:
-self.reader.console.wait(100)


I don't think this is needed. Works for me without it. For both, Linux and Windows, this will immediately return in case of pending data, and just needlessly waits 100 ms if there is no more data.

ambv

This is perfect. You utilized my getpending() implementation without adding any double-buffering. It is indeed speeding things up dramatically while not breaking the existing bracketed paste behavior.

miss-islington-app · 2025-05-25T13:17:46Z

Thanks @chris-eibl for the PR, and @ambv for merging it 🌮🎉.. I'm working now to backport this PR to: 3.14.
🐍🍒⛏🤖

…gh-133728) (cherry picked from commit 91b4886) Co-authored-by: Chris Eibl <138194463+chris-eibl@users.noreply.github.com>

bedevere-app · 2025-05-25T13:17:59Z

GH-134653 is a backport of this pull request to the 3.14 branch.

…3728) (#134653) (cherry picked from commit 91b4886) Co-authored-by: Chris Eibl <138194463+chris-eibl@users.noreply.github.com>

bedevere-bot · 2025-05-25T14:24:17Z

⚠️⚠️⚠️ Buildbot failure ⚠️⚠️⚠️

Hi! The buildbot AMD64 Windows10 3.x (tier-1) has failed when building commit 91b4886.

What do you need to do:

Don't panic.
Check the buildbot page in the devguide if you don't know what the buildbots are or how they work.
Go to the page of the buildbot that failed (https://buildbot.python.org/#/builders/146/builds/11526) and take a look at the build logs.
Check if the failure is related to this commit (91b4886) or if it is a false positive.
If the failure is related to this commit, please, reflect that on the issue and make a new Pull Request with a fix.

You can take a look at the buildbot page here:

https://buildbot.python.org/#/builders/146/builds/11526

Failed tests:

test_pyrepl

Failed subtests:

test_up_vt - test.test_pyrepl.test_windows_console.WindowsConsoleGetEventTests.test_up_vt
test_left - test.test_pyrepl.test_windows_console.WindowsConsoleGetEventTests.test_left
test_KEY_EVENT_up_ignored - test.test_pyrepl.test_windows_console.WindowsConsoleGetEventTests.test_KEY_EVENT_up_ignored
test_enter_vt - test.test_pyrepl.test_windows_console.WindowsConsoleGetEventTests.test_enter_vt
test_m_LEFT_ALT_PRESSED_and_LEFT_CTRL_PRESSED - test.test_pyrepl.test_windows_console.WindowsConsoleGetEventTests.test_m_LEFT_ALT_PRESSED_and_LEFT_CTRL_PRESSED
test_WINDOW_BUFFER_SIZE_EVENT - test.test_pyrepl.test_windows_console.WindowsConsoleGetEventTests.test_WINDOW_BUFFER_SIZE_EVENT
test_backspace_vt - test.test_pyrepl.test_windows_console.WindowsConsoleGetEventTests.test_backspace_vt
test_left_RIGHT_CTRL_PRESSED - test.test_pyrepl.test_windows_console.WindowsConsoleGetEventTests.test_left_RIGHT_CTRL_PRESSED
test_AltGr_m - test.test_pyrepl.test_windows_console.WindowsConsoleGetEventTests.test_AltGr_m
test_EmptyBuffer - test.test_pyrepl.test_windows_console.WindowsConsoleGetEventTests.test_EmptyBuffer
test_unhandled_events - test.test_pyrepl.test_windows_console.WindowsConsoleGetEventTests.test_unhandled_events
test_left_RIGHT_ALT_PRESSED - test.test_pyrepl.test_windows_console.WindowsConsoleGetEventTests.test_left_RIGHT_ALT_PRESSED
test_AltGr_7 - test.test_pyrepl.test_windows_console.WindowsConsoleGetEventTests.test_AltGr_7
test_m_LEFT_ALT_PRESSED - test.test_pyrepl.test_windows_console.WindowsConsoleGetEventTests.test_m_LEFT_ALT_PRESSED
test_backspace - test.test_pyrepl.test_windows_console.WindowsConsoleGetEventTests.test_backspace
test_enter - test.test_pyrepl.test_windows_console.WindowsConsoleGetEventTests.test_enter
test_M - test.test_pyrepl.test_windows_console.WindowsConsoleGetEventTests.test_M
test_m - test.test_pyrepl.test_windows_console.WindowsConsoleGetEventTests.test_m
test_umlaut_a_german - test.test_pyrepl.test_windows_console.WindowsConsoleGetEventTests.test_umlaut_a_german
test_left_LEFT_ALT_PRESSED - test.test_pyrepl.test_windows_console.WindowsConsoleGetEventTests.test_left_LEFT_ALT_PRESSED
test_left_LEFT_CTRL_PRESSED - test.test_pyrepl.test_windows_console.WindowsConsoleGetEventTests.test_left_LEFT_CTRL_PRESSED
test_m_RIGHT_ALT_PRESSED - test.test_pyrepl.test_windows_console.WindowsConsoleGetEventTests.test_m_RIGHT_ALT_PRESSED

Summary of the results of the build (if available):

==

Click to see traceback logs

Traceback (most recent call last): File "D:\buildarea\3.x.bolen-windows10\build\Lib\test\test_pyrepl\test_windows_console.py", line 415, in test_WINDOW_BUFFER_SIZE_EVENTself.assertEqual(self.get_event([ir]), Event("resize", "")) ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^AssertionError: None != Event(evt='resize', data='', raw=b'') Traceback (most recent call last): File "D:\buildarea\3.x.bolen-windows10\build\Lib\test\test_pyrepl\test_windows_console.py", line 466, in test_left_RIGHT_CTRL_PRESSEDself.assertEqual( ~~~~~~~~~~~~~~~~^self.get_event([ir]), Event("key", "ctrl left")) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^AssertionError: None != Event(evt='key', data='ctrl left', raw=b'') Traceback (most recent call last): File "D:\buildarea\3.x.bolen-windows10\build\Lib\test\test_pyrepl\test_windows_console.py", line 534, in test_AltGr_7self.assertEqual(self.get_event([ir]), Event("key", "{")) ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^AssertionError: None != Event(evt='key', data='{', raw=b'') Traceback (most recent call last): File "D:\buildarea\3.x.bolen-windows10\build\Lib\test\test_pyrepl\test_windows_console.py", line 454, in test_Mself.assertEqual(self.get_event([ir]), Event("key", "M")) ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^AssertionError: None != Event(evt='key', data='M', raw=b'') Traceback (most recent call last): File "D:\buildarea\3.x.bolen-windows10\build\Lib\test\test_pyrepl\test_windows_console.py", line 562, in test_enter_vtself.assertEqual(self.get_event([ir], vt_support=True), ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Event("key", "\n")) ^^^^^^^^^^^^^^^^^^^AssertionError: None != Event(evt='key', data='\n', raw=b'') Traceback (most recent call last): File "D:\buildarea\3.x.bolen-windows10\build\Lib\test\test_pyrepl\test_windows_console.py", line 449, in test_mself.assertEqual(self.get_event([ir]), Event("key", "m")) ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^AssertionError: None != Event(evt='key', data='m', raw=b'') Traceback (most recent call last): File "D:\buildarea\3.x.bolen-windows10\build\Lib\test\test_pyrepl\test_windows_console.py", line 510, in test_m_LEFT_ALT_PRESSEDself.assertEqual(self.get_event([ir]), Event(evt="key", data="\033")) ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^AssertionError: None != Event(evt='key', data='\x1b', raw=b'') Traceback (most recent call last): File "D:\buildarea\3.x.bolen-windows10\build\Lib\test\test_pyrepl\test_windows_console.py", line 548, in test_umlaut_a_germanself.assertEqual(self.get_event([ir]), Event("key", "�")) ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^AssertionError: None != Event(evt='key', data='�', raw=b'') Traceback (most recent call last): File "D:\buildarea\3.x.bolen-windows10\build\Lib\test\test_pyrepl\test_windows_console.py", line 517, in test_m_RIGHT_ALT_PRESSEDself.assertEqual(self.get_event([ir]), Event(evt="key", data="\033")) ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^AssertionError: None != Event(evt='key', data='\x1b', raw=b'') Traceback (most recent call last): File "D:\buildarea\3.x.bolen-windows10\build\Lib\test\test_pyrepl\test_windows_console.py", line 543, in test_AltGr_mself.assertEqual(self.get_event([ir]), Event("key", "�")) ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^AssertionError: None != Event(evt='key', data='�', raw=b'') Traceback (most recent call last): File "D:\buildarea\3.x.bolen-windows10\build\Lib\test\test_pyrepl\test_windows_console.py", line 443, in test_backspaceself.assertEqual( ~~~~~~~~~~~~~~~~^self.get_event([ir]), Event("key", "backspace")) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^AssertionError: None != Event(evt='key', data='backspace', raw=b'') Traceback (most recent call last): File "D:\buildarea\3.x.bolen-windows10\build\Lib\test\test_pyrepl\test_windows_console.py", line 473, in test_left_LEFT_CTRL_PRESSEDself.assertEqual( ~~~~~~~~~~~~~~~~^self.get_event([ir]), Event("key", "ctrl left")) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^AssertionError: None != Event(evt='key', data='ctrl left', raw=b'') Traceback (most recent call last): File "D:\buildarea\3.x.bolen-windows10\build\Lib\test\test_pyrepl\test_windows_console.py", line 480, in test_left_RIGHT_ALT_PRESSEDself.assertEqual(self.get_event([ir]), Event(evt="key", data="\033")) ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^AssertionError: None != Event(evt='key', data='\x1b', raw=b'') Traceback (most recent call last): File "D:\buildarea\3.x.bolen-windows10\build\Lib\test\test_pyrepl\test_windows_console.py", line 460, in test_leftself.assertEqual(self.get_event([ir]), Event("key", "left")) ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^AssertionError: None != Event(evt='key', data='left', raw=b'') Traceback (most recent call last): File "D:\buildarea\3.x.bolen-windows10\build\Lib\test\test_pyrepl\test_windows_console.py", line 434, in test_unhandled_eventsself.assertEqual(self.mock.call_count, 1) ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^AssertionError: 0 != 1 Traceback (most recent call last): File "D:\buildarea\3.x.bolen-windows10\build\Lib\test\test_pyrepl\test_windows_console.py", line 438, in test_enterself.assertEqual(self.get_event([ir]), Event("key", "\n")) ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^AssertionError: None != Event(evt='key', data='\n', raw=b'') Traceback (most recent call last): File "D:\buildarea\3.x.bolen-windows10\build\Lib\test\test_pyrepl\test_windows_console.py", line 407, in test_EmptyBufferself.assertEqual(self.mock.call_count, 1) ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^AssertionError: 0 != 1 Traceback (most recent call last): File "D:\buildarea\3.x.bolen-windows10\build\Lib\test\test_pyrepl\test_windows_console.py", line 490, in test_left_LEFT_ALT_PRESSEDself.assertEqual(self.get_event([ir]), Event(evt="key", data="\033")) ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^AssertionError: None != Event(evt='key', data='\x1b', raw=b'') Traceback (most recent call last): File "D:\buildarea\3.x.bolen-windows10\build\Lib\test\test_pyrepl\test_windows_console.py", line 574, in test_up_vtself.assertEqual(self.get_event(irs, vt_support=True), ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Event(evt='key', data='up', raw=bytearray(b'\x1b[A'))) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^AssertionError: None != Event(evt='key', data='up', raw=bytearray(b'\x1b[A')) Traceback (most recent call last): File "D:\buildarea\3.x.bolen-windows10\build\Lib\test\test_pyrepl\test_windows_console.py", line 505, in test_m_LEFT_ALT_PRESSED_and_LEFT_CTRL_PRESSEDself.assertEqual(self.mock.call_count, 1) ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^AssertionError: 0 != 1 Traceback (most recent call last): File "D:\buildarea\3.x.bolen-windows10\build\Lib\test\test_pyrepl\test_windows_console.py", line 568, in test_backspace_vtself.assertEqual(self.get_event([ir], vt_support=True), ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Event("key", "backspace", b"\x7f")) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^AssertionError: None != Event(evt='key', data='backspace', raw=b'\x7f') Traceback (most recent call last): File "D:\buildarea\3.x.bolen-windows10\build\Lib\test\test_pyrepl\test_windows_console.py", line 424, in test_KEY_EVENT_up_ignoredself.assertEqual(self.mock.call_count, 1) ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^AssertionError: 0 != 1

ambv · 2025-05-25T14:36:41Z

@chris-eibl this buildbot failure ^^^ is real. I can't explain that yet and cannot reproduce it locally on Win 11, but if I don't manage to fix it before tomorrow's 3.14 beta 2, I will have to revert the backport.

bedevere-bot · 2025-05-25T14:59:01Z

⚠️⚠️⚠️ Buildbot failure ⚠️⚠️⚠️

Hi! The buildbot AMD64 Windows10 3.14 (tier-1) has failed when building commit 93aee56.

What do you need to do:

Don't panic.
Check the buildbot page in the devguide if you don't know what the buildbots are or how they work.
Go to the page of the buildbot that failed (https://buildbot.python.org/#/builders/1711/builds/99) and take a look at the build logs.
Check if the failure is related to this commit (93aee56) or if it is a false positive.
If the failure is related to this commit, please, reflect that on the issue and make a new Pull Request with a fix.

You can take a look at the buildbot page here:

https://buildbot.python.org/#/builders/1711/builds/99

Failed tests:

test_pyrepl

Failed subtests:

test_up_vt - test.test_pyrepl.test_windows_console.WindowsConsoleGetEventTests.test_up_vt
test_left - test.test_pyrepl.test_windows_console.WindowsConsoleGetEventTests.test_left
test_KEY_EVENT_up_ignored - test.test_pyrepl.test_windows_console.WindowsConsoleGetEventTests.test_KEY_EVENT_up_ignored
test_enter_vt - test.test_pyrepl.test_windows_console.WindowsConsoleGetEventTests.test_enter_vt
test_m_LEFT_ALT_PRESSED_and_LEFT_CTRL_PRESSED - test.test_pyrepl.test_windows_console.WindowsConsoleGetEventTests.test_m_LEFT_ALT_PRESSED_and_LEFT_CTRL_PRESSED
test_WINDOW_BUFFER_SIZE_EVENT - test.test_pyrepl.test_windows_console.WindowsConsoleGetEventTests.test_WINDOW_BUFFER_SIZE_EVENT
test_backspace_vt - test.test_pyrepl.test_windows_console.WindowsConsoleGetEventTests.test_backspace_vt
test_left_RIGHT_CTRL_PRESSED - test.test_pyrepl.test_windows_console.WindowsConsoleGetEventTests.test_left_RIGHT_CTRL_PRESSED
test_AltGr_m - test.test_pyrepl.test_windows_console.WindowsConsoleGetEventTests.test_AltGr_m
test_EmptyBuffer - test.test_pyrepl.test_windows_console.WindowsConsoleGetEventTests.test_EmptyBuffer
test_unhandled_events - test.test_pyrepl.test_windows_console.WindowsConsoleGetEventTests.test_unhandled_events
test_left_RIGHT_ALT_PRESSED - test.test_pyrepl.test_windows_console.WindowsConsoleGetEventTests.test_left_RIGHT_ALT_PRESSED
test_AltGr_7 - test.test_pyrepl.test_windows_console.WindowsConsoleGetEventTests.test_AltGr_7
test_m_LEFT_ALT_PRESSED - test.test_pyrepl.test_windows_console.WindowsConsoleGetEventTests.test_m_LEFT_ALT_PRESSED
test_backspace - test.test_pyrepl.test_windows_console.WindowsConsoleGetEventTests.test_backspace
test_enter - test.test_pyrepl.test_windows_console.WindowsConsoleGetEventTests.test_enter
test_M - test.test_pyrepl.test_windows_console.WindowsConsoleGetEventTests.test_M
test_m - test.test_pyrepl.test_windows_console.WindowsConsoleGetEventTests.test_m
test_umlaut_a_german - test.test_pyrepl.test_windows_console.WindowsConsoleGetEventTests.test_umlaut_a_german
test_left_LEFT_ALT_PRESSED - test.test_pyrepl.test_windows_console.WindowsConsoleGetEventTests.test_left_LEFT_ALT_PRESSED
test_left_LEFT_CTRL_PRESSED - test.test_pyrepl.test_windows_console.WindowsConsoleGetEventTests.test_left_LEFT_CTRL_PRESSED
test_m_RIGHT_ALT_PRESSED - test.test_pyrepl.test_windows_console.WindowsConsoleGetEventTests.test_m_RIGHT_ALT_PRESSED

Summary of the results of the build (if available):

==

Click to see traceback logs

Traceback (most recent call last): File "D:\buildarea\3.14.bolen-windows10\build\Lib\test\test_pyrepl\test_windows_console.py", line 466, in test_left_RIGHT_CTRL_PRESSEDself.assertEqual( ~~~~~~~~~~~~~~~~^self.get_event([ir]), Event("key", "ctrl left")) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^AssertionError: None != Event(evt='key', data='ctrl left', raw=b'') Traceback (most recent call last): File "D:\buildarea\3.14.bolen-windows10\build\Lib\test\test_pyrepl\test_windows_console.py", line 449, in test_mself.assertEqual(self.get_event([ir]), Event("key", "m")) ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^AssertionError: None != Event(evt='key', data='m', raw=b'') Traceback (most recent call last): File "D:\buildarea\3.14.bolen-windows10\build\Lib\test\test_pyrepl\test_windows_console.py", line 480, in test_left_RIGHT_ALT_PRESSEDself.assertEqual(self.get_event([ir]), Event(evt="key", data="\033")) ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^AssertionError: None != Event(evt='key', data='\x1b', raw=b'') Traceback (most recent call last): File "D:\buildarea\3.14.bolen-windows10\build\Lib\test\test_pyrepl\test_windows_console.py", line 473, in test_left_LEFT_CTRL_PRESSEDself.assertEqual( ~~~~~~~~~~~~~~~~^self.get_event([ir]), Event("key", "ctrl left")) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^AssertionError: None != Event(evt='key', data='ctrl left', raw=b'') Traceback (most recent call last): File "D:\buildarea\3.14.bolen-windows10\build\Lib\test\test_pyrepl\test_windows_console.py", line 407, in test_EmptyBufferself.assertEqual(self.mock.call_count, 1) ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^AssertionError: 0 != 1 Traceback (most recent call last): File "D:\buildarea\3.14.bolen-windows10\build\Lib\test\test_pyrepl\test_windows_console.py", line 454, in test_Mself.assertEqual(self.get_event([ir]), Event("key", "M")) ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^AssertionError: None != Event(evt='key', data='M', raw=b'') Traceback (most recent call last): File "D:\buildarea\3.14.bolen-windows10\build\Lib\test\test_pyrepl\test_windows_console.py", line 438, in test_enterself.assertEqual(self.get_event([ir]), Event("key", "\n")) ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^AssertionError: None != Event(evt='key', data='\n', raw=b'') Traceback (most recent call last): File "D:\buildarea\3.14.bolen-windows10\build\Lib\test\test_pyrepl\test_windows_console.py", line 460, in test_leftself.assertEqual(self.get_event([ir]), Event("key", "left")) ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^AssertionError: None != Event(evt='key', data='left', raw=b'') Traceback (most recent call last): File "D:\buildarea\3.14.bolen-windows10\build\Lib\test\test_pyrepl\test_windows_console.py", line 424, in test_KEY_EVENT_up_ignoredself.assertEqual(self.mock.call_count, 1) ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^AssertionError: 0 != 1 Traceback (most recent call last): File "D:\buildarea\3.14.bolen-windows10\build\Lib\test\test_pyrepl\test_windows_console.py", line 568, in test_backspace_vtself.assertEqual(self.get_event([ir], vt_support=True), ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Event("key", "backspace", b"\x7f")) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^AssertionError: None != Event(evt='key', data='backspace', raw=b'\x7f') Traceback (most recent call last): File "D:\buildarea\3.14.bolen-windows10\build\Lib\test\test_pyrepl\test_windows_console.py", line 534, in test_AltGr_7self.assertEqual(self.get_event([ir]), Event("key", "{")) ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^AssertionError: None != Event(evt='key', data='{', raw=b'') Traceback (most recent call last): File "D:\buildarea\3.14.bolen-windows10\build\Lib\test\test_pyrepl\test_windows_console.py", line 510, in test_m_LEFT_ALT_PRESSEDself.assertEqual(self.get_event([ir]), Event(evt="key", data="\033")) ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^AssertionError: None != Event(evt='key', data='\x1b', raw=b'') Traceback (most recent call last): File "D:\buildarea\3.14.bolen-windows10\build\Lib\test\test_pyrepl\test_windows_console.py", line 548, in test_umlaut_a_germanself.assertEqual(self.get_event([ir]), Event("key", "�")) ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^AssertionError: None != Event(evt='key', data='�', raw=b'') Traceback (most recent call last): File "D:\buildarea\3.14.bolen-windows10\build\Lib\test\test_pyrepl\test_windows_console.py", line 415, in test_WINDOW_BUFFER_SIZE_EVENTself.assertEqual(self.get_event([ir]), Event("resize", "")) ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^AssertionError: None != Event(evt='resize', data='', raw=b'') Traceback (most recent call last): File "D:\buildarea\3.14.bolen-windows10\build\Lib\test\test_pyrepl\test_windows_console.py", line 543, in test_AltGr_mself.assertEqual(self.get_event([ir]), Event("key", "�")) ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^AssertionError: None != Event(evt='key', data='�', raw=b'') Traceback (most recent call last): File "D:\buildarea\3.14.bolen-windows10\build\Lib\test\test_pyrepl\test_windows_console.py", line 574, in test_up_vtself.assertEqual(self.get_event(irs, vt_support=True), ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Event(evt='key', data='up', raw=bytearray(b'\x1b[A'))) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^AssertionError: None != Event(evt='key', data='up', raw=bytearray(b'\x1b[A')) Traceback (most recent call last): File "D:\buildarea\3.14.bolen-windows10\build\Lib\test\test_pyrepl\test_windows_console.py", line 434, in test_unhandled_eventsself.assertEqual(self.mock.call_count, 1) ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^AssertionError: 0 != 1 Traceback (most recent call last): File "D:\buildarea\3.14.bolen-windows10\build\Lib\test\test_pyrepl\test_windows_console.py", line 517, in test_m_RIGHT_ALT_PRESSEDself.assertEqual(self.get_event([ir]), Event(evt="key", data="\033")) ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^AssertionError: None != Event(evt='key', data='\x1b', raw=b'') Traceback (most recent call last): File "D:\buildarea\3.14.bolen-windows10\build\Lib\test\test_pyrepl\test_windows_console.py", line 490, in test_left_LEFT_ALT_PRESSEDself.assertEqual(self.get_event([ir]), Event(evt="key", data="\033")) ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^AssertionError: None != Event(evt='key', data='\x1b', raw=b'') Traceback (most recent call last): File "D:\buildarea\3.14.bolen-windows10\build\Lib\test\test_pyrepl\test_windows_console.py", line 505, in test_m_LEFT_ALT_PRESSED_and_LEFT_CTRL_PRESSEDself.assertEqual(self.mock.call_count, 1) ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^AssertionError: 0 != 1 Traceback (most recent call last): File "D:\buildarea\3.14.bolen-windows10\build\Lib\test\test_pyrepl\test_windows_console.py", line 443, in test_backspaceself.assertEqual( ~~~~~~~~~~~~~~~~^self.get_event([ir]), Event("key", "backspace")) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^AssertionError: None != Event(evt='key', data='backspace', raw=b'') Traceback (most recent call last): File "D:\buildarea\3.14.bolen-windows10\build\Lib\test\test_pyrepl\test_windows_console.py", line 562, in test_enter_vtself.assertEqual(self.get_event([ir], vt_support=True), ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Event("key", "\n")) ^^^^^^^^^^^^^^^^^^^AssertionError: None != Event(evt='key', data='\n', raw=b'')

chris-eibl · 2025-05-25T15:58:34Z

I can reproduce these failures on Windows 10, too, for both main and 3.14 when running in a legacy console. In fact, the fix is simple:

--- a/Lib/test/test_pyrepl/test_windows_console.py +++ b/Lib/test/test_pyrepl/test_windows_console.py @@ -386,6 +386,7 @@ def get_event(self, input_records, **kwargs) -> Console: self.console._read_input = self.mock self.console._WindowsConsole__vt_support = kwargs.get("vt_support", False) + self.console.wait = MagicMock(return_value=True)

in these tests I've introduced in #132439. They shouldn't depend on wait, which was moved in this PR into get_event. Particulary WaitForSingleObject(InHandle) is no good here, and I just wonder now, why they pass when the tests are run in a Windows terminal.

I must have run the tests back then in a Windows terminal, and most probably all the GH bots are Windows 11 based, too?

chris-eibl · 2025-05-25T15:59:14Z

Do you want me to create a PR or is the above diff sufficient?

pythongh-134660) (cherry picked from commit 1000283) Co-authored-by: Chris Eibl <138194463+chris-eibl@users.noreply.github.com>

…-134660) (gh-134666) (cherry picked from commit 1000283) Co-authored-by: Chris Eibl <138194463+chris-eibl@users.noreply.github.com>

chris-eibl added 3 commits May 9, 2025 08:34

initial minimal needed changes
e352f79

move blocking / waiting to get_event like in unix_console.py
4dc4238

remove console.wait(100) in perform_bracketed_paste
7f02127

chris-eibl requested review from ambv, lysnikolaou and pablogsal as code owners May 9, 2025 06:57

bedevere-appbot mentioned this pull request May 9, 2025
Pasting in new REPL is slow #130328
Closed

bedevere-appbot added the awaiting review label May 9, 2025

chris-eibl commented May 9, 2025
View reviewed changes

blurb it
7c06902

ambv approved these changes May 25, 2025
View reviewed changes

bedevere-appbot added awaiting merge and removed awaiting review labels May 25, 2025

ambv added the needs backport to 3.14 bugs and security fixes label May 25, 2025

ambv merged commit 91b4886 into python:mainMay 25, 2025
53 checks passed

bedevere-appbot removed the awaiting merge label May 25, 2025

bedevere-appbot removed the needs backport to 3.14 bugs and security fixes label May 25, 2025

ambv pushed a commit that referenced this pull request May 25, 2025
[3.14]GH-130328: Speedup pasting in legacy console on Windows (gh-13…
93aee56
…3728) (#134653) (cherry picked from commit 91b4886) Co-authored-by: Chris Eibl <138194463+chris-eibl@users.noreply.github.com>

chris-eibl mentioned this pull request May 25, 2025
GH-130328: Fix WindowsConsoleGetEventTests after gh-133728 #134660
Merged

ambv pushed a commit that referenced this pull request May 25, 2025
GH-130328: Fix WindowsConsoleGetEventTests aftergh-133728(gh-134660)
1000283

ambv pushed a commit that referenced this pull request May 25, 2025
[3.14]GH-130328: Fix WindowsConsoleGetEventTests aftergh-133728(gh…
6917fbf
…-134660) (gh-134666) (cherry picked from commit 1000283) Co-authored-by: Chris Eibl <138194463+chris-eibl@users.noreply.github.com>

chris-eibl deleted the speedup_paste_legacy branch May 26, 2025 05:55

Pranjal095 pushed a commit to Pranjal095/cpython that referenced this pull request Jul 12, 2025
pythonGH-130328: Speedup pasting in legacy console on Windows (python…
a295503
…gh-133728)

Pranjal095 pushed a commit to Pranjal095/cpython that referenced this pull request Jul 12, 2025
pythonGH-130328: Fix WindowsConsoleGetEventTests afterpythongh-133728(
c248bcb
pythongh-134660)

taegyunkim pushed a commit to taegyunkim/cpython that referenced this pull request Aug 4, 2025
pythonGH-130328: Speedup pasting in legacy console on Windows (python…
7d95b30
…gh-133728)

taegyunkim pushed a commit to taegyunkim/cpython that referenced this pull request Aug 4, 2025
pythonGH-130328: Fix WindowsConsoleGetEventTests afterpythongh-133728(
3a120b1
pythongh-134660)

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

GH-130328: Speedup pasting in legacy console on Windows#133728

GH-130328: Speedup pasting in legacy console on Windows #133728

Uh oh!

chris-eibl commented May 9, 2025•
edited by bedevere-app bot
Loading

Uh oh!

chris-eiblMay 9, 2025

Uh oh!

chris-eiblMay 9, 2025

Uh oh!

chris-eiblMay 9, 2025

Uh oh!

ambv left a comment

Uh oh!

Uh oh!

miss-islington-appbot commented May 25, 2025

Uh oh!

bedevere-appbot commented May 25, 2025

Uh oh!

bedevere-bot commented May 25, 2025

Uh oh!

ambv commented May 25, 2025

Uh oh!

bedevere-bot commented May 25, 2025

Uh oh!

chris-eibl commented May 25, 2025•
edited
Loading

Uh oh!

chris-eibl commented May 25, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Uh oh!

GH-130328: Speedup pasting in legacy console on Windows#133728

GH-130328: Speedup pasting in legacy console on Windows #133728

Uh oh!

Conversation

chris-eibl commented May 9, 2025• edited by bedevere-app botLoading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

chris-eiblMay 9, 2025

Choose a reason for hiding this comment

Uh oh!

chris-eiblMay 9, 2025

Choose a reason for hiding this comment

Uh oh!

chris-eiblMay 9, 2025

Choose a reason for hiding this comment

Uh oh!

ambv left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

miss-islington-appbot commented May 25, 2025

Uh oh!

bedevere-appbot commented May 25, 2025

Uh oh!

bedevere-bot commented May 25, 2025

⚠️⚠️⚠️ Buildbot failure ⚠️⚠️⚠️

Uh oh!

ambv commented May 25, 2025

Uh oh!

bedevere-bot commented May 25, 2025

⚠️⚠️⚠️ Buildbot failure ⚠️⚠️⚠️

Uh oh!

chris-eibl commented May 25, 2025• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

chris-eibl commented May 25, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

chris-eibl commented May 9, 2025•
edited by bedevere-app bot
Loading

chris-eibl commented May 25, 2025•
edited
Loading