◐ Shell
reader mode source ↗
Skip to content

bpo-45711: Remove type and traceback from exc_info#30122

Merged
markshannon merged 16 commits into
python:mainfrom
iritkatriel:45711-exc_info_just_value
Dec 17, 2021
Merged

bpo-45711: Remove type and traceback from exc_info#30122
markshannon merged 16 commits into
python:mainfrom
iritkatriel:45711-exc_info_just_value

Conversation

@iritkatriel

@iritkatriel iritkatriel commented Dec 15, 2021

Copy link
Copy Markdown
Member

@iritkatriel iritkatriel added the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Dec 15, 2021
@bedevere-bot

Copy link
Copy Markdown

🤖 New build scheduled with the buildbot fleet by @iritkatriel for commit 9b04160 🤖

If you want to schedule another build, you need to add the ":hammer: test-with-buildbots" label again.

@bedevere-bot bedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Dec 15, 2021
@markshannon

Copy link
Copy Markdown
Member

This is great. 🍾

Removes code, should make any exception handling code faster and reduces stack consumption; the perfect combination.
A couple of very minor quibbles, otherwise looks really good.

Co-authored-by: Oleg Iarygin <dralife@yandex.ru>
@iritkatriel

Copy link
Copy Markdown
Member Author

The windows test failure is unrelated

File "D:\a\cpython\cpython\Lib\test\support_init_.py", line 197, in _force_run
1232
return func(*args)
1233
^^^^^^^^^^^
1234
PermissionError: [WinError 32] The process cannot access the file because it is being used by another process: 'D:\a\cpython\cpython\build\test_python_4272�\test_python_worker_516�'

@markshannon

Copy link
Copy Markdown
Member

Excellent.

@markshannon markshannon merged commit 396b583 into python:main Dec 17, 2021
@iritkatriel iritkatriel deleted the 45711-exc_info_just_value branch January 13, 2022 15:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants