fix: Correctly replace linux natives for LWJGL 3.3.1 (#20) #101
clippy
8 warnings
Details
Results
Message level | Amount |
---|---|
Internal compiler error | 0 |
Error | 0 |
Warning | 8 |
Note | 0 |
Help | 0 |
Versions
- rustc 1.80.0 (051478957 2024-07-21)
- cargo 1.80.0 (376290515 2024-07-16)
- clippy 0.1.80 (0514789 2024-07-21)
Annotations
Check warning on line 162 in daedalus_client/src/util.rs
github-actions / clippy
the borrowed expression implements the required traits
warning: the borrowed expression implements the required traits
--> daedalus_client/src/util.rs:162:18
|
162 | .get(&url.replace("http://", "https://"))
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: change this to: `url.replace("http://", "https://")`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
= note: `#[warn(clippy::needless_borrows_for_generic_args)]` on by default
Check warning on line 60 in daedalus_client/src/fabric.rs
github-actions / clippy
this function has too many arguments (8/7)
warning: this function has too many arguments (8/7)
--> daedalus_client/src/fabric.rs:50:1
|
50 | #[tracing::instrument(skip(semaphore, upload_files, mirror_artifacts))]
| ^----------------------------------------------------------------------
| |
| _in this procedural macro expansion
| |
51 | | async fn fetch(
52 | | format_version: usize,
53 | | mod_loader: &str,
... |
59 | | mirror_artifacts: &DashMap<String, MirrorArtifact>,
60 | | ) -> Result<(), Error> {
| |______________________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#too_many_arguments
= note: `#[warn(clippy::too_many_arguments)]` on by default
= note: this warning originates in the attribute macro `tracing::instrument` (in Nightly builds, run with -Z macro-backtrace for more info)
Check warning on line 445 in daedalus_client/src/forge.rs
github-actions / clippy
field `path` is never read
warning: field `path` is never read
--> daedalus_client/src/forge.rs:445:25
|
440 | struct ForgeInstallerProfileV2 {
| ----------------------- field in this struct
...
445 | pub path: Option<String>,
| ^^^^
|
= note: `ForgeInstallerProfileV2` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis
= note: `#[warn(dead_code)]` on by default
Check warning on line 163 in daedalus_client/src/main.rs
github-actions / clippy
variable does not need to be mutable
warning: variable does not need to be mutable
--> daedalus_client/src/main.rs:163:9
|
163 | let mut val = mirror_artifacts
| ----^^^
| |
| help: remove this `mut`
|
= note: `#[warn(unused_mut)]` on by default
Check warning on line 2 in daedalus_client/src/forge.rs
github-actions / clippy
unused import: `sha1_async`
warning: unused import: `sha1_async`
--> daedalus_client/src/forge.rs:2:55
|
2 | download_file, fetch_json, fetch_xml, format_url, sha1_async,
| ^^^^^^^^^^
|
= note: `#[warn(unused_imports)]` on by default
Check warning on line 36 in daedalus/src/modded.rs
github-actions / clippy
use of deprecated method `chrono::TimeZone::datetime_from_str`: use `DateTime::parse_from_str` or `NaiveDateTime::parse_from_str` with `and_utc()` or `and_local_timezone()` instead
warning: use of deprecated method `chrono::TimeZone::datetime_from_str`: use `DateTime::parse_from_str` or `NaiveDateTime::parse_from_str` with `and_utc()` or `and_local_timezone()` instead
--> daedalus/src/modded.rs:36:26
|
36 | .or_else(|_| Utc.datetime_from_str(&s, "%Y-%m-%dT%H:%M:%S%.9f"))
| ^^^^^^^^^^^^^^^^^
|
= note: `#[warn(deprecated)]` on by default
Check warning on line 436 in daedalus/src/minecraft.rs
github-actions / clippy
unexpected `cfg` condition value: `bincode`
warning: unexpected `cfg` condition value: `bincode`
--> daedalus/src/minecraft.rs:436:16
|
436 | #[cfg_attr(feature = "bincode", bincode(with_serde))]
| ^^^^^^^^^^^^^^^^^^^ help: remove the condition
|
= note: no expected values for `feature`
= help: consider adding `bincode` as a feature in `Cargo.toml`
= note: see <https://doc.rust-lang.org/nightly/rustc/check-cfg/cargo-specifics.html> for more information about checking conditional configuration
Check warning on line 433 in daedalus/src/minecraft.rs
github-actions / clippy
unexpected `cfg` condition value: `bincode`
warning: unexpected `cfg` condition value: `bincode`
--> daedalus/src/minecraft.rs:433:16
|
433 | #[cfg_attr(feature = "bincode", bincode(with_serde))]
| ^^^^^^^^^^^^^^^^^^^ help: remove the condition
|
= note: no expected values for `feature`
= help: consider adding `bincode` as a feature in `Cargo.toml`
= note: see <https://doc.rust-lang.org/nightly/rustc/check-cfg/cargo-specifics.html> for more information about checking conditional configuration
= note: `#[warn(unexpected_cfgs)]` on by default