FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

[3.11] gh-109538: Catch closed loop runtime error and issue warning (GH-111983) by dpr-0 · Pull Request #112141 · python/cpython · GitHub

Repository navigation

[3.11] gh-109538: Catch closed loop runtime error and issue warning (GH-111983) - #112141

Merged
gvanrossum merged 2 commits into
python:3.11from
dpr-0:backport-e0f5127-3.11
Nov 20, 2023
Merged

gvanrossum merged 2 commits into
python:3.11from
dpr-0:backport-e0f5127-3.11

Conversation

dpr-0 commented Nov 16, 2023 •
edited
Loading

Copy link
Copy Markdown
Contributor

Issue a ResourceWarning instead.

(cherry picked from commit e0f5127) gh-109538: Avoid RuntimeError when StreamWriter is deleted with closed loop (#111983)

Issue a ResourceWarning instead.

Co-authored-by: Hugo van Kemenade hugovk@users.noreply.github.com
(cherry picked from commit e0f5127)

… closed loop (python#111983)

Issue a ResourceWarning instead.

Co-authored-by: Hugo van Kemenade <hugovk@users.noreply.github.com>
(cherry picked from commit e0f5127)
dpr-0 requested review from 1st1 and asvetlov as code owners November 16, 2023 07:11
bedevere-app Bot added the type-bug An unexpected behavior, bug, or error label Nov 16, 2023
dpr-0 changed the title [3.11] gh-109538: Catch closed loop runtime error and issue warning (GH-111983) [3.12] gh-109538: Catch closed loop runtime error and issue warning (GH-111983) Nov 16, 2023
dpr-0 changed the title [3.12] gh-109538: Catch closed loop runtime error and issue warning (GH-111983) [3.11] gh-109538: Catch closed loop runtime error and issue warning (GH-111983) Nov 16, 2023

gvanrossum left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Yeah, looks good. I guess this backported a test for this code.

gvanrossum left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Okay let’s merge this!

gvanrossum merged commit 1397505 into python:3.11 Nov 20, 2023
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type-bug An unexpected behavior, bug, or error

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL