refactor(client_core): ♻️ Port FFE to Rust and remove all C++ code #4480
clippy
31 warnings
Details
Results
Message level | Amount |
---|---|
Internal compiler error | 0 |
Error | 0 |
Warning | 31 |
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
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 158 in alvr/client_mock/src/main.rs
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
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
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
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
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
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
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
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
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
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
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
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
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 535 in alvr/server_core/src/tracking/mod.rs
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
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
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
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
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 364 in alvr/client_core/src/lib.rs
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
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
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 839 in alvr/client_core/src/c_api.rs
github-actions / clippy
variants `Float`, `Int32`, `Int64`, and `String` are never constructed
warning: variants `Float`, `Int32`, `Int64`, and `String` are never constructed
--> alvr/client_core/src/c_api.rs:839:5
|
838 | pub enum AlvrMediacodecPropType {
| ---------------------- variants in this enum
839 | Float,
| ^^^^^
840 | Int32,
| ^^^^^
841 | Int64,
| ^^^^^
842 | String,
| ^^^^^^
Check warning on line 78 in alvr/client_core/src/c_api.rs
github-actions / clippy
field `codec` is never read
warning: field `codec` is never read
--> alvr/client_core/src/c_api.rs:78:9
|
77 | DecoderConfig {
| ------------- field in this variant
78 | codec: AlvrCodec,
| ^^^^^
Check warning on line 71 in alvr/client_core/src/c_api.rs
github-actions / clippy
fields `device_id`, `duration_s`, `frequency`, and `amplitude` are never read
warning: fields `device_id`, `duration_s`, `frequency`, and `amplitude` are never read
--> alvr/client_core/src/c_api.rs:71:9
|
70 | Haptics {
| ------- fields in this variant
71 | device_id: u64,
| ^^^^^^^^^
72 | duration_s: f32,
| ^^^^^^^^^^
73 | frequency: f32,
| ^^^^^^^^^
74 | amplitude: f32,
| ^^^^^^^^^