Skip to content

Conversation

@ericsnowcurrently
Copy link
Member

@ericsnowcurrentlyericsnowcurrently commented Jan 25, 2023

A PyThreadState can be in one of many states in its lifecycle, represented by some status value. Those statuses haven't been particularly clear, so we're addressing that here. Specifically:

  • made the distinct lifecycle statuses clear on PyThreadState
  • identified expectations of how various lifecycle-related functions relate to status
  • noted the various places where those expectations don't match the actual behavior

At some point we'll need to address the mismatches.

(This change also includes some cleanup.)

@ericsnowcurrentlyericsnowcurrently marked this pull request as ready for review January 30, 2023 18:56
@ericsnowcurrentlyericsnowcurrently merged commit e11fc03 into python:mainJan 30, 2023
@ericsnowcurrentlyericsnowcurrently deleted the pystate-lifecycle-cleanup branch January 30, 2023 19:07
mdboom pushed a commit to mdboom/cpython that referenced this pull request Jan 31, 2023
…01308) A PyThreadState can be in one of many states in its lifecycle, represented by some status value. Those statuses haven't been particularly clear, so we're addressing that here. Specifically: * made the distinct lifecycle statuses clear on PyThreadState * identified expectations of how various lifecycle-related functions relate to status * noted the various places where those expectations don't match the actual behavior At some point we'll need to address the mismatches. (This change also includes some cleanup.) python#59956
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@ericsnowcurrently@bedevere-bot