Title: bpo-45711: Remove unnecessary normalization of exc_info by iritkatriel · Pull Request #29922 · python/cpython · GitHub
Open Graph Title: bpo-45711: Remove unnecessary normalization of exc_info by iritkatriel · Pull Request #29922 · python/cpython
X Title: bpo-45711: Remove unnecessary normalization of exc_info by iritkatriel · Pull Request #29922 · python/cpython
Description: exc_info should always be normalized, so it is not necessary to normalize it here. It is also not necessary to copy the traceback to the exception instance. I added an assertion that the traceback does not change, and it broke only one test: test_exception_chaining_after_await_with_context_cycle This is a test that raises the same exception twice, and makes sure that no context cycle is created. The test is not concerned with the traceback at all. We discussed the issue of the traceback change in this test in PR29780. https://bugs.python.org/issue45711
Open Graph Description: exc_info should always be normalized, so it is not necessary to normalize it here. It is also not necessary to copy the traceback to the exception instance. I added an assertion that the traceback ...
X Description: exc_info should always be normalized, so it is not necessary to normalize it here. It is also not necessary to copy the traceback to the exception instance. I added an assertion that the traceback ...
Opengraph URL: https://github.com/python/cpython/pull/29922
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:26537ecc-740f-d2fe-d425-68dc7c3baddb |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | 83A2:2E8FB4:3BC3ED5:4EED117:6A5D34B1 |
| html-safe-nonce | e0fb5d9c495ae4536a47ec04ece15a23279e9291b002f0bb784b9e6c8a9ab4a1 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiI4M0EyOjJFOEZCNDozQkMzRUQ1OjRFRUQxMTc6NkE1RDM0QjEiLCJ2aXNpdG9yX2lkIjoiMzczNTA0MzAxNTY3OTIyNzA1NyIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9 |
| visitor-hmac | 37cd147f0d0c3ad4e8570facba574f335719828aab93346753f5b4a1fc8de5b2 |
| hovercard-subject-tag | pull_request:795338597 |
| github-keyboard-shortcuts | repository,pull-request-list,pull-request-conversation,pull-request-files-changed,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/python/cpython/pull/29922/files |
| twitter:image | https://avatars.githubusercontent.com/u/1055913?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/1055913?s=400&v=4 |
| og:image:alt | exc_info should always be normalized, so it is not necessary to normalize it here. It is also not necessary to copy the traceback to the exception instance. I added an assertion that the traceback ... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | 5290d7e14309ad1e76106a9c4237bd1041517e83ea182c8ab756752cb0c6940b |
| turbo-cache-control | no-preview |
| diff-view | unified |
| go-import | github.com/python/cpython git https://github.com/python/cpython.git |
| octolytics-dimension-user_id | 1525981 |
| octolytics-dimension-user_login | python |
| octolytics-dimension-repository_id | 81598961 |
| octolytics-dimension-repository_nwo | python/cpython |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 81598961 |
| octolytics-dimension-repository_network_root_nwo | python/cpython |
| turbo-body-classes | logged-out env-production page-responsive full-width |
| disable-turbo | true |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 9c975978430e9ad293956f2bbdaf153b1bd84a99 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width