Skip to content

sync: fix documentation for new_spin_lock function#24795

Merged
spytheman merged 2 commits into
vlang:masterfrom
lcheylus:docs-spinlock
Jun 26, 2025
Merged

sync: fix documentation for new_spin_lock function#24795
spytheman merged 2 commits into
vlang:masterfrom
lcheylus:docs-spinlock

Conversation

@lcheylus

Copy link
Copy Markdown
Contributor

Add final dot in documentation for new_spin_lock function.

Reviewed in PR #24788 but not fixed before merge :(

Signed-off-by: Laurent Cheylus <foxy@free.fr>
@huly-for-github

Copy link
Copy Markdown

Connected to Huly®: V_0.6-23209

@spytheman

Copy link
Copy Markdown
Contributor

Reviewed in PR #24788 but not fixed before merge :(

I did not see a review/remark in that PR.
Can you please link to the comment?

Comment thread vlib/sync/sync.c.v Outdated
@lcheylus

Copy link
Copy Markdown
Contributor Author

I did not see a review/remark in that PR. Can you please link to the comment?

My review to fix comment in new_spin_lock function => https://github.com/vlang/v/pull/24788/files/3c98798bdb591b536883683e69bdaf39d5e6e10c#diff-60365d617031f02fcf17685b4fce03d9e7c9f094f9aba23d049d83e8598e5eb2

Signed-off-by: Laurent Cheylus <foxy@free.fr>
@spytheman

Copy link
Copy Markdown
Contributor

I still do not see it 🤔.
This is what is shown on my end:
image

@spytheman spytheman merged commit 3d320af into vlang:master Jun 26, 2025
75 checks passed
@lcheylus

Copy link
Copy Markdown
Contributor Author

I still do not see it 🤔. This is what is shown on my end...

No worries, it's maybe a mistake on my part: my comment is still in "Pending..." status.

@lcheylus lcheylus deleted the docs-spinlock branch June 27, 2025 06:35
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.

2 participants