Skip to content

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