Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix misleading mentions of tp_size in comments #9093

Merged
merged 1 commit into from
Sep 10, 2018
Merged

Fix misleading mentions of tp_size in comments #9093

merged 1 commit into from
Sep 10, 2018

Conversation

petere
Copy link
Contributor

@petere petere commented Sep 7, 2018

Many type object initializations labeled a field tp_size in the comment, but the name of that field is tp_basicsize.

@petere
Fix misleading mentions of tp_size in comments
c904433 
Many type object initializations labeled a field "tp_size" in the
comment, but the name of that field is tp_basicsize.
@petere petere requested a review from rhettinger as a code owner September 7, 2018 09:05
@benjaminp benjaminp merged commit 0e0bc4e into python:master Sep 10, 2018
@miss-islington
Copy link
Contributor

Thanks @petere for the PR, and @benjaminp for merging it 🌮🎉.. I'm working now to backport this PR to: 2.7, 3.6, 3.7.
🐍🍒⛏🤖

@miss-islington
Copy link
Contributor

Sorry, @petere and @benjaminp, I could not cleanly backport this to 3.7 due to a conflict.
Please backport using cherry_picker on command line.
cherry_picker 0e0bc4e221f592f305d335faf5f8046484eb9238 3.7

@miss-islington
Copy link
Contributor

Sorry, @petere and @benjaminp, I could not cleanly backport this to 3.6 due to a conflict.
Please backport using cherry_picker on command line.
cherry_picker 0e0bc4e221f592f305d335faf5f8046484eb9238 3.6

@miss-islington
Copy link
Contributor

Sorry, @petere and @benjaminp, I could not cleanly backport this to 2.7 due to a conflict.
Please backport using cherry_picker on command line.
cherry_picker 0e0bc4e221f592f305d335faf5f8046484eb9238 2.7

benjaminp pushed a commit that referenced this pull request Sep 10, 2018
@petere @benjaminp
[3.7] Fix misleading mentions of tp_size in comments (GH-9093)
b6a6008 
Many type object initializations labeled a field "tp_size" in the
comment, but the name of that field is tp_basicsize..
(cherry picked from commit 0e0bc4e)

Co-authored-by: Peter Eisentraut <peter@eisentraut.org>
@bedevere-bot
Copy link

GH-9136 is a backport of this pull request to the 3.7 branch.

benjaminp pushed a commit that referenced this pull request Sep 10, 2018
@petere @benjaminp
[3.6] Fix misleading mentions of tp_size in comments (GH-9093)
444b1cd 
Many type object initializations labeled a field "tp_size" in the
comment, but the name of that field is tp_basicsize..
(cherry picked from commit 0e0bc4e)

Co-authored-by: Peter Eisentraut <peter@eisentraut.org>
@bedevere-bot
Copy link

GH-9137 is a backport of this pull request to the 3.6 branch.

benjaminp pushed a commit that referenced this pull request Sep 10, 2018
@petere @benjaminp
[2.7] Fix misleading mentions of tp_size in comments (GH-9093)
70013bd 
Many type object initializations labeled a field "tp_size" in the
comment, but the name of that field is tp_basicsize..
(cherry picked from commit 0e0bc4e)

Co-authored-by: Peter Eisentraut <peter@eisentraut.org>
@bedevere-bot
Copy link

GH-9138 is a backport of this pull request to the 2.7 branch.

@jdemeyer jdemeyer mentioned this pull request May 28, 2019
Merged
@petere petere deleted the fix-tp_size branch February 12, 2020 10:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@auvipy auvipy auvipy approved these changes

@rhettinger rhettinger Awaiting requested review from rhettinger

Assignees

@benjaminp benjaminp

Labels
skip issue skip news
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@petere @miss-islington @bedevere-bot @auvipy @benjaminp @the-knights-who-say-ni