Skip to content

bpo-32980 Remove dead code in _Py_InitializeCore()#5953

Closed
ApproximateIdentity wants to merge 1 commit intopython:masterfrom
ApproximateIdentity:fix-issue-32980
Closed

bpo-32980 Remove dead code in _Py_InitializeCore()#5953
ApproximateIdentity wants to merge 1 commit intopython:masterfrom
ApproximateIdentity:fix-issue-32980

Conversation

@ApproximateIdentity
Copy link
Copy Markdown
Contributor

@ApproximateIdentity ApproximateIdentity commented Mar 1, 2018

@benjaminp
Copy link
Copy Markdown
Contributor

Thanks for the patch. We should delete _PyFrame_Init if it's empty.

@ncoghlan
Copy link
Copy Markdown
Contributor

ncoghlan commented Mar 2, 2018

@benjaminp For now, I closed the entire issue based on avoidance of churn in the exported C API (https://bugs.python.org/issue32980#msg313130), but I'd also be happy to amend that to just dropping the nominally private _PyFrame_Init API entirely (including from pylifecycle.h).

@ncoghlan ncoghlan closed this Mar 2, 2018
@ApproximateIdentity ApproximateIdentity deleted the fix-issue-32980 branch March 2, 2018 15:15
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.

5 participants