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

Nightly clippy fixes. #340

Merged
merged 2 commits into from
Aug 2, 2023
Merged

Nightly clippy fixes. #340

merged 2 commits into from
Aug 2, 2023

Conversation

cpu
Copy link
Member

@cpu cpu commented Aug 2, 2023

Description

Small branch fixing some failures caught by the cron CI job's nightly clippy task.

lib: fix small rustdoc mistake

New nightly clippy is flagging one of the lines of rustdoc comment in lib.rs:

error: this item has comments with 4 forward slashes (////). These look like doc comments, but they aren't

The fix is simple, use only 3 forward slashes :-)

cipher: remove redundant call.

Nightly clippy is flagging a redundant call in cipher.rs:

call to .deref() on a reference in this situation does nothing

This commit removes the call to deref and also the now unnecessary import of the std::ops::Deref trait.

cpu added 2 commits August 2, 2023 08:32
New nightly clippy is flagging one of the lines of rustdoc comment in
`lib.rs`:

> error: this item has comments with 4 forward slashes (`////`). These
> look like doc comments, but they aren't

The fix is simple, use only 3 forward slashes :-)
Nightly clippy is flagging a redundant call in `cipher.rs`:

> call to `.deref()` on a reference in this situation does nothing

This commit removes the call to `deref` and also the now unnecessary
import of the `std::ops::Deref` trait.
@cpu cpu self-assigned this Aug 2, 2023
@cpu cpu requested a review from ctz August 2, 2023 12:38
@cpu cpu merged commit 9008488 into rustls:main Aug 2, 2023
@cpu cpu deleted the cpu-fix-nightly-clippy branch August 2, 2023 13: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