Skip to content

slightly tweak description of b_list in comments - #28925

Merged
serhiy-storchaka merged 2 commits into
python:mainfrom
sl4mmy:patch-1
Aug 12, 2026
Merged

slightly tweak description of b_list in comments#28925
serhiy-storchaka merged 2 commits into
python:mainfrom
sl4mmy:patch-1

Conversation

@sl4mmy

@sl4mmy sl4mmy commented Oct 13, 2021

Copy link
Copy Markdown
Contributor

I noticed a small typo in the comment describing the use of b_list in
struct basicblock_ ("... that the block are allocated"). Reviewing the
other comments describing use of b_list I then noticed a slight
inconsistency in the description of what it actually points to ("next,"
"early") which I also decided to address here.

I noticed a small typo in the comment describing the use of b_list in
struct basicblock_ ("... that the block are allocated").  Reviewing the
other comments describing use of b_list I then noticed a slight
inconsistency in the description of what it actually points to ("next,"
"early") which I also decided to address here.
@the-knights-who-say-ni

Copy link
Copy Markdown

Hello, and thanks for your contribution!

I'm a bot set up to make sure that the project can legally accept this contribution by verifying everyone involved has signed the PSF contributor agreement (CLA).

Recognized GitHub username

We couldn't find a bugs.python.org (b.p.o) account corresponding to the following GitHub usernames:

@sl4mmy

This might be simply due to a missing "GitHub Name" entry in one's b.p.o account settings. This is necessary for legal reasons before we can look at this contribution. Please follow the steps outlined in the CPython devguide to rectify this issue.

You can check yourself to see if the CLA has been received.

Thanks again for the contribution, we look forward to reviewing it!

@MaxwellDupre MaxwellDupre left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Better.

@github-actions

github-actions Bot commented Apr 9, 2026

Copy link
Copy Markdown

This PR is stale because it has been open for 30 days with no activity.

@github-actions github-actions Bot added the stale Stale PR or inactive for long period of time. label Apr 9, 2026
@sl4mmy

sl4mmy commented Apr 9, 2026

Copy link
Copy Markdown
Contributor Author

I signed the CLA, is there anything else I need to do to get this merged?

@github-actions github-actions Bot removed the stale Stale PR or inactive for long period of time. label May 11, 2026
@bedevere-app

bedevere-app Bot commented Aug 11, 2026

Copy link
Copy Markdown

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

@serhiy-storchaka serhiy-storchaka 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.

LGTM. 👍

@serhiy-storchaka
serhiy-storchaka merged commit 3f99ebe into python:main Aug 12, 2026
59 checks passed
mbeijen pushed a commit to mbeijen/cpython that referenced this pull request Aug 14, 2026
b_list points to the previously allocated block, not to the next one.
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.

6 participants