Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Clarify threading documentation: atomic list.append #481

Open
mdickinson opened this issue Aug 9, 2021 · 0 comments
Open

Clarify threading documentation: atomic list.append #481

mdickinson opened this issue Aug 9, 2021 · 0 comments
Labels
component: documentation Issues related to developer and user documentation type: bug Something isn't working
Milestone

Comments

@mdickinson
Copy link
Member

Some confusion was reported with regard to this sentence in the threading docs:

For example, list.append may happen to be atomic in current versions of CPython, but there’s no guarantee that that will remain the case, and you may find that your code is actually working with a subclass of list (like TraitList) for which append is not thread-safe.

The sentence could be broken up, and it should be rephrased to make clear that "working" does not mean "functioning correctly", but rather something along the lines of "dealing with".

@mdickinson mdickinson added type: bug Something isn't working component: documentation Issues related to developer and user documentation labels Aug 9, 2021
@mdickinson mdickinson added this to the Release 0.4.0 milestone Sep 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: documentation Issues related to developer and user documentation type: bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant