Skip to content

bpo-30604: clean up co_extra support#2144

Merged
1st1 merged 4 commits intopython:masterfrom
DinoV:30636
Jun 21, 2017
Merged

bpo-30604: clean up co_extra support#2144
1st1 merged 4 commits intopython:masterfrom
DinoV:30636

Conversation

@DinoV
Copy link
Contributor

@DinoV DinoV commented Jun 13, 2017

bpo-30604: port fix from 3.6 dropping binary compatibility tweaks

@mention-bot
Copy link

@DinoV, thanks for your PR! By analyzing the history of the files in this pull request, we identified @mhammond, @benjaminp and @serhiy-storchaka to be potential reviewers.

@brettcannon brettcannon changed the title bpo-30604: port fix from 3.6 dropping binary compatibility tweaks bpo-30604: clean up co_extra support Jun 13, 2017
Copy link
Member

@brettcannon brettcannon left a comment

Choose a reason for hiding this comment

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

I updated the NEWS entry, but otherwise LGTM.

{
if (co->co_extra != NULL) {
PyThreadState *tstate = PyThreadState_Get();
PyInterpreterState *interp= PyThreadState_Get()->interp;
Copy link
Member

Choose a reason for hiding this comment

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

nit: space before =

Copy link
Member

Choose a reason for hiding this comment

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

Never mind, I figured I can fix this myself :)

@brettcannon brettcannon reopened this Jun 21, 2017
@1st1 1st1 merged commit f3cffd2 into python:master Jun 21, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants