cmake & Cargo.lock update #433
Annotations
120 warnings
prettier
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actionsx/prettier@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
prettier
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v2, actionsx/prettier@v2. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
current MSRV (Minimum Supported Rust Version) is `1.58.0` but this item is stable since `1.64.0`:
C:\Users\runneradmin\.cargo\registry\src\index.crates.io-6f17d22bba15001f\tokio-1.41.1\src\macros\select.rs#L515
warning: current MSRV (Minimum Supported Rust Version) is `1.58.0` but this item is stable since `1.64.0`
--> alvr\server\src\lib.rs:394:17
|
394 | / tokio::select! {
395 | | _ = connection::connection_lifecycle_loop() => (),
396 | | _ = SHUTDOWN_NOTIFIER.notified() => (),
397 | | }
| |_________________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#incompatible_msrv
= note: this warning originates in the macro `$crate::select` which comes from the expansion of the macro `tokio::select` (in Nightly builds, run with -Z macro-backtrace for more info)
|
current MSRV (Minimum Supported Rust Version) is `1.58.0` but this item is stable since `1.64.0`:
C:\Users\runneradmin\.cargo\registry\src\index.crates.io-6f17d22bba15001f\tokio-1.41.1\src\macros\select.rs#L505
warning: current MSRV (Minimum Supported Rust Version) is `1.58.0` but this item is stable since `1.64.0`
--> alvr\server\src\lib.rs:394:17
|
394 | / tokio::select! {
395 | | _ = connection::connection_lifecycle_loop() => (),
396 | | _ = SHUTDOWN_NOTIFIER.notified() => (),
397 | | }
| |_________________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#incompatible_msrv
= note: this warning originates in the macro `$crate::select` which comes from the expansion of the macro `tokio::select` (in Nightly builds, run with -Z macro-backtrace for more info)
|
current MSRV (Minimum Supported Rust Version) is `1.58.0` but this item is stable since `1.64.0`:
C:\Users\runneradmin\.cargo\registry\src\index.crates.io-6f17d22bba15001f\tokio-1.41.1\src\macros\select.rs#L515
warning: current MSRV (Minimum Supported Rust Version) is `1.58.0` but this item is stable since `1.64.0`
--> alvr\server\src\lib.rs:241:13
|
241 | / tokio::select! {
242 | | _ = web_server => (),
243 | | _ = SHUTDOWN_NOTIFIER.notified() => (),
244 | | }
| |_____________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#incompatible_msrv
= note: this warning originates in the macro `$crate::select` which comes from the expansion of the macro `tokio::select` (in Nightly builds, run with -Z macro-backtrace for more info)
|
current MSRV (Minimum Supported Rust Version) is `1.58.0` but this item is stable since `1.64.0`:
C:\Users\runneradmin\.cargo\registry\src\index.crates.io-6f17d22bba15001f\tokio-1.41.1\src\macros\select.rs#L505
warning: current MSRV (Minimum Supported Rust Version) is `1.58.0` but this item is stable since `1.64.0`
--> alvr\server\src\lib.rs:241:13
|
241 | / tokio::select! {
242 | | _ = web_server => (),
243 | | _ = SHUTDOWN_NOTIFIER.notified() => (),
244 | | }
| |_____________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#incompatible_msrv
= note: this warning originates in the macro `$crate::select` which comes from the expansion of the macro `tokio::select` (in Nightly builds, run with -Z macro-backtrace for more info)
|
very complex type used. Consider factoring parts into `type` definitions:
alvr\server\src\lib.rs#L54
warning: very complex type used. Consider factoring parts into `type` definitions
--> alvr\server\src\lib.rs:54:30
|
54 | static ref VIDEO_SENDER: Mutex<Option<mpsc::UnboundedSender<(VideoFrameHeaderPacket, Vec<u8>)>>> =
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#type_complexity
= note: `#[warn(clippy::type_complexity)]` on by default
|
current MSRV (Minimum Supported Rust Version) is `1.58.0` but this item is stable since `1.64.0`:
C:\Users\runneradmin\.cargo\registry\src\index.crates.io-6f17d22bba15001f\tokio-1.41.1\src\macros\join.rs#L107
warning: current MSRV (Minimum Supported Rust Version) is `1.58.0` but this item is stable since `1.64.0`
--> alvr\server\src\connection.rs:1107:9
|
1107 | / tokio::join!(
1108 | | async {
1109 | | alvr_common::show_err(connection_pipeline().await);
... |
1114 | | time::sleep(RETRY_CONNECT_MIN_INTERVAL),
1115 | | );
| |_________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#incompatible_msrv
= note: this warning originates in the macro `$crate::join` which comes from the expansion of the macro `tokio::join` (in Nightly builds, run with -Z macro-backtrace for more info)
|
current MSRV (Minimum Supported Rust Version) is `1.58.0` but this item is stable since `1.64.0`:
C:\Users\runneradmin\.cargo\registry\src\index.crates.io-6f17d22bba15001f\tokio-1.41.1\src\macros\select.rs#L515
warning: current MSRV (Minimum Supported Rust Version) is `1.58.0` but this item is stable since `1.64.0`
--> alvr\server\src\connection.rs:1071:5
|
1071 | / tokio::select! {
1072 | | // Spawn new tasks and let the runtime manage threading
1073 | | res = spawn_cancelable(receive_loop) => {
1074 | | alvr_session::log_event(ServerEvent::ClientDisconnected);
... |
1101 | | }
1102 | | }
| |_____^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#incompatible_msrv
= note: this warning originates in the macro `$crate::select` which comes from the expansion of the macro `tokio::select` (in Nightly builds, run with -Z macro-backtrace for more info)
|
current MSRV (Minimum Supported Rust Version) is `1.58.0` but this item is stable since `1.64.0`:
C:\Users\runneradmin\.cargo\registry\src\index.crates.io-6f17d22bba15001f\tokio-1.41.1\src\macros\select.rs#L505
warning: current MSRV (Minimum Supported Rust Version) is `1.58.0` but this item is stable since `1.64.0`
--> alvr\server\src\connection.rs:1071:5
|
1071 | / tokio::select! {
1072 | | // Spawn new tasks and let the runtime manage threading
1073 | | res = spawn_cancelable(receive_loop) => {
1074 | | alvr_session::log_event(ServerEvent::ClientDisconnected);
... |
1101 | | }
1102 | | }
| |_____^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#incompatible_msrv
= note: this warning originates in the macro `$crate::select` which comes from the expansion of the macro `tokio::select` (in Nightly builds, run with -Z macro-backtrace for more info)
|
length comparison to zero:
alvr\server\src\connection.rs#L1019
warning: length comparison to zero
--> alvr\server\src\connection.rs:1019:59
|
1019 | if src_ham.vertices.len() == 0 || src_ham.indices.len() == 0 {
| ^^^^^^^^^^^^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `src_ham.indices.is_empty()`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
|
length comparison to zero:
alvr\server\src\connection.rs#L1019
warning: length comparison to zero
--> alvr\server\src\connection.rs:1019:28
|
1019 | if src_ham.vertices.len() == 0 || src_ham.indices.len() == 0 {
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `src_ham.vertices.is_empty()`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
= note: `#[warn(clippy::len_zero)]` on by default
|
current MSRV (Minimum Supported Rust Version) is `1.58.0` but this item is stable since `1.64.0`:
C:\Users\runneradmin\.cargo\registry\src\index.crates.io-6f17d22bba15001f\tokio-1.41.1\src\macros\select.rs#L515
warning: current MSRV (Minimum Supported Rust Version) is `1.58.0` but this item is stable since `1.64.0`
--> alvr\server\src\connection.rs:594:25
|
594 | let stream_socket = tokio::select! {
| _________________________^
595 | | res = StreamSocketBuilder::connect_to_client(
596 | | client_ip,
597 | | settings.connection.stream_port,
... |
605 | | }
606 | | };
| |_____^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#incompatible_msrv
= note: this warning originates in the macro `$crate::select` which comes from the expansion of the macro `tokio::select` (in Nightly builds, run with -Z macro-backtrace for more info)
|
current MSRV (Minimum Supported Rust Version) is `1.58.0` but this item is stable since `1.64.0`:
C:\Users\runneradmin\.cargo\registry\src\index.crates.io-6f17d22bba15001f\tokio-1.41.1\src\macros\select.rs#L505
warning: current MSRV (Minimum Supported Rust Version) is `1.58.0` but this item is stable since `1.64.0`
--> alvr\server\src\connection.rs:594:25
|
594 | let stream_socket = tokio::select! {
| _________________________^
595 | | res = StreamSocketBuilder::connect_to_client(
596 | | client_ip,
597 | | settings.connection.stream_port,
... |
605 | | }
606 | | };
| |_____^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#incompatible_msrv
= note: this warning originates in the macro `$crate::select` which comes from the expansion of the macro `tokio::select` (in Nightly builds, run with -Z macro-backtrace for more info)
|
current MSRV (Minimum Supported Rust Version) is `1.58.0` but this item is stable since `1.64.0`:
C:\Users\runneradmin\.cargo\registry\src\index.crates.io-6f17d22bba15001f\tokio-1.41.1\src\macros\select.rs#L515
warning: current MSRV (Minimum Supported Rust Version) is `1.58.0` but this item is stable since `1.64.0`
--> alvr\server\src\connection.rs:542:9
|
542 | / tokio::select! {
543 | | res = try_connection_future => {
544 | | match res {
545 | | Either::Left(Ok(client_ip)) => {
... |
562 | | _ = CLIENTS_UPDATED_NOTIFIER.notified() => return Ok(()),
563 | | };
| |_________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#incompatible_msrv
= note: this warning originates in the macro `$crate::select` which comes from the expansion of the macro `tokio::select` (in Nightly builds, run with -Z macro-backtrace for more info)
|
current MSRV (Minimum Supported Rust Version) is `1.58.0` but this item is stable since `1.64.0`:
C:\Users\runneradmin\.cargo\registry\src\index.crates.io-6f17d22bba15001f\tokio-1.41.1\src\macros\select.rs#L505
warning: current MSRV (Minimum Supported Rust Version) is `1.58.0` but this item is stable since `1.64.0`
--> alvr\server\src\connection.rs:542:9
|
542 | / tokio::select! {
543 | | res = try_connection_future => {
544 | | match res {
545 | | Either::Left(Ok(client_ip)) => {
... |
562 | | _ = CLIENTS_UPDATED_NOTIFIER.notified() => return Ok(()),
563 | | };
| |_________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#incompatible_msrv
= note: `#[warn(clippy::incompatible_msrv)]` on by default
= note: this warning originates in the macro `$crate::select` which comes from the expansion of the macro `tokio::select` (in Nightly builds, run with -Z macro-backtrace for more info)
|
field `version` is never read:
alvr\server\src\connection.rs#L88
warning: field `version` is never read
--> alvr\server\src\connection.rs:88:5
|
86 | struct ConnectionInfo {
| -------------- field in this struct
87 | client_ip: IpAddr,
88 | version: Option<Version>,
| ^^^^^^^
|
field `hostname` is never read:
alvr\server\src\connection.rs#L53
warning: field `hostname` is never read
--> alvr\server\src\connection.rs:53:5
|
52 | struct ClientId {
| -------- field in this struct
53 | hostname: String,
| ^^^^^^^^
|
= note: `ClientId` has a derived impl for the trait `Clone`, but this is intentionally ignored during dead code analysis
= note: `#[warn(dead_code)]` on by default
|
unused import: `prelude::*`:
alvr\server\src\lib.rs#L19
warning: unused import: `prelude::*`
--> alvr\server\src\lib.rs:19:37
|
19 | use alvr_common::{lazy_static, log, prelude::*, ALVR_VERSION};
| ^^^^^^^^^^
|
unused imports: `Mat4` and `log`:
alvr\server\src\connection.rs#L8
warning: unused imports: `Mat4` and `log`
--> alvr\server\src\connection.rs:8:12
|
8 | glam::{Mat4, Quat, Vec2, Vec3},
| ^^^^
9 | log,
| ^^^
|
= note: `#[warn(unused_imports)]` on by default
|
the borrowed expression implements the required traits:
alvr\launcher\src\commands.rs#L62
warning: the borrowed expression implements the required traits
--> alvr\launcher\src\commands.rs:62:15
|
62 | .args(&["/PID", &pid.to_string(), "/F"])
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: change this to: `["/PID", &pid.to_string(), "/F"]`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
|
the borrowed expression implements the required traits:
alvr\launcher\src\commands.rs#L52
warning: the borrowed expression implements the required traits
--> alvr\launcher\src\commands.rs:52:50
|
52 | spawn_no_window(Command::new("cmd").args(&["/C", "start", "steam://rungameid/250820"]));
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: change this to: `["/C", "start", "steam://rungameid/250820"]`
|
= 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
|
useless use of `vec!`:
alvr\xtask\src/main.rs#L1165
warning: useless use of `vec!`
--> alvr\xtask\src/main.rs:1165:36
|
1165 | let flavours = vec![
| ____________________________________^
1166 | | (for_generic, AndroidFlavor::Generic, abi_target.clone()),
1167 | | (for_pico, AndroidFlavor::Pico, Option::None),
1168 | | ];
| |_____________________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_vec
= note: `#[warn(clippy::useless_vec)]` on by default
help: you can use an array directly
|
1165 ~ let flavours = [(for_generic, AndroidFlavor::Generic, abi_target.clone()),
1166 ~ (for_pico, AndroidFlavor::Pico, Option::None)];
|
|
struct update has no effect, all the fields in the struct have already been specified:
alvr\xtask\src/main.rs#L1098
warning: struct update has no effect, all the fields in the struct have already been specified
--> alvr\xtask\src/main.rs:1098:27
|
1098 | ..Default::default()
| ^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_update
= note: `#[warn(clippy::needless_update)]` on by default
|
this expression creates a reference which is immediately dereferenced by the compiler:
alvr\xtask\src/main.rs#L977
warning: this expression creates a reference which is immediately dereferenced by the compiler
--> alvr\xtask\src/main.rs:977:41
|
977 | .filter(|entry| is_package_file(&entry))
| ^^^^^^ help: change this to: `entry`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
|
unneeded `return` statement:
alvr\xtask\src/main.rs#L969
warning: unneeded `return` statement
--> alvr\xtask\src/main.rs:969:13
|
969 | return ["apk", "aar", "idsig"].contains(&ext_str);
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
help: remove `return`
|
969 - return ["apk", "aar", "idsig"].contains(&ext_str);
969 + ["apk", "aar", "idsig"].contains(&ext_str)
|
|
this expression creates a reference which is immediately dereferenced by the compiler:
alvr\xtask\src/main.rs#L850
warning: this expression creates a reference which is immediately dereferenced by the compiler
--> alvr\xtask\src/main.rs:850:61
|
850 | let run_ait_cmd = |cmd: &str| command::run_in(&ait_dir, &cmd).unwrap();
| ^^^^ help: change this to: `cmd`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
|
manual implementation of an assign operation:
alvr\xtask\src/main.rs#L823
warning: manual implementation of an assign operation
--> alvr\xtask\src/main.rs:823:9
|
823 | archs = archs + "_debug";
| ^^^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `archs += "_debug"`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assign_op_pattern
= note: `#[warn(clippy::assign_op_pattern)]` on by default
|
the borrowed expression implements the required traits:
alvr\xtask\src/main.rs#L817
warning: the borrowed expression implements the required traits
--> alvr\xtask\src/main.rs:817:36
|
817 | alxr_client_build_dir.join(&cert),
| ^^^^^ help: change this to: `cert`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
|
the borrowed expression implements the required traits:
alvr\xtask\src/main.rs#L816
warning: the borrowed expression implements the required traits
--> alvr\xtask\src/main.rs:816:30
|
816 | alxr_client_dir.join(&cert),
| ^^^^^ help: change this to: `cert`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
|
redundant closure:
alvr\xtask\src/main.rs#L786
warning: redundant closure
--> alvr\xtask\src/main.rs:786:57
|
786 | for arch in [UWPArch::X86_64, UWPArch::Aarch64].map(|x| batch_arch_str(x)) {
| ^^^^^^^^^^^^^^^^^^^^^ help: replace the closure with the function itself: `batch_arch_str`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_closure
= note: `#[warn(clippy::redundant_closure)]` on by default
|
the borrowed expression implements the required traits:
alvr\xtask\src/main.rs#L783
warning: the borrowed expression implements the required traits
--> alvr\xtask\src/main.rs:783:52
|
783 | let pack_map_path = alxr_client_build_dir.join(&pack_map_fname);
| ^^^^^^^^^^^^^^^ help: change this to: `pack_map_fname`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
|
this expression creates a reference which is immediately dereferenced by the compiler:
alvr\xtask\src/main.rs#L779
warning: this expression creates a reference which is immediately dereferenced by the compiler
--> alvr\xtask\src/main.rs:779:24
|
779 | .unwrap_or(&alxr_client_build_dir),
| ^^^^^^^^^^^^^^^^^^^^^^ help: change this to: `alxr_client_build_dir`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
|
the borrowed expression implements the required traits:
alvr\xtask\src/main.rs#L733
warning: the borrowed expression implements the required traits
--> alvr\xtask\src/main.rs:733:69
|
733 | let relative_path = linked_depend_file.strip_prefix(&linked_path).unwrap();
| ^^^^^^^^^^^^ help: change this to: `linked_path`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
|
this expression creates a reference which is immediately dereferenced by the compiler:
alvr\xtask\src/main.rs#L731
warning: this expression creates a reference which is immediately dereferenced by the compiler
--> alvr\xtask\src/main.rs:731:56
|
731 | .filter(|entry| is_linked_depends_file(&entry))
| ^^^^^^ help: change this to: `entry`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
|
this expression creates a reference which is immediately dereferenced by the compiler:
alvr\xtask\src/main.rs#L683
warning: this expression creates a reference which is immediately dereferenced by the compiler
--> alvr\xtask\src/main.rs:683:36
|
683 | if afs::is_dynlib_file(&path) {
| ^^^^^ help: change this to: `path`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
|
unneeded `return` statement:
alvr\xtask\src/main.rs#L703
warning: unneeded `return` statement
--> alvr\xtask\src/main.rs:703:13
|
703 | return false;
| ^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
help: remove `return`
|
703 - return false;
703 + false
|
|
the borrowed expression implements the required traits:
alvr\xtask\src/main.rs#L612
warning: the borrowed expression implements the required traits
--> alvr\xtask\src/main.rs:612:40
|
612 | alxr_client_build_dir.join(&pdb_fname),
| ^^^^^^^^^^ help: change this to: `pdb_fname`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
|
the borrowed expression implements the required traits:
alvr\xtask\src/main.rs#L611
warning: the borrowed expression implements the required traits
--> alvr\xtask\src/main.rs:611:32
|
611 | artifacts_dir.join(&pdb_fname),
| ^^^^^^^^^^ help: change this to: `pdb_fname`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
|
this expression creates a reference which is immediately dereferenced by the compiler:
alvr\xtask\src/main.rs#L599
warning: this expression creates a reference which is immediately dereferenced by the compiler
--> alvr\xtask\src/main.rs:599:52
|
599 | .filter(|entry| is_linked_depends_file(&entry))
| ^^^^^^ help: change this to: `entry`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
|
this expression creates a reference which is immediately dereferenced by the compiler:
alvr\xtask\src/main.rs#L568
warning: this expression creates a reference which is immediately dereferenced by the compiler
--> alvr\xtask\src/main.rs:568:32
|
568 | if afs::is_dynlib_file(&path) {
| ^^^^^ help: change this to: `path`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
|
unneeded `return` statement:
alvr\xtask\src/main.rs#L588
warning: unneeded `return` statement
--> alvr\xtask\src/main.rs:588:9
|
588 | return false;
| ^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
= note: `#[warn(clippy::needless_return)]` on by default
help: remove `return`
|
588 - return false;
588 + false
|
|
the borrowed expression implements the required traits:
alvr\xtask\src/main.rs#L525
warning: the borrowed expression implements the required traits
--> alvr\xtask\src/main.rs:525:62
|
525 | if so_filename.to_string_lossy().starts_with(&key) {
| ^^^^ help: change this to: `key`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
|
this expression creates a reference which is immediately dereferenced by the compiler:
alvr\xtask\src/main.rs#L522
warning: this expression creates a reference which is immediately dereferenced by the compiler
--> alvr\xtask\src/main.rs:522:52
|
522 | .filter(|path| afs::is_dynlib_file(&path))
| ^^^^^ help: change this to: `path`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
|
this expression creates a reference which is immediately dereferenced by the compiler:
alvr\xtask\src/main.rs#L508
warning: this expression creates a reference which is immediately dereferenced by the compiler
--> alvr\xtask\src/main.rs:508:13
|
508 | &ffmpeg_build_dir,
| ^^^^^^^^^^^^^^^^^ help: change this to: `ffmpeg_build_dir`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
= note: `#[warn(clippy::needless_borrow)]` on by default
|
length comparison to zero:
alvr\xtask\src/main.rs#L475
warning: length comparison to zero
--> alvr\xtask\src/main.rs:475:16
|
475 | if build_str.len() > 0 {
| ^^^^^^^^^^^^^^^^^^^ help: using `!is_empty` is clearer and more explicit: `!build_str.is_empty()`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
|
length comparison to zero:
alvr\xtask\src/main.rs#L474
warning: length comparison to zero
--> alvr\xtask\src/main.rs:474:12
|
474 | if features.len() > 0 {
| ^^^^^^^^^^^^^^^^^^ help: using `!is_empty` is clearer and more explicit: `!features.is_empty()`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
|
writing `&Vec` instead of `&[_]` involves a new object where a slice will do:
alvr\xtask\src/main.rs#L462
warning: writing `&Vec` instead of `&[_]` involves a new object where a slice will do
--> alvr\xtask\src/main.rs:462:26
|
462 | fn to_str_vec(m: &Vec<(bool, &'static str)>) -> Vec<&'static str> {
| ^^^^^^^^^^^^^^^^^^^^^^^^^^ help: change this to: `&[(bool, &'static str)]`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#ptr_arg
= note: `#[warn(clippy::ptr_arg)]` on by default
|
you seem to be trying to use `match` for destructuring a single pattern. Consider using `if let`:
alvr\xtask\src/main.rs#L407
warning: you seem to be trying to use `match` for destructuring a single pattern. Consider using `if let`
--> alvr\xtask\src/main.rs:407:21
|
407 | / match lp.as_str().strip_prefix("native=") {
408 | | Some(p) => {
409 | | linked_path_set.insert(p.into());
410 | | }
411 | | None => (),
412 | | }
| |_____________________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_match
help: try
|
407 ~ if let Some(p) = lp.as_str().strip_prefix("native=") {
408 + linked_path_set.insert(p.into());
409 + }
|
|
you seem to be trying to use `match` for destructuring a single pattern. Consider using `if let`:
alvr\xtask\src/main.rs#L404
warning: you seem to be trying to use `match` for destructuring a single pattern. Consider using `if let`
--> alvr\xtask\src/main.rs:404:9
|
404 | / match message? {
405 | | Message::BuildScriptExecuted(script) => {
406 | | for lp in script.linked_paths.iter() {
407 | | match lp.as_str().strip_prefix("native=") {
... |
415 | | _ => (),
416 | | }
| |_________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_match
= note: `#[warn(clippy::single_match)]` on by default
help: try
|
404 ~ if let Message::BuildScriptExecuted(script) = message? {
405 + for lp in script.linked_paths.iter() {
406 + match lp.as_str().strip_prefix("native=") {
407 + Some(p) => {
408 + linked_path_set.insert(p.into());
409 + }
410 + None => (),
411 + }
412 + }
413 + }
|
|
the borrowed expression implements the required traits:
alvr\xtask\src/main.rs#L389
warning: the borrowed expression implements the required traits
--> alvr\xtask\src/main.rs:389:50
|
389 | let mut command = std::process::Command::new(&cmd);
| ^^^^ help: change this to: `cmd`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
|
length comparison to zero:
alvr\xtask\src/main.rs#L375
warning: length comparison to zero
--> alvr\xtask\src/main.rs:375:8
|
375 | if nightly_version.len() > 0 {
| ^^^^^^^^^^^^^^^^^^^^^^^^^ help: using `!is_empty` is clearer and more explicit: `!nightly_version.is_empty()`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
|
the borrowed expression implements the required traits:
alvr\xtask\src/main.rs#L337
warning: the borrowed expression implements the required traits
--> alvr\xtask\src/main.rs:337:24
|
337 | fs::create_dir_all(&afs::build_dir().join(&artifact_name)).unwrap();
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: change this to: `afs::build_dir().join(&artifact_name)`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
|
the borrowed expression implements the required traits:
alvr\xtask\src/main.rs#L284
warning: the borrowed expression implements the required traits
--> alvr\xtask\src/main.rs:284:24
|
284 | fs::create_dir_all(&layout.dashboard_dir()).unwrap();
| ^^^^^^^^^^^^^^^^^^^^^^^ help: change this to: `layout.dashboard_dir()`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
|
the borrowed expression implements the required traits:
alvr\xtask\src/main.rs#L271
warning: the borrowed expression implements the required traits
--> alvr\xtask\src/main.rs:271:24
|
271 | fs::create_dir_all(&layout.resources_dir()).unwrap();
| ^^^^^^^^^^^^^^^^^^^^^^^ help: change this to: `layout.resources_dir()`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
|
the borrowed expression implements the required traits:
alvr\xtask\src/main.rs#L163
warning: the borrowed expression implements the required traits
--> alvr\xtask\src/main.rs:163:48
|
163 | fs::copy(lib.clone(), lib_dir.join(&lib_filename)).unwrap();
| ^^^^^^^^^^^^^ help: change this to: `lib_filename`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
|
the borrowed expression implements the required traits:
alvr\xtask\src/main.rs#L136
warning: the borrowed expression implements the required traits
--> alvr\xtask\src/main.rs:136:24
|
136 | fs::create_dir_all(&layout.launcher_exe().parent().unwrap()).unwrap();
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: change this to: `layout.launcher_exe().parent().unwrap()`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
|
the borrowed expression implements the required traits:
alvr\xtask\src/main.rs#L135
warning: the borrowed expression implements the required traits
--> alvr\xtask\src/main.rs:135:24
|
135 | fs::create_dir_all(&layout.openvr_driver_lib().parent().unwrap()).unwrap();
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: change this to: `layout.openvr_driver_lib().parent().unwrap()`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
|
the borrowed expression implements the required traits:
alvr\xtask\src/main.rs#L134
warning: the borrowed expression implements the required traits
--> alvr\xtask\src/main.rs:134:24
|
134 | fs::create_dir_all(&afs::server_build_dir()).unwrap();
| ^^^^^^^^^^^^^^^^^^^^^^^^ help: change this to: `afs::server_build_dir()`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
|
the borrowed expression implements the required traits:
alvr\xtask\src/main.rs#L133
warning: the borrowed expression implements the required traits
--> alvr\xtask\src/main.rs:133:24
|
133 | fs::remove_dir_all(&afs::server_build_dir()).ok();
| ^^^^^^^^^^^^^^^^^^^^^^^^ help: change this to: `afs::server_build_dir()`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
|
this function has too many arguments (8/7):
alvr\xtask\src/main.rs#L79
warning: this function has too many arguments (8/7)
--> alvr\xtask\src/main.rs:79:1
|
79 | / pub fn build_server(
80 | | is_release: bool,
81 | | experiments: bool,
82 | | fetch_crates: bool,
... |
87 | | reproducible: bool,
88 | | ) {
| |_^
|
= 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
|
the borrowed expression implements the required traits:
alvr\xtask\src\version.rs#L177
warning: the borrowed expression implements the required traits
--> alvr\xtask\src\version.rs:177:28
|
177 | .map(|d| base_dir.join(&d).to_str().unwrap().to_owned())
| ^^ help: change this to: `d`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
|
`filter` for `Ok` followed by `unwrap`:
alvr\xtask\src\dependencies.rs#L204
warning: `filter` for `Ok` followed by `unwrap`
--> alvr\xtask\src\dependencies.rs:204:18
|
204 | .filter(|result| result.is_ok())
| __________________^
205 | | .map(|pp| pp.unwrap())
| |______________________________________^ help: consider using `flatten` instead: `flatten()`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#result_filter_map
= note: `#[warn(clippy::result_filter_map)]` on by default
|
the borrowed expression implements the required traits:
alvr\xtask\src\dependencies.rs#L196
warning: the borrowed expression implements the required traits
--> alvr\xtask\src\dependencies.rs:196:15
|
196 | .args(&["-c", &cmdline])
| ^^^^^^^^^^^^^^^^^ help: change this to: `["-c", &cmdline]`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
|
length comparison to zero:
alvr\xtask\src\dependencies.rs#L124
warning: length comparison to zero
--> alvr\xtask\src\dependencies.rs:124:8
|
124 | if install_prefix.len() > 0 {
| ^^^^^^^^^^^^^^^^^^^^^^^^ help: using `!is_empty` is clearer and more explicit: `!install_prefix.is_empty()`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
|
length comparison to zero:
alvr\xtask\src\dependencies.rs#L52
warning: length comparison to zero
--> alvr\xtask\src\dependencies.rs:52:22
|
52 | Some(ips) if ips.len() > 0 => {
| ^^^^^^^^^^^^^ help: using `!is_empty` is clearer and more explicit: `!ips.is_empty()`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
= note: `#[warn(clippy::len_zero)]` on by default
|
the borrowed expression implements the required traits:
alvr\xtask\src\dependencies.rs#L13
warning: the borrowed expression implements the required traits
--> alvr\xtask\src\dependencies.rs:13:24
|
13 | fs::create_dir_all(&destination).unwrap();
| ^^^^^^^^^^^^ help: change this to: `destination`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
|
the borrowed expression implements the required traits:
alvr\xtask\src\dependencies.rs#L12
warning: the borrowed expression implements the required traits
--> alvr\xtask\src\dependencies.rs:12:24
|
12 | fs::remove_dir_all(&destination).ok();
| ^^^^^^^^^^^^ help: change this to: `destination`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
|
useless use of `format!`:
alvr\xtask\src\command.rs#L154
warning: useless use of `format!`
--> alvr\xtask\src\command.rs:154:31
|
154 | let result = Command::new(format!("cargo"))
| ^^^^^^^^^^^^^^^^ help: consider using `.to_string()`: `"cargo".to_string()`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_format
= note: `#[warn(clippy::useless_format)]` on by default
|
used consecutive `str::replace` call:
alvr\xtask\src\command.rs#L149
warning: used consecutive `str::replace` call
--> alvr\xtask\src\command.rs:149:10
|
149 | .replace('\r', "")
| __________^
150 | | .replace('\n', "")
| |__________________________^ help: replace with: `replace(['\r', '\n'], "")`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#collapsible_str_replace
= note: `#[warn(clippy::collapsible_str_replace)]` on by default
|
the borrowed expression implements the required traits:
alvr\xtask\src\command.rs#L141
warning: the borrowed expression implements the required traits
--> alvr\xtask\src\command.rs:141:19
|
141 | .args(&["-u", "+%Y.%m.%d"])
| ^^^^^^^^^^^^^^^^^^^^ help: change this to: `["-u", "+%Y.%m.%d"]`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
|
the borrowed expression implements the required traits:
alvr\xtask\src\command.rs#L29
warning: the borrowed expression implements the required traits
--> alvr\xtask\src\command.rs:29:15
|
29 | .args(&[shell_flag, cmd])
| ^^^^^^^^^^^^^^^^^^ help: change this to: `[shell_flag, cmd]`
|
= 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
|
variable does not need to be mutable:
alvr\xtask\src/main.rs#L1057
warning: variable does not need to be mutable
--> alvr\xtask\src/main.rs:1057:13
|
1057 | let mut ffmpeg_version: String =
| ----^^^^^^^^^^^^^^
| |
| help: remove this `mut`
|
= note: `#[warn(unused_mut)]` on by default
|
redundant field names in struct initialization:
alvr\xtask\src/main.rs#L1203
warning: redundant field names in struct initialization
--> alvr\xtask\src/main.rs:1203:25
|
1203 | no_decoder: no_decoder,
| ^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `no_decoder`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
|
redundant field names in struct initialization:
alvr\xtask\src/main.rs#L1199
warning: redundant field names in struct initialization
--> alvr\xtask\src/main.rs:1199:25
|
1199 | reproducible: reproducible,
| ^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `reproducible`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
|
redundant field names in struct initialization:
alvr\xtask\src/main.rs#L1198
warning: redundant field names in struct initialization
--> alvr\xtask\src/main.rs:1198:25
|
1198 | is_release: is_release,
| ^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `is_release`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
|
redundant field names in struct initialization:
alvr\xtask\src/main.rs#L1189
warning: redundant field names in struct initialization
--> alvr\xtask\src/main.rs:1189:25
|
1189 | no_decoder: no_decoder,
| ^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `no_decoder`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
|
redundant field names in struct initialization:
alvr\xtask\src/main.rs#L1185
warning: redundant field names in struct initialization
--> alvr\xtask\src/main.rs:1185:25
|
1185 | reproducible: reproducible,
| ^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `reproducible`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
|
redundant field names in struct initialization:
alvr\xtask\src/main.rs#L1184
warning: redundant field names in struct initialization
--> alvr\xtask\src/main.rs:1184:25
|
1184 | is_release: is_release,
| ^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `is_release`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
|
redundant field names in struct initialization:
alvr\xtask\src/main.rs#L1162
warning: redundant field names in struct initialization
--> alvr\xtask\src/main.rs:1162:25
|
1162 | no_decoder: no_decoder,
| ^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `no_decoder`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
|
redundant field names in struct initialization:
alvr\xtask\src/main.rs#L1158
warning: redundant field names in struct initialization
--> alvr\xtask\src/main.rs:1158:25
|
1158 | reproducible: reproducible,
| ^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `reproducible`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
|
redundant field names in struct initialization:
alvr\xtask\src/main.rs#L1157
warning: redundant field names in struct initialization
--> alvr\xtask\src/main.rs:1157:25
|
1157 | is_release: is_release,
| ^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `is_release`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
|
redundant field names in struct initialization:
alvr\xtask\src/main.rs#L1151
warning: redundant field names in struct initialization
--> alvr\xtask\src/main.rs:1151:25
|
1151 | no_decoder: no_decoder,
| ^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `no_decoder`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
|
redundant field names in struct initialization:
alvr\xtask\src/main.rs#L1149
warning: redundant field names in struct initialization
--> alvr\xtask\src/main.rs:1149:25
|
1149 | bundle_ffmpeg: bundle_ffmpeg,
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `bundle_ffmpeg`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
|
redundant field names in struct initialization:
alvr\xtask\src/main.rs#L1148
warning: redundant field names in struct initialization
--> alvr\xtask\src/main.rs:1148:25
|
1148 | no_nvidia: no_nvidia,
| ^^^^^^^^^^^^^^^^^^^^ help: replace it with: `no_nvidia`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
|
redundant field names in struct initialization:
alvr\xtask\src/main.rs#L1147
warning: redundant field names in struct initialization
--> alvr\xtask\src/main.rs:1147:25
|
1147 | reproducible: reproducible,
| ^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `reproducible`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
|
redundant field names in struct initialization:
alvr\xtask\src/main.rs#L1146
warning: redundant field names in struct initialization
--> alvr\xtask\src/main.rs:1146:25
|
1146 | is_release: is_release,
| ^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `is_release`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
|
redundant field names in struct initialization:
alvr\xtask\src/main.rs#L1134
warning: redundant field names in struct initialization
--> alvr\xtask\src/main.rs:1134:25
|
1134 | no_decoder: no_decoder,
| ^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `no_decoder`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
|
redundant field names in struct initialization:
alvr\xtask\src/main.rs#L1130
warning: redundant field names in struct initialization
--> alvr\xtask\src/main.rs:1130:25
|
1130 | reproducible: reproducible,
| ^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `reproducible`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
|
redundant field names in struct initialization:
alvr\xtask\src/main.rs#L1129
warning: redundant field names in struct initialization
--> alvr\xtask\src/main.rs:1129:25
|
1129 | is_release: is_release,
| ^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `is_release`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
|
redundant field names in struct initialization:
alvr\xtask\src/main.rs#L1123
warning: redundant field names in struct initialization
--> alvr\xtask\src/main.rs:1123:25
|
1123 | no_decoder: no_decoder,
| ^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `no_decoder`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
|
redundant field names in struct initialization:
alvr\xtask\src/main.rs#L1119
warning: redundant field names in struct initialization
--> alvr\xtask\src/main.rs:1119:25
|
1119 | reproducible: reproducible,
| ^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `reproducible`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
|
redundant field names in struct initialization:
alvr\xtask\src/main.rs#L1118
warning: redundant field names in struct initialization
--> alvr\xtask\src/main.rs:1118:25
|
1118 | is_release: is_release,
| ^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `is_release`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
|
redundant field names in struct initialization:
alvr\xtask\src/main.rs#L1110
warning: redundant field names in struct initialization
--> alvr\xtask\src/main.rs:1110:25
|
1110 | no_decoder: no_decoder,
| ^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `no_decoder`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
|
redundant field names in struct initialization:
alvr\xtask\src/main.rs#L1106
warning: redundant field names in struct initialization
--> alvr\xtask\src/main.rs:1106:25
|
1106 | reproducible: reproducible,
| ^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `reproducible`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
|
redundant field names in struct initialization:
alvr\xtask\src/main.rs#L1105
warning: redundant field names in struct initialization
--> alvr\xtask\src/main.rs:1105:25
|
1105 | is_release: is_release,
| ^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `is_release`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
|
redundant field names in struct initialization:
alvr\xtask\src/main.rs#L1097
warning: redundant field names in struct initialization
--> alvr\xtask\src/main.rs:1097:25
|
1097 | oculus_ext: oculus_ext,
| ^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `oculus_ext`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
|
redundant field names in struct initialization:
alvr\xtask\src/main.rs#L1096
warning: redundant field names in struct initialization
--> alvr\xtask\src/main.rs:1096:25
|
1096 | no_decoder: no_decoder,
| ^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `no_decoder`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
|
redundant field names in struct initialization:
alvr\xtask\src/main.rs#L1094
warning: redundant field names in struct initialization
--> alvr\xtask\src/main.rs:1094:25
|
1094 | bundle_ffmpeg: bundle_ffmpeg,
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `bundle_ffmpeg`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
|
redundant field names in struct initialization:
alvr\xtask\src/main.rs#L1093
warning: redundant field names in struct initialization
--> alvr\xtask\src/main.rs:1093:25
|
1093 | no_nvidia: no_nvidia,
| ^^^^^^^^^^^^^^^^^^^^ help: replace it with: `no_nvidia`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
|
redundant field names in struct initialization:
alvr\xtask\src/main.rs#L1092
warning: redundant field names in struct initialization
--> alvr\xtask\src/main.rs:1092:25
|
1092 | reproducible: reproducible,
| ^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `reproducible`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
|
redundant field names in struct initialization:
alvr\xtask\src/main.rs#L1091
warning: redundant field names in struct initialization
--> alvr\xtask\src/main.rs:1091:25
|
1091 | is_release: is_release,
| ^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `is_release`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
= note: `#[warn(clippy::redundant_field_names)]` on by default
|
deref which would be done by auto-deref:
alvr\audio\src\lib.rs#L638
warning: deref which would be done by auto-deref
--> alvr\audio\src\lib.rs:638:17
|
638 | &mut *self.sample_buffer.lock(),
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `&mut self.sample_buffer.lock()`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#explicit_auto_deref
= note: `#[warn(clippy::explicit_auto_deref)]` on by default
|
accessing first element with `from_openvr_paths(paths_json).get(0)`:
alvr\commands\src\openvrpaths.rs#L66
warning: accessing first element with `from_openvr_paths(paths_json).get(0)`
--> alvr\commands\src\openvrpaths.rs:66:17
|
66 | trace_none!(from_openvr_paths(paths_json).get(0).cloned())
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `from_openvr_paths(paths_json).first()`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#get_first
= note: `#[warn(clippy::get_first)]` on by default
|
unneeded late initialization:
alvr\commands\src\firewall.rs#L20
warning: unneeded late initialization
--> alvr\commands\src\firewall.rs:20:5
|
20 | let exit_status;
| ^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_late_init
= note: `#[warn(clippy::needless_late_init)]` on by default
help: move the declaration `exit_status` here and remove the assignments from the branches
|
20 ~
21 |
22 ~ let exit_status = if cfg!(target_os = "linux") {
23 | let action = if add { "add" } else { "remove" };
24 | // run as normal user since we use pkexec to sudo
25 ~ Command::new("bash")
26 | .arg("/usr/libexec/alvr/alvr_fw_config.sh")
27 | .arg(action)
28 | .status()
29 ~ .map_err(|_| -1)?
30 | } else {
...
61 | // run with admin privileges
62 ~ runas::Command::new(script_path)
63 | .gui(true) // UAC, if available
64 | .status()
65 ~ .map_err(|_| -1)?
66 ~ };
|
|
unused import: `alvr_filesystem as afs`:
alvr\server\build.rs#L1
warning: unused import: `alvr_filesystem as afs`
--> alvr\server\build.rs:1:5
|
1 | use alvr_filesystem as afs;
| ^^^^^^^^^^^^^^^^^^^^^^
|
= note: `#[warn(unused_imports)]` on by default
|
current MSRV (Minimum Supported Rust Version) is `1.58.0` but this item is stable since `1.64.0`:
C:\Users\runneradmin\.cargo\registry\src\index.crates.io-6f17d22bba15001f\tokio-1.41.1\src\macros\select.rs#L515
warning: current MSRV (Minimum Supported Rust Version) is `1.58.0` but this item is stable since `1.64.0`
--> alvr\sockets\src\lib.rs:74:17
|
74 | / tokio::select! {
75 | | res = future => res,
76 | | _ = cancel_receiver => Ok(()),
77 | | }
| |_________________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#incompatible_msrv
= note: this warning originates in the macro `$crate::select` which comes from the expansion of the macro `tokio::select` (in Nightly builds, run with -Z macro-backtrace for more info)
|
current MSRV (Minimum Supported Rust Version) is `1.58.0` but this item is stable since `1.64.0`:
C:\Users\runneradmin\.cargo\registry\src\index.crates.io-6f17d22bba15001f\tokio-1.41.1\src\macros\select.rs#L505
warning: current MSRV (Minimum Supported Rust Version) is `1.58.0` but this item is stable since `1.64.0`
--> alvr\sockets\src\lib.rs:74:17
|
74 | / tokio::select! {
75 | | res = future => res,
76 | | _ = cancel_receiver => Ok(()),
77 | | }
| |_________________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#incompatible_msrv
= note: `#[warn(clippy::incompatible_msrv)]` on by default
= note: this warning originates in the macro `$crate::select` which comes from the expansion of the macro `tokio::select` (in Nightly builds, run with -Z macro-backtrace for more info)
|
field `stream_id` is never read:
alvr\sockets\src\stream_socket\mod.rs#L220
warning: field `stream_id` is never read
--> alvr\sockets\src\stream_socket\mod.rs:220:5
|
219 | pub struct StreamReceiver<T> {
| -------------- field in this struct
220 | stream_id: StreamId,
| ^^^^^^^^^
|
= note: `#[warn(dead_code)]` on by default
|
unused import: `collections::HashMap`:
alvr\sockets\src\packets.rs#L1
warning: unused import: `collections::HashMap`
--> alvr\sockets\src\packets.rs:1:11
|
1 | use std::{collections::HashMap, time::Duration};
| ^^^^^^^^^^^^^^^^^^^^
|
= note: `#[warn(unused_imports)]` on by default
|
the borrowed expression implements the required traits:
alvr\session\src\lib.rs#L617
warning: the borrowed expression implements the required traits
--> alvr\session\src\lib.rs:617:53
|
617 | let session_desc = match fs::read_to_string(&session_path) {
| ^^^^^^^^^^^^^ help: change this to: `session_path`
|
= 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
|
unused import: `DictionaryDefault`:
alvr\session\src\settings.rs#L4
warning: unused import: `DictionaryDefault`
--> alvr\session\src\settings.rs:4:23
|
4 | use settings_schema::{DictionaryDefault, EntryData, SettingsSchema, Switch, SwitchDefault};
| ^^^^^^^^^^^^^^^^^
|
unused import: `alvr_common::ALVR_VERSION`:
alvr\session\src\settings.rs#L1
warning: unused import: `alvr_common::ALVR_VERSION`
--> alvr\session\src\settings.rs:1:5
|
1 | use alvr_common::ALVR_VERSION;
| ^^^^^^^^^^^^^^^^^^^^^^^^^
|
= note: `#[warn(unused_imports)]` on by default
|
useless use of `format!`:
alvr\filesystem\src\lib.rs#L101
warning: useless use of `format!`
--> alvr\filesystem\src\lib.rs:101:27
|
101 | let alxr_client_dir = format!("alxr_client_uwp");
| ^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using `.to_string()`: `"alxr_client_uwp".to_string()`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_format
= note: `#[warn(clippy::useless_format)]` on by default
|
unneeded `return` statement:
alvr\filesystem\src\lib.rs#L29
warning: unneeded `return` statement
--> alvr\filesystem\src\lib.rs:29:5
|
29 | return false;
| ^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
= note: `#[warn(clippy::needless_return)]` on by default
help: remove `return`
|
29 - return false;
29 + false
|
|
useless use of `format!`:
alvr\filesystem\src\lib.rs#L101
warning: useless use of `format!`
--> alvr\filesystem\src\lib.rs:101:27
|
101 | let alxr_client_dir = format!("alxr_client_uwp");
| ^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using `.to_string()`: `"alxr_client_uwp".to_string()`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_format
= note: `#[warn(clippy::useless_format)]` on by default
|
unneeded `return` statement:
alvr\filesystem\src\lib.rs#L29
warning: unneeded `return` statement
--> alvr\filesystem\src\lib.rs:29:5
|
29 | return false;
| ^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
= note: `#[warn(clippy::needless_return)]` on by default
help: remove `return`
|
29 - return false;
29 + false
|
|
the borrowed expression implements the required traits:
alvr\settings-schema-derive\src\lib.rs#L406
warning: the borrowed expression implements the required traits
--> alvr\settings-schema-derive\src\lib.rs:406:40
|
406 | _ => error("Unsupported type", &ty),
| ^^^ help: change this to: `ty`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
|
the borrowed expression implements the required traits:
alvr\settings-schema-derive\src\lib.rs#L403
warning: the borrowed expression implements the required traits
--> alvr\settings-schema-derive\src\lib.rs:403:78
|
403 | error("Generics are supported only for Option, Switch, Vec", &ty)
| ^^^ help: change this to: `ty`
|
= 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
|
deref which would be done by auto-deref:
alvr\settings-schema-derive\src\lib.rs#L276
warning: deref which would be done by auto-deref
--> alvr\settings-schema-derive\src\lib.rs:276:29
|
276 | } = type_schema(&*ty_array.elem, schema_attrs)?;
| ^^^^^^^^^^^^^^^ help: try: `&ty_array.elem`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#explicit_auto_deref
= note: `#[warn(clippy::explicit_auto_deref)]` on by default
|