Skip to content

Bump rustix from 0.37.23 to 0.37.25 in /examples (#82) #171

Bump rustix from 0.37.23 to 0.37.25 in /examples (#82)

Bump rustix from 0.37.23 to 0.37.25 in /examples (#82) #171

GitHub Actions / clippy succeeded Oct 19, 2023 in 0s

clippy

29 warnings

Details

Results

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

Versions

  • rustc 1.73.0 (cc66ad468 2023-10-03)
  • cargo 1.73.0 (9c4383fb5 2023-08-26)
  • clippy 0.1.73 (cc66ad4 2023-10-03)

Annotations

Check warning on line 277 in src/dialdbg/main.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

redundant pattern matching, consider using `is_ok()`

warning: redundant pattern matching, consider using `is_ok()`
   --> src/dialdbg/main.rs:277:16
    |
277 |         if let Ok(_) = log_path.try_exists() {
    |         -------^^^^^------------------------ help: try: `if log_path.try_exists().is_ok()`
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_pattern_matching

Check warning on line 223 in src/dialdbg/main.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

redundant pattern matching, consider using `is_ok()`

warning: redundant pattern matching, consider using `is_ok()`
   --> src/dialdbg/main.rs:223:16
    |
223 |         if let Ok(_) = log_path.try_exists() {
    |         -------^^^^^------------------------ help: try: `if log_path.try_exists().is_ok()`
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_pattern_matching
    = note: `#[warn(clippy::redundant_pattern_matching)]` on by default

Check warning on line 131 in src/dialdbg/main.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

length comparison to zero

warning: length comparison to zero
   --> src/dialdbg/main.rs:131:8
    |
131 |     if responses.len() == 0 {
    |        ^^^^^^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `responses.is_empty()`
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero

Check warning on line 13 in src/dialdbg/stats.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

you seem to want to iterate on a map's values

warning: you seem to want to iterate on a map's values
  --> src/dialdbg/stats.rs:13:27
   |
13 |         for (_, value) in &self.0.reports {
   |                           ^^^^^^^^^^^^^^^
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#for_kv_map
   = note: `#[warn(clippy::for_kv_map)]` on by default
help: use the corresponding method
   |
13 |         for value in self.0.reports.values() {
   |             ~~~~~    ~~~~~~~~~~~~~~~~~~~~~~~

Check warning on line 159 in src/dialdbg/parse.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

length comparison to zero

warning: length comparison to zero
   --> src/dialdbg/parse.rs:159:8
    |
159 |     if split_log.len() == 0 {
    |        ^^^^^^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `split_log.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

Check warning on line 10 in src/dialdbg/parse.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

constants have by default a `'static` lifetime

warning: constants have by default a `'static` lifetime
  --> src/dialdbg/parse.rs:10:38
   |
10 | pub(crate) const DIAL_ERROR_PREFIX: &'static str = "unexpected dial connect error";
   |                                     -^^^^^^^---- help: consider removing `'static`: `&str`
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_static_lifetimes
   = note: `#[warn(clippy::redundant_static_lifetimes)]` on by default

Check warning on line 353 in src/rpc/webrtc.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

you seem to be trying to use `match` for destructuring a single pattern. Consider using `if let`

warning: you seem to be trying to use `match` for destructuring a single pattern. Consider using `if let`
   --> src/rpc/webrtc.rs:346:5
    |
346 | /     match proto.status {
347 | |         Some(ref status) => {
348 | |             let key = HeaderName::from_str("Grpc-Message").unwrap();
349 | |             let val = HeaderValue::from_str(&status.message).unwrap();
...   |
352 | |         None => (),
353 | |     }
    | |_____^
    |
    = 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
    |
346 ~     if let Some(ref status) = proto.status {
347 +         let key = HeaderName::from_str("Grpc-Message").unwrap();
348 +         let val = HeaderValue::from_str(&status.message).unwrap();
349 +         trailers.insert(key, val);
350 +     }
    |

Check warning on line 608 in src/rpc/dial.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

using `clone` on type `Option<&Authority>` which implements the `Copy` trait

warning: using `clone` on type `Option<&Authority>` which implements the `Copy` trait
   --> src/rpc/dial.rs:608:22
    |
608 |         let domain = original_uri.authority().clone().unwrap().to_string();
    |                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try removing the `clone` call: `original_uri.authority()`
    |
    = 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 523 in src/rpc/dial.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

this expression creates a reference which is immediately dereferenced by the compiler

warning: this expression creates a reference which is immediately dereferenced by the compiler
   --> src/rpc/dial.rs:523:67
    |
523 |                 Self::create_channel(self.config.allow_downgrade, &domain, uri.clone(), false)
    |                                                                   ^^^^^^^ help: change this to: `domain`
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow

Check warning on line 505 in src/rpc/dial.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

this expression creates a reference which is immediately dereferenced by the compiler

warning: this expression creates a reference which is immediately dereferenced by the compiler
   --> src/rpc/dial.rs:505:67
    |
505 |                 Self::create_channel(self.config.allow_downgrade, &domain, uri, true).await
    |                                                                   ^^^^^^^ help: change this to: `domain`
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow

Check warning on line 408 in src/rpc/dial.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

use of `or_insert` to construct default value

warning: use of `or_insert` to construct default value
   --> src/rpc/dial.rs:408:30
    |
408 |                 map.entry(k).or_insert(vec![]).push(v);
    |                              ^^^^^^^^^^^^^^^^^ help: try: `or_default()`
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unwrap_or_default
    = note: `#[warn(clippy::unwrap_or_default)]` on by default

Check warning on line 114 in src/rpc/client_stream.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

passing a unit value to a function

warning: passing a unit value to a function
   --> src/rpc/client_stream.rs:114:47
    |
114 |             Some(Type::Trailers(trailers)) => Ok(self.process_trailers(trailers.to_owned()).await),
    |                                               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unit_arg
    = note: `#[warn(clippy::unit_arg)]` on by default
help: move the expression in front of the call and replace it with the unit literal `()`
    |
114 ~             Some(Type::Trailers(trailers)) => {
115 +                 self.process_trailers(trailers.to_owned()).await;
116 +                 Ok(())
117 ~             },
    |

Check warning on line 27 in src/rpc/client_stream.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

called `unwrap` on `data` after checking its variant with `is_some`

warning: called `unwrap` on `data` after checking its variant with `is_some`
  --> src/rpc/client_stream.rs:27:40
   |
26 |                     if data.is_some() {
   |                     ----------------- help: try: `if let Some(..) = data`
27 |                         let mut data = data.unwrap();
   |                                        ^^^^^^^^^^^^^
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_unwrap
   = note: `#[warn(clippy::unnecessary_unwrap)]` on by default

Check warning on line 275 in src/ffi/spatialmath/quaternion.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

this expression creates a reference which is immediately dereferenced by the compiler

warning: this expression creates a reference which is immediately dereferenced by the compiler
   --> src/ffi/spatialmath/quaternion.rs:275:20
    |
275 |     to_raw_pointer(&quat)
    |                    ^^^^^ help: change this to: `quat`
    |
    = 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

Check warning on line 265 in src/ffi/dial_ffi.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

non-binding `let` on a future

warning: non-binding `let` on a future
   --> src/ffi/dial_ffi.rs:263:5
    |
263 | /     let _ = runtime.spawn(async {
264 | |         let _ = server.await;
265 | |     });
    | |_______^
    |
    = help: consider awaiting the future or dropping explicitly with `std::mem::drop`
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_underscore_future
    = note: `#[warn(clippy::let_underscore_future)]` on by default

Check warning on line 229 in src/ffi/dial_ffi.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
   --> src/ffi/dial_ffi.rs:229:17
    |
226 | /                 let res = timeout(
227 | |                     timeout_duration, 
228 | |                     dial_without_cred(uri_str, allow_insec, disable_webrtc)?.connect()).await?;
    | |_______________________________________________________________________________________________- unnecessary `let` binding
229 |                   res
    |                   ^^^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_and_return
help: return the expression directly
    |
226 ~                 
227 ~                 timeout(
228 +                     timeout_duration, 
229 +                     dial_without_cred(uri_str, allow_insec, disable_webrtc)?.connect()).await?
    |

Check warning on line 223 in src/ffi/dial_ffi.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
   --> src/ffi/dial_ffi.rs:223:17
    |
210 | /                 let res = timeout(
211 | |                     timeout_duration, 
212 | |                     dial_with_cred(
213 | |                     uri_str,
...   |
221 | |                 )
222 | |                 .await?;
    | |________________________- unnecessary `let` binding
223 |                   res
    |                   ^^^
    |
    = 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
    |
210 ~                 
211 ~                 timeout(
212 +                     timeout_duration, 
213 +                     dial_with_cred(
214 +                     uri_str,
215 +                     entity_opt,
216 +                     t.to_str()?,
217 +                     p.to_str()?,
218 +                     allow_insec,
219 +                     disable_webrtc,
220 +                 )?
221 +                 .connect()
222 +                 )
223 +                 .await?
    |

Check warning on line 22 in src/rpc/log_prefixes.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

constants have by default a `'static` lifetime

warning: constants have by default a `'static` lifetime
  --> src/rpc/log_prefixes.rs:22:34
   |
22 | pub const ICE_CONNECTED_EXTERN: &'static str = "ICE connection state changed: connected";
   |                                 -^^^^^^^---- help: consider removing `'static`: `&str`
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_static_lifetimes

Check warning on line 18 in src/rpc/log_prefixes.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

constants have by default a `'static` lifetime

warning: constants have by default a `'static` lifetime
  --> src/rpc/log_prefixes.rs:18:32
   |
18 | pub const CANDIDATE_SELECTED: &'static str = "Selected candidate pair";
   |                               -^^^^^^^---- help: consider removing `'static`: `&str`
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_static_lifetimes

Check warning on line 16 in src/rpc/log_prefixes.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

constants have by default a `'static` lifetime

warning: constants have by default a `'static` lifetime
  --> src/rpc/log_prefixes.rs:16:27
   |
16 | pub const DIALED_WEBRTC: &'static str = "Connected via WebRTC";
   |                          -^^^^^^^---- help: consider removing `'static`: `&str`
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_static_lifetimes

Check warning on line 15 in src/rpc/log_prefixes.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

constants have by default a `'static` lifetime

warning: constants have by default a `'static` lifetime
  --> src/rpc/log_prefixes.rs:15:25
   |
15 | pub const DIALED_GRPC: &'static str = "Connected via gRPC";
   |                        -^^^^^^^---- help: consider removing `'static`: `&str`
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_static_lifetimes

Check warning on line 14 in src/rpc/log_prefixes.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

constants have by default a `'static` lifetime

warning: constants have by default a `'static` lifetime
  --> src/rpc/log_prefixes.rs:14:26
   |
14 | pub const DIAL_ATTEMPT: &'static str = "Dialing";
   |                         -^^^^^^^---- help: consider removing `'static`: `&str`
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_static_lifetimes

Check warning on line 12 in src/rpc/log_prefixes.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

constants have by default a `'static` lifetime

warning: constants have by default a `'static` lifetime
  --> src/rpc/log_prefixes.rs:12:43
   |
12 | pub const END_LOCAL_SESSION_DESCRIPTION: &'static str = "End local session description";
   |                                          -^^^^^^^---- help: consider removing `'static`: `&str`
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_static_lifetimes

Check warning on line 11 in src/rpc/log_prefixes.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

constants have by default a `'static` lifetime

warning: constants have by default a `'static` lifetime
  --> src/rpc/log_prefixes.rs:11:45
   |
11 | pub const START_LOCAL_SESSION_DESCRIPTION: &'static str = "Start local session description";
   |                                            -^^^^^^^---- help: consider removing `'static`: `&str`
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_static_lifetimes

Check warning on line 9 in src/rpc/log_prefixes.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

constants have by default a `'static` lifetime

warning: constants have by default a `'static` lifetime
 --> src/rpc/log_prefixes.rs:9:33
  |
9 | pub const ACQUIRED_AUTH_TOKEN: &'static str = "Acquired auth token";
  |                                -^^^^^^^---- help: consider removing `'static`: `&str`
  |
  = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_static_lifetimes