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

gh-121040: Remove fallthrough warnings compiler option by nohlson · Pull Request #121041 · python/cpython · GitHub

/ cpython Public

gh-121040: Remove fallthrough warnings compiler option - #121041

Merged
corona10 merged 1 commit into
python:mainfrom
nohlson:revert-fallthrough-warnings
Jun 26, 2024
Merged

gh-121040: Remove fallthrough warnings compiler option#121041
corona10 merged 1 commit into
python:mainfrom
nohlson:revert-fallthrough-warnings

Conversation

nohlson commented Jun 26, 2024
edited by bedevere-app Bot
Loading

Copy link
Copy Markdown
Contributor

Until tooling is implemented remove -Wimplicit-fallthrough compiler flag from BASEFLAGS

corona10 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

lgtm!

corona10 changed the title gh-#121040: Remove fallthrough warnings gh-121040: Remove fallthrough warnings Jun 26, 2024
corona10 enabled auto-merge (squash) June 26, 2024 09:45
corona10 disabled auto-merge June 26, 2024 09:45
corona10 enabled auto-merge (squash) June 26, 2024 09:45
corona10 changed the title gh-121040: Remove fallthrough warnings gh-121040: Remove fallthrough warnings compiler option Jun 26, 2024
corona10 merged commit ef28f6d into python:main Jun 26, 2024
mrahtz pushed a commit to mrahtz/cpython that referenced this pull request Jun 30, 2024
noahbkim pushed a commit to hudson-trading/cpython that referenced this pull request Jul 11, 2024
estyxx pushed a commit to estyxx/cpython that referenced this pull request Jul 17, 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