Skip to content

bpo-29102: Add a unique ID to PyInterpreterState.#1639

Merged
ericsnowcurrently merged 17 commits into
python:masterfrom
ericsnowcurrently:interpreter-id
May 23, 2017
Merged

bpo-29102: Add a unique ID to PyInterpreterState.#1639
ericsnowcurrently merged 17 commits into
python:masterfrom
ericsnowcurrently:interpreter-id

Conversation

@ericsnowcurrently
Copy link
Copy Markdown
Member

@mention-bot
Copy link
Copy Markdown

@ericsnowcurrently, thanks for your PR! By analyzing the history of the files in this pull request, we identified @zooba, @birkenfeld and @ncoghlan to be potential reviewers.

Comment thread Doc/c-api/init.rst Outdated
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.

Python 3.6 now uses stdint.h: please use directly uint64_t.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Why unsigned? Did you mean int64_t?

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.

Oh right, int64_t. By the way, extract of master:

#define PY_INT64_T int64_t

@ericsnowcurrently ericsnowcurrently force-pushed the interpreter-id branch 5 times, most recently from 4bcf3f2 to da8439a Compare May 21, 2017 06:04
@ericsnowcurrently ericsnowcurrently merged commit e377416 into python:master May 23, 2017
@ericsnowcurrently ericsnowcurrently deleted the interpreter-id branch May 23, 2017 04:07
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