Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Fixed clippy warning #382

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

Merged
merged 1 commit into from
Apr 30, 2024
Merged

Fixed clippy warning #382

merged 1 commit into from
Apr 30, 2024

Conversation

palant
Copy link
Contributor

@palant palant commented Apr 29, 2024

Description

This fixes a clippy warning caused by test code added in #376:

error: importing a legacy numeric constant
   --> src/directory_listing.rs:582:9
    |
582 |     use std::u64::MAX;
    |         ^^^^^^^^^^^^^
    |
    = help: remove this import and use the associated constant `u64::MAX` from the primitive type instead
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#legacy_numeric_constants
note: the lint level is defined here
   --> src/lib.rs:99:9
    |
99  | #![deny(warnings)]
    |         ^^^^^^^^
    = note: `#[deny(clippy::legacy_numeric_constants)]` implied by `#[deny(warnings)]`

error: usage of a legacy numeric constant
   --> src/directory_listing.rs:624:20
    |
624 |         let size = MAX;
    |                    ^^^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#legacy_numeric_constants
help: use the associated constant instead
    |
624 |         let size = u64::MAX;
    |                    ~~~~~~~~

Copy link

semanticdiff-com bot commented Apr 29, 2024

Review changes with SemanticDiff.

Analyzed 1 of 1 files.

Filename Status
✔️ src/directory_listing.rs Analyzed

@joseluisq joseluisq added v2 v2 release bugfix This is PR fixes a bug codebase Related to the project's source code labels Apr 30, 2024
@joseluisq joseluisq merged commit 1b13a74 into static-web-server:master Apr 30, 2024
37 checks passed
@joseluisq joseluisq added this to the v2.31.0 milestone Apr 30, 2024
@palant palant deleted the clippy-fix branch April 30, 2024 07:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix This is PR fixes a bug codebase Related to the project's source code v2 v2 release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants