2

Reword `arc_with_non_send_sync` note and help messages by Alexendoo · Pull Reque...

 3 weeks ago
source link: https://github.com/rust-lang/rust-clippy/pull/12609
Go to the source link to view the article. You can view the picture content, updated content and better typesetting reading experience. If the link is broken, please click the button below to view the snapshot at that time.

Conversation

Member

Addresses #12608 (comment)

Makes the note more concise and reframes the Rc suggestion around whether it crosses threads currently due to a manual Send/Sync impl or may do in the future

changelog: none

Collaborator

r? @Jarcho

rustbot has assigned @Jarcho.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

rustbot

added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label

Apr 1, 2024

Contributor

@bors r+

Collaborator

📌 Commit 9d4a368 has been approved by Jarcho

It is now in the queue for this repository.

Collaborator

⌛ Testing commit 9d4a368 with merge a73e751...

bors

merged commit a73e751 into

rust-lang:master

Apr 4, 2024

5 checks passed

Alexendoo

deleted the arc-with-non-send-sync-message branch

April 5, 2024 16:02

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

Jarcho

Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties
Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

None yet

4 participants

report

Recommend

About Joyk


Aggregate valuable and interesting links.
Joyk means Joy of geeK