Skip to content

Annotate Token as must_use#4

Merged
jtracey merged 1 commit intothunderbird:mainfrom
jtracey:annotations
Apr 2, 2026
Merged

Annotate Token as must_use#4
jtracey merged 1 commit intothunderbird:mainfrom
jtracey:annotations

Conversation

@jtracey
Copy link
Copy Markdown
Contributor

@jtracey jtracey commented Apr 2, 2026

I looked at some of the other annotations used by MutexGuard, but they don't seem to apply here.

Based on #1 for tests and CI.

@jtracey jtracey requested a review from a team as a code owner April 2, 2026 18:16
Copy link
Copy Markdown
Member

@babolivier babolivier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, good catch! I've only got a small question; considering it's not a huge deal and my upcoming long weekend I didn't want to make it blocking.

@jtracey jtracey merged commit 6938372 into thunderbird:main Apr 2, 2026
1 check passed
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