Skip to content

panic unwinding

panic unwinding #1130

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

clippy

19 warnings

Details

Results

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

Versions

  • rustc 1.75.0-nightly (e20cb7702 2023-10-12)
  • cargo 1.75.0-nightly (6fa6fdc76 2023-10-10)
  • clippy 0.1.75 (e20cb77 2023-10-12)

Annotations

Check warning on line 94 in sse/src/lib.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

use of `default` to create a unit struct

warning: use of `default` to create a unit struct
  --> sse/src/lib.rs:94:31
   |
94 |             event: PhantomData::default(),
   |                               ^^^^^^^^^^^ help: remove this call to `default`
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#default_constructed_unit_structs
   = note: `#[warn(clippy::default_constructed_unit_structs)]` on by default

Check warning on line 19 in panic-boundary/src/lib.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

deref which would be done by auto-deref

warning: deref which would be done by auto-deref
  --> panic-boundary/src/lib.rs:19:9
   |
19 |         &*self.0
   |         ^^^^^^^^ help: try: `&self.0`
   |
   = 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

Check warning on line 209 in proxy/src/lib.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

explicit call to `.into_iter()` in function argument accepting `IntoIterator`

warning: explicit call to `.into_iter()` in function argument accepting `IntoIterator`
   --> proxy/src/lib.rs:209:29
    |
209 |                     .extend(std::mem::take(client_conn.response_headers_mut()).into_iter());
    |                             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider removing the `.into_iter()`: `std::mem::take(client_conn.response_headers_mut())`
    |
note: this parameter accepts any `IntoIterator`, so you don't need to call `.into_iter()`
   --> /rustc/e20cb7702117f1ad8127a16406ba9edd230c4f65/library/core/src/iter/traits/collect.rs:372:18
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_conversion
    = note: `#[warn(clippy::useless_conversion)]` on by default

Check warning on line 451 in client/src/conn.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

the borrowed expression implements the required traits

warning: the borrowed expression implements the required traits
   --> client/src/conn.rs:451:13
    |
451 |             &mut self.response_body_state,
    |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: change this to: `self.response_body_state`
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
    = note: `#[warn(clippy::needless_borrows_for_generic_args)]` on by default

Check warning on line 273 in trillium/src/conn.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

item in documentation is missing backticks

warning: item in documentation is missing backticks
   --> trillium/src/conn.rs:273:16
    |
273 |     Returns a [ReceivedBody] that references this `Conn`. The `Conn`
    |                ^^^^^^^^^^^^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#doc_markdown
note: the lint level is defined here
   --> trillium/src/lib.rs:10:23
    |
10  | #![warn(missing_docs, clippy::pedantic, clippy::nursery, clippy::cargo)]
    |                       ^^^^^^^^^^^^^^^^
    = note: `#[warn(clippy::doc_markdown)]` implied by `#[warn(clippy::pedantic)]`
help: try
    |
273 |     Returns a [`ReceivedBody`] that references this `Conn`. The `Conn`
    |                ~~~~~~~~~~~~~~

Check warning on line 54 in http/src/transport/boxed_transport.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

item in documentation is missing backticks

warning: item in documentation is missing backticks
  --> http/src/transport/boxed_transport.rs:54:18
   |
54 |     Create a new BoxedTransport from some Transport.
   |                  ^^^^^^^^^^^^^^
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#doc_markdown
help: try
   |
54 |     Create a new `BoxedTransport` from some Transport.
   |                  ~~~~~~~~~~~~~~~~

Check warning on line 69 in http/src/headers/header_values.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

`IntoIterator` implemented for a reference type without an `iter` method

warning: `IntoIterator` implemented for a reference type without an `iter` method
  --> http/src/headers/header_values.rs:61:1
   |
61 | / impl<'a> IntoIterator for &'a HeaderValues {
62 | |     type Item = &'a HeaderValue;
63 | |
64 | |     type IntoIter = std::slice::Iter<'a, HeaderValue>;
...  |
68 | |     }
69 | | }
   | |_^
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#into_iter_without_iter
   = note: `#[warn(clippy::into_iter_without_iter)]` implied by `#[warn(clippy::pedantic)]`
help: consider implementing `iter`
   |
61 + 
62 + impl HeaderValues {
63 +     fn iter(&self) -> std::slice::Iter<'a, HeaderValue> {
64 +         <&Self as IntoIterator>::into_iter(self)
65 +     }
66 + }
   |

Check warning on line 93 in http/src/headers/header_name.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

matching over `()` is more explicit

warning: matching over `()` is more explicit
  --> http/src/headers/header_name.rs:93:21
   |
93 |                 Err(_) => UnknownHeader(UnknownHeaderName(SmartCow::Owned(SmartString::from(s)))),
   |                     ^ help: use `()` instead of `_`: `()`
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#ignored_unit_patterns

Check warning on line 78 in http/src/state_set.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

docs for function which may panic missing `# Panics` section

warning: docs for function which may panic missing `# Panics` section
  --> http/src/state_set.rs:75:5
   |
75 | /     pub fn get_or_insert_with<F, T>(&mut self, default: F) -> &mut T
76 | |     where
77 | |         F: FnOnce() -> T,
78 | |         T: Send + Sync + 'static,
   | |_________________________________^
   |
note: first possible panic found here
  --> http/src/state_set.rs:80:9
   |
80 | /         self.0
81 | |             .entry(TypeId::of::<T>())
82 | |             .or_insert_with(|| Box::new(default()))
83 | |             .downcast_mut()
84 | |             .expect("StateSet maintains the invariant the value associated with a given TypeId is always the type associated with that TypeId.")
   | |________________________________________________________________________________________________________________________________________________^
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#missing_panics_doc

Check warning on line 65 in http/src/state_set.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

docs for function which may panic missing `# Panics` section

warning: docs for function which may panic missing `# Panics` section
  --> http/src/state_set.rs:65:5
   |
65 |     pub fn get_or_insert<T: Send + Sync + 'static>(&mut self, default: T) -> &mut T {
   |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   |
note: first possible panic found here
  --> http/src/state_set.rs:66:9
   |
66 | /         self.0
67 | |             .entry(TypeId::of::<T>())
68 | |             .or_insert_with(|| Box::new(default))
69 | |             .downcast_mut()
70 | |             .expect("StateSet maintains the invariant the value associated with a given TypeId is always the type associated with that TypeId.")
   | |________________________________________________________________________________________________________________________________________________^
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#missing_panics_doc
   = note: `#[warn(clippy::missing_panics_doc)]` implied by `#[warn(clippy::pedantic)]`

Check warning on line 111 in http/src/upgrade.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

manual `Debug` impl does not include all fields

warning: manual `Debug` impl does not include all fields
   --> http/src/upgrade.rs:99:1
    |
99  | / impl<Transport> Debug for Upgrade<Transport> {
100 | |     fn fmt(&self, f: &mut Formatter<'_>) -> fmt::Result {
101 | |         f.debug_struct(&format!("Upgrade<{}>", std::any::type_name::<Transport>()))
102 | |             .field("request_headers", &self.request_headers)
...   |
110 | |     }
111 | | }
    | |_^
    |
note: this field is unused
   --> http/src/upgrade.rs:32:5
    |
32  |     pub state: StateSet,
    |     ^^^^^^^^^^^^^^^^^^^
note: this field is unused
   --> http/src/upgrade.rs:35:5
    |
35  |     pub transport: Transport,
    |     ^^^^^^^^^^^^^^^^^^^^^^^^
note: this field is unused
   --> http/src/upgrade.rs:43:5
    |
43  |     pub stopper: Stopper,
    |     ^^^^^^^^^^^^^^^^^^^^
    = help: consider including all fields in this `Debug` impl
    = help: consider calling `.finish_non_exhaustive()` if you intend to ignore fields
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#missing_fields_in_debug
    = note: `#[warn(clippy::missing_fields_in_debug)]` implied by `#[warn(clippy::pedantic)]`

Check warning on line 93 in http/src/synthetic.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

matching over `()` is more explicit

warning: matching over `()` is more explicit
  --> http/src/synthetic.rs:93:13
   |
93 |     fn from(_: ()) -> Self {
   |             ^ help: use `()` instead of `_`: `()`
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#ignored_unit_patterns
   = note: `#[warn(clippy::ignored_unit_patterns)]` implied by `#[warn(clippy::pedantic)]`

Check warning on line 810 in http/src/conn.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

it is more concise to loop over references to containers instead of using explicit iteration methods

warning: it is more concise to loop over references to containers instead of using explicit iteration methods
   --> http/src/conn.rs:810:33
    |
810 |         for (header, values) in self.response_headers.iter() {
    |                                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: to write this more concisely, try: `&self.response_headers`
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#explicit_iter_loop
    = note: `#[warn(clippy::explicit_iter_loop)]` implied by `#[warn(clippy::pedantic)]`

Check warning on line 637 in http/src/conn.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

item in documentation is missing backticks

warning: item in documentation is missing backticks
   --> http/src/conn.rs:637:22
    |
637 |     processing in an after_send hook, please open an issue. This hook
    |                      ^^^^^^^^^^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#doc_markdown
help: try
    |
637 |     processing in an `after_send` hook, please open an issue. This hook
    |                      ~~~~~~~~~~~~

Check warning on line 435 in http/src/conn.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

item in documentation is missing backticks

warning: item in documentation is missing backticks
   --> http/src/conn.rs:435:5
    |
435 |     ReceivedBody provides an interface to read body content
    |     ^^^^^^^^^^^^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#doc_markdown
help: try
    |
435 |     `ReceivedBody` provides an interface to read body content
    |     ~~~~~~~~~~~~~~

Check warning on line 433 in http/src/conn.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

item in documentation is missing backticks

warning: item in documentation is missing backticks
   --> http/src/conn.rs:433:16
    |
433 |     returns a [ReceivedBody] that references this conn. the conn
    |                ^^^^^^^^^^^^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#doc_markdown
help: try
    |
433 |     returns a [`ReceivedBody`] that references this conn. the conn
    |                ~~~~~~~~~~~~~~

Check warning on line 417 in http/src/conn.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

item in documentation is missing backticks

warning: item in documentation is missing backticks
   --> http/src/conn.rs:417:18
    |
417 |     returns the [encoding_rs::Encoding] for this response, as
    |                  ^^^^^^^^^^^^^^^^^^^^^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#doc_markdown
help: try
    |
417 |     returns the [`encoding_rs::Encoding`] for this response, as
    |                  ~~~~~~~~~~~~~~~~~~~~~~~

Check warning on line 401 in http/src/conn.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

item in documentation is missing backticks

warning: item in documentation is missing backticks
   --> http/src/conn.rs:401:18
    |
401 |     returns the [encoding_rs::Encoding] for this request, as
    |                  ^^^^^^^^^^^^^^^^^^^^^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#doc_markdown
note: the lint level is defined here
   --> http/src/lib.rs:10:23
    |
10  | #![warn(missing_docs, clippy::pedantic, clippy::perf, clippy::cargo)]
    |                       ^^^^^^^^^^^^^^^^
    = note: `#[warn(clippy::doc_markdown)]` implied by `#[warn(clippy::pedantic)]`
help: try
    |
401 |     returns the [`encoding_rs::Encoding`] for this request, as
    |                  ~~~~~~~~~~~~~~~~~~~~~~~

Check warning on line 394 in http/src/conn.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

the borrowed expression implements the required traits

warning: the borrowed expression implements the required traits
   --> http/src/conn.rs:394:13
    |
394 |             &mut self.request_body_state,
    |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: change this to: `self.request_body_state`
    |
    = 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