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

gh-118849: Fix "code will never be executed" warning in `dictobject.c` by sobolevn · Pull Request #118850 · python/cpython · GitHub

/ cpython Public

gh-118849: Fix "code will never be executed" warning in dictobject.c - #118850

Merged
sobolevn merged 2 commits into
python:mainfrom
sobolevn:issue-118849
May 9, 2024
Merged

gh-118849: Fix "code will never be executed" warning in dictobject.c#118850
sobolevn merged 2 commits into
python:mainfrom
sobolevn:issue-118849

Conversation

sobolevn commented May 9, 2024
edited by bedevere-app Bot
Loading

Copy link
Copy Markdown
Member

Comment thread Objects/dictobject.c Outdated
sobolevn merged commit 82abe75 into python:main May 9, 2024

Copy link
Copy Markdown

Thanks @sobolevn for the PR 🌮🎉.. I'm working now to backport this PR to: 3.13.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request May 9, 2024
…ject.c` (pythonGH-118850)

(cherry picked from commit 82abe75)

Co-authored-by: Nikita Sobolev <mail@sobolevn.me>

bedevere-app Bot commented May 9, 2024

Copy link
Copy Markdown

GH-118859 is a backport of this pull request to the 3.13 branch.

bedevere-app Bot removed the needs backport to 3.13 bugs and security fixes label May 9, 2024

sobolevn commented May 9, 2024

Copy link
Copy Markdown
Member Author

Thanks for the good catch and for the review! :)

sobolevn added a commit that referenced this pull request May 9, 2024
…bject.c` (GH-118850) (#118859)

gh-118849: Fix "code will never be executed" warning in `dictobject.c` (GH-118850)
(cherry picked from commit 82abe75)

Co-authored-by: Nikita Sobolev <mail@sobolevn.me>
SonicField pushed a commit to SonicField/cpython that referenced this pull request May 10, 2024
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL