Skip to content

Zmerp staging

Zmerp staging #4456

GitHub Actions / clippy succeeded Nov 11, 2024 in 0s

clippy

35 warnings

Details

Results

Message level Amount
Internal compiler error 0
Error 0
Warning 35
Note 0
Help 0

Versions

  • rustc 1.82.0 (f6e511eec 2024-10-15)
  • cargo 1.82.0 (8f40fc59f 2024-08-21)
  • clippy 0.1.82 (f6e511ee 2024-10-15)

Annotations

Check warning on line 8 in alvr/dashboard/src/dashboard/components/settings_controls/notice.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

function `notice` is never used

warning: function `notice` is never used
 --> alvr/dashboard/src/dashboard/components/settings_controls/notice.rs:8:8
  |
8 | pub fn notice(ui: &mut Ui, text: &str) {
  |        ^^^^^^
  |
  = note: `#[warn(dead_code)]` on by default

Check warning on line 535 in alvr/server_core/src/tracking/mod.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

this `.filter_map` can be written more simply using `.map`

warning: this `.filter_map` can be written more simply using `.map`
   --> alvr/server_core/src/tracking/mod.rs:526:38
    |
526 |                   let device_motions = device_motion_keys
    |  ______________________________________^
527 | |                     .iter()
528 | |                     .filter_map(move |id| {
529 | |                         Some((
...   |
534 | |                         ))
535 | |                     })
    | |______________________^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_filter_map

Check warning on line 506 in alvr/server_core/src/tracking/mod.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

this `.filter_map` can be written more simply using `.map`

warning: this `.filter_map` can be written more simply using `.map`
   --> alvr/server_core/src/tracking/mod.rs:497:38
    |
497 |                   let device_motions = device_motion_keys
    |  ______________________________________^
498 | |                     .iter()
499 | |                     .filter_map(move |id| {
500 | |                         Some((
...   |
505 | |                         ))
506 | |                     })
    | |______________________^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_filter_map
    = note: `#[warn(clippy::unnecessary_filter_map)]` on by default

Check warning on line 67 in alvr/server_core/src/tracking/vmc.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

float has excessive precision

warning: float has excessive precision
  --> alvr/server_core/src/tracking/vmc.rs:67:48
   |
67 |             Quat::from_xyzw(0.70228, -0.08246, 0.7071100, 0.00000),
   |                                                ^^^^^^^^^
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#excessive_precision
help: consider changing the type or truncating it to
   |
67 |             Quat::from_xyzw(0.70228, -0.08246, 0.707_11, 0.00000),
   |                                                ~~~~~~~~

Check warning on line 59 in alvr/server_core/src/tracking/vmc.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

float has excessive precision

warning: float has excessive precision
  --> alvr/server_core/src/tracking/vmc.rs:59:58
   |
59 |             Quat::from_xyzw(-0.59103, 0.38818, 0.00000, -0.7071100),
   |                                                          ^^^^^^^^^
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#excessive_precision
   = note: `#[warn(clippy::excessive_precision)]` on by default
help: consider changing the type or truncating it to
   |
59 |             Quat::from_xyzw(-0.59103, 0.38818, 0.00000, -0.707_11),
   |                                                          ~~~~~~~~

Check warning on line 133 in alvr/server_core/src/bitrate.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

casting to the same type is unnecessary (`f32` -> `f32`)

warning: casting to the same type is unnecessary (`f32` -> `f32`)
   --> alvr/server_core/src/bitrate.rs:133:25
    |
133 |                         self.packet_bytes_average.get_average() as f32,
    |                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `self.packet_bytes_average.get_average()`
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_cast
    = note: `#[warn(clippy::unnecessary_cast)]` on by default

Check warning on line 158 in alvr/client_mock/src/main.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

using `clone` on type `ViewParams` which implements the `Copy` trait

warning: using `clone` on type `ViewParams` which implements the `Copy` trait
   --> alvr/client_mock/src/main.rs:158:31
    |
158 |     context.send_view_params([views_params.clone(), views_params]);
    |                               ^^^^^^^^^^^^^^^^^^^^ help: try removing the `clone` call: `views_params`
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#clone_on_copy
    = note: `#[warn(clippy::clone_on_copy)]` on by default

Check warning on line 41 in alvr/client_openxr/src/passthrough.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

transmute used without annotations

warning: transmute used without annotations
  --> alvr/client_openxr/src/passthrough.rs:41:27
   |
41 |             unsafe { mem::transmute(handle.layer()) }
   |                           ^^^^^^^^^ help: consider adding missing annotations: `transmute::<&openxr::openxr_sys::CompositionLayerPassthroughHTC, &openxr::CompositionLayerBase<'_, openxr::OpenGlEs>>`
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#missing_transmute_annotations

Check warning on line 39 in alvr/client_openxr/src/passthrough.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

transmute used without annotations

warning: transmute used without annotations
  --> alvr/client_openxr/src/passthrough.rs:39:27
   |
39 |             unsafe { mem::transmute(handle.layer()) }
   |                           ^^^^^^^^^ help: consider adding missing annotations: `transmute::<&openxr::openxr_sys::CompositionLayerPassthroughFB, &openxr::CompositionLayerBase<'_, openxr::OpenGlEs>>`
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#missing_transmute_annotations
   = note: `#[warn(clippy::missing_transmute_annotations)]` on by default

Check warning on line 28 in alvr/client_openxr/src/interaction.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

fields `aim_action` and `aim_space` are never read

warning: fields `aim_action` and `aim_space` are never read
  --> alvr/client_openxr/src/interaction.rs:28:9
   |
24 | pub struct HandInteraction {
   |            --------------- fields in this struct
...
28 |     pub aim_action: xr::Action<xr::Posef>,
   |         ^^^^^^^^^^
29 |     pub aim_space: xr::Space,
   |         ^^^^^^^^^

Check warning on line 92 in alvr/client_openxr/src/lib.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

field `system` is never read

warning: field `system` is never read
  --> alvr/client_openxr/src/lib.rs:92:5
   |
90 | pub struct XrContext {
   |            --------- field in this struct
91 |     instance: xr::Instance,
92 |     system: xr::SystemId,
   |     ^^^^^^
   |
   = note: `XrContext` has a derived impl for the trait `Clone`, but this is intentionally ignored during dead code analysis
   = note: `#[warn(dead_code)]` on by default

Check warning on line 197 in alvr/client_openxr/src/lib.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

variable does not need to be mutable

warning: variable does not need to be mutable
   --> alvr/client_openxr/src/lib.rs:197:9
    |
197 |     let mut parsed_stream_config = None::<ParsedStreamConfig>;
    |         ----^^^^^^^^^^^^^^^^^^^^
    |         |
    |         help: remove this `mut`

Check warning on line 196 in alvr/client_openxr/src/lib.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

variable does not need to be mutable

warning: variable does not need to be mutable
   --> alvr/client_openxr/src/lib.rs:196:9
    |
196 |     let mut last_lobby_message = String::new();
    |         ----^^^^^^^^^^^^^^^^^^
    |         |
    |         help: remove this `mut`
    |
    = note: `#[warn(unused_mut)]` on by default

Check warning on line 209 in alvr/client_openxr/src/lib.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

unused variable: `xr_frame_stream`

warning: unused variable: `xr_frame_stream`
   --> alvr/client_openxr/src/lib.rs:209:51
    |
209 |         let (xr_session, mut xr_frame_waiter, mut xr_frame_stream) = unsafe {
    |                                                   ^^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_xr_frame_stream`

Check warning on line 209 in alvr/client_openxr/src/lib.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

unused variable: `xr_frame_waiter`

warning: unused variable: `xr_frame_waiter`
   --> alvr/client_openxr/src/lib.rs:209:30
    |
209 |         let (xr_session, mut xr_frame_waiter, mut xr_frame_stream) = unsafe {
    |                              ^^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_xr_frame_waiter`

Check warning on line 209 in alvr/client_openxr/src/lib.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

unused variable: `xr_session`

warning: unused variable: `xr_session`
   --> alvr/client_openxr/src/lib.rs:209:14
    |
209 |         let (xr_session, mut xr_frame_waiter, mut xr_frame_stream) = unsafe {
    |              ^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_xr_session`

Check warning on line 197 in alvr/client_openxr/src/lib.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

unused variable: `parsed_stream_config`

warning: unused variable: `parsed_stream_config`
   --> alvr/client_openxr/src/lib.rs:197:13
    |
197 |     let mut parsed_stream_config = None::<ParsedStreamConfig>;
    |             ^^^^^^^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_parsed_stream_config`

Check warning on line 196 in alvr/client_openxr/src/lib.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

unused variable: `last_lobby_message`

warning: unused variable: `last_lobby_message`
   --> alvr/client_openxr/src/lib.rs:196:13
    |
196 |     let mut last_lobby_message = String::new();
    |             ^^^^^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_last_lobby_message`
    |
    = note: `#[warn(unused_variables)]` on by default

Check warning on line 215 in alvr/client_openxr/src/lib.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

unreachable definition

warning: unreachable definition
   --> alvr/client_openxr/src/lib.rs:215:13
    |
211 |                 .create_session(xr_system, &graphics::session_create_info(&graphics_context))
    |                                             ------------------------------------------------ any code following this expression is unreachable
...
215 |         let xr_context = XrContext {
    |             ^^^^^^^^^^ unreachable definition
    |
note: this expression has type `openxr::opengles::SessionCreateInfo`, which is uninhabited
   --> alvr/client_openxr/src/lib.rs:211:45
    |
211 |                 .create_session(xr_system, &graphics::session_create_info(&graphics_context))
    |                                             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    = note: `#[warn(unreachable_code)]` on by default

Check warning on line 364 in alvr/client_core/src/lib.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

returning the result of a `let` binding from a block

warning: returning the result of a `let` binding from a block
   --> alvr/client_core/src/lib.rs:364:9
    |
353 | /         let view_params = [
354 | |             ViewParams {
355 | |                 pose: head_pose * view_params[0].pose,
356 | |                 fov: view_params[0].fov,
...   |
361 | |             },
362 | |         ];
    | |__________- unnecessary `let` binding
363 |
364 |           view_params
    |           ^^^^^^^^^^^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_and_return
    = note: `#[warn(clippy::let_and_return)]` on by default
help: return the expression directly
    |
353 ~         
354 |
355 ~         [
356 +             ViewParams {
357 +                 pose: head_pose * view_params[0].pose,
358 +                 fov: view_params[0].fov,
359 +             },
360 +             ViewParams {
361 +                 pose: head_pose * view_params[1].pose,
362 +                 fov: view_params[1].fov,
363 +             },
364 +         ]
    |

Check warning on line 312 in alvr/client_core/src/lib.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

very complex type used. Consider factoring parts into `type` definitions

warning: very complex type used. Consider factoring parts into `type` definitions
   --> alvr/client_core/src/lib.rs:312:19
    |
312 |         callback: Box<dyn FnMut(Duration, &[u8]) -> bool + Send>,
    |                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#type_complexity

Check warning on line 65 in alvr/client_core/src/connection.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

very complex type used. Consider factoring parts into `type` definitions

warning: very complex type used. Consider factoring parts into `type` definitions
  --> alvr/client_core/src/connection.rs:65:27
   |
65 |     pub decoder_callback: Mutex<Option<Box<dyn FnMut(Duration, &[u8]) -> bool + Send>>>,
   |                           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   |
   = 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

Check warning on line 47 in alvr/adb/src/commands.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

redundant closure

warning: redundant closure
  --> alvr/adb/src/commands.rs:47:14
   |
47 |         .map(|s| Vec::with_capacity(s))
   |              ^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace the closure with the function itself: `Vec::with_capacity`
   |
   = 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

Check warning on line 128 in alvr/adb/src/parse.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

fields `remote` and `serial` are never read

warning: fields `remote` and `serial` are never read
   --> alvr/adb/src/parse.rs:128:9
    |
126 | pub struct ForwardedPorts {
    |            -------------- fields in this struct
127 |     pub local: u16,
128 |     pub remote: u16,
    |         ^^^^^^
129 |     pub serial: String,
    |         ^^^^^^
    |
    = note: `ForwardedPorts` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis

Check warning on line 75 in alvr/adb/src/parse.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

fields `connection_state`, `device`, `model`, `product`, and `transport_type` are never read

warning: fields `connection_state`, `device`, `model`, `product`, and `transport_type` are never read
  --> alvr/adb/src/parse.rs:75:9
   |
74 | pub struct Device {
   |            ------ fields in this struct
75 |     pub connection_state: Option<ConnectionState>,
   |         ^^^^^^^^^^^^^^^^
76 |     pub device: Option<String>,
   |         ^^^^^^
77 |     pub model: Option<String>,
   |         ^^^^^
78 |     pub product: Option<String>,
   |         ^^^^^^^
79 |     pub serial: Option<String>,
80 |     pub transport_type: Option<TransportType>,
   |         ^^^^^^^^^^^^^^
   |
   = note: `Device` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis