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

gh-102941: Fix "‘subobj’ may be used uninitialized in this function" warning in `bytes_methods.c` by sobolevn · Pull Request #102942 · python/cpython · GitHub

/ cpython Public

gh-102941: Fix "‘subobj’ may be used uninitialized in this function" warning in bytes_methods.c - #102942

Merged
JelleZijlstra merged 1 commit into
python:mainfrom
sobolevn:issue-102941
Mar 27, 2023
Merged

gh-102941: Fix "‘subobj’ may be used uninitialized in this function" warning in bytes_methods.c#102942
JelleZijlstra merged 1 commit into
python:mainfrom
sobolevn:issue-102941

Conversation

sobolevn commented Mar 23, 2023
edited by bedevere-bot
Loading

Copy link
Copy Markdown
Member

I still think that this warning is a false positive and there's no way to actually abuse it.
So, I hope that = NULL will just make it go away.

sobolevn commented Mar 23, 2023
edited
Loading

Copy link
Copy Markdown
Member Author

The warning is gone 🎉
The second one is solved here: #102940

@kumaraditya303 can you please take a look? :)

sobolevn requested a review from kumaraditya303 March 23, 2023 10:05

Copy link
Copy Markdown
Member Author

@erlend-aasland can you please take a look? :)
All PRs get this warning, it is kinda annoying and might scare people.

Copy link
Copy Markdown
Contributor

I'm travelling currently, please find another reviewer or wait till I'm back.

JelleZijlstra merged commit 2cdc518 into python:main Mar 27, 2023
Fidget-Spinner pushed a commit to Fidget-Spinner/cpython that referenced this pull request Mar 27, 2023
warsaw pushed a commit to warsaw/cpython that referenced this pull request Apr 11, 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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants


Back | FazBrowse Home | New Git URL