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

docs(skiplist): update insert function document #1124

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Tianion
Copy link
Contributor

@Tianion Tianion commented Jul 13, 2024

"it will be removed before inserting the new one", it looks like there are two steps involved in the process described.
#1122

@taiki-e
Copy link
Member

taiki-e commented Dec 8, 2024

Thanks. It seems to me that “using CAS” is a description of implementation details and does not explain what specific behavior is provided. (I guess you want to document the semantics that no intermediate state is observed during the update.)

@taiki-e taiki-e added crossbeam-skiplist docs S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author labels Dec 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
crossbeam-skiplist docs S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author
Development

Successfully merging this pull request may close these issues.

2 participants