warning: this function has too many arguments (9/7)
   --> pilota-build/src/lib.rs:242:5
    |
242 | /     pub fn build_cx(
243 | |         services: Vec<IdlService>,
244 | |         out: Option<Output>,
245 | |         mut parser: P,
...   |
251 | |         dedups: Vec<FastStr>,
252 | |     ) -> Context {
    | |________________^
    |
    = 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

warning: you seem to be trying to use `match` for destructuring a single pattern. Consider using `if let`
   --> pilota-build/src/plugin/mod.rs:129:13
    |
129 | /             match &node.kind {
130 | |                 NodeKind::Item(item) => p.on_item(cx, *def_id, item.clone()),
131 | |                 _ => {}
132 | |             }
    | |_____________^ help: try: `if let NodeKind::Item(item) = &node.kind { p.on_item(cx, *def_id, item.clone()) }`
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_match

warning: you seem to be trying to use `match` for destructuring a single pattern. Consider using `if let`
  --> pilota-build/src/plugin/workspace.rs:34:9
   |
34 | /         match &*item {
35 | |             Item::Service(s) => {
36 | |                 if let Some(loc) = cx.location_map.get(&def_id) {
37 | |                     if let Some(mut gen) = cx.plugin_gen.get_mut(loc) {
...  |
42 | |             _ => {}
43 | |         }
   | |_________^
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_match
help: try
   |
34 ~         if let Item::Service(s) = &*item {
35 +             if let Some(loc) = cx.location_map.get(&def_id) {
36 +                 if let Some(mut gen) = cx.plugin_gen.get_mut(loc) {
37 +                     gen.push_str(&format!("pub struct {};", s.name.sym));
38 +                 }
39 +             };
40 +         }
   |

warning: you seem to be trying to use `match` for destructuring a single pattern. Consider using `if let`
  --> pilota-build/src/plugin/workspace.rs:19:21
   |
19 | /                     match &node.kind {
20 | |                         NodeKind::Item(item) => self.on_item(cx, *def_id, item.clone()),
21 | |                         _ => {}
22 | |                     }
   | |_____________________^ help: try: `if let NodeKind::Item(item) = &node.kind { self.on_item(cx, *def_id, item.clone()) }`
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_match

warning: accessing first element with `status.r#mod.get(0)`
   --> pilota-build/src/resolve.rs:458:31
    |
458 |             Namespace::Mod => status.r#mod.get(0),
    |                               ^^^^^^^^^^^^^^^^^^^ help: try: `status.r#mod.first()`
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#get_first

warning: accessing first element with `status.ty.get(0)`
   --> pilota-build/src/resolve.rs:457:30
    |
457 |             Namespace::Ty => status.ty.get(0),
    |                              ^^^^^^^^^^^^^^^^ help: try: `status.ty.first()`
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#get_first

warning: accessing first element with `status.value.get(0)`
   --> pilota-build/src/resolve.rs:456:33
    |
456 |             Namespace::Value => status.value.get(0),
    |                                 ^^^^^^^^^^^^^^^^^^^ help: try: `status.value.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

warning: you seem to be trying to use `match` for destructuring a single pattern. Consider using `if let`
   --> pilota-build/src/resolve.rs:167:13
    |
167 | /             match &item.kind {
168 | |                 ir::ItemKind::Enum(e) => e.variants.iter().for_each(|e| {
169 | |                     self.def_sym(Namespace::Value, (*e.name).clone());
170 | |                 }),
171 | |                 _ => {}
172 | |             }
    | |_____________^
    |
    = 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
    |
167 ~             if let ir::ItemKind::Enum(e) = &item.kind { e.variants.iter().for_each(|e| {
168 +                 self.def_sym(Namespace::Value, (*e.name).clone());
169 +             }) }
    |

warning: parameter is only used in recursion
  --> pilota-build/src/parser/protobuf/mod.rs:66:9
   |
66 |         message_name: Option<&str>,
   |         ^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_message_name`
   |
note: parameter used here
  --> pilota-build/src/parser/protobuf/mod.rs:81:33
   |
81 | ...                   message_name,
   |                       ^^^^^^^^^^^^
...
87 | ...                   message_name,
   |                       ^^^^^^^^^^^^
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#only_used_in_recursion
   = note: `#[warn(clippy::only_used_in_recursion)]` on by default

warning: this expression creates a reference which is immediately dereferenced by the compiler
   --> pilota-build/src/middle/context.rs:226:62
    |
226 |                         PathCollector { cx, set }.visit_path(&p);
    |                                                              ^^ help: change this to: `p`
    |
    = 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

warning: this lifetime isn't used in the function definition
   --> pilota-build/src/codegen/mod.rs:461:24
    |
461 |     pub fn write_items<'a>(&self, stream: &mut String, items: impl Iterator<Item = CodegenItem>)
    |                        ^^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#extra_unused_lifetimes
    = note: `#[warn(clippy::extra_unused_lifetimes)]` on by default

warning: useless use of `format!`
   --> pilota-build/src/codegen/thrift/mod.rs:716:33
    |
716 | / ...                   format!(
717 | | ...                       r#"Err(::pilota::thrift::DecodeError::new(
718 | | ...                           ::pilota::thrift::DecodeErrorKind::InvalidData,
719 | | ...                           "received empty union from remote Message")
720 | | ...                       )"#
721 | | ...                   )
    | |_______________________^
    |
    = 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
help: consider using `.to_string()`
    |
716 ~                                 r#"Err(::pilota::thrift::DecodeError::new(
717 +                                         ::pilota::thrift::DecodeErrorKind::InvalidData,
718 +                                         "received empty union from remote Message")
719 +                                     )"#.to_string()
    |

warning: writing `&mut String` instead of `&mut str` involves a new object where a slice will do
   --> pilota-build/src/codegen/thrift/mod.rs:350:59
    |
350 |     fn codegen_entry_enum(&self, _def_id: DefId, _stream: &mut String, _e: &rir::Enum) {
    |                                                           ^^^^^^^^^^^ help: change this to: `&mut 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

warning: unneeded `return` statement
    --> pilota/src/thrift/compact.rs:1648:18
     |
1648 |         unsafe { return Ok(FastStr::from_bytes_unchecked(bytes)) }
     |                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
     |
     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
help: remove `return`
     |
1648 |         unsafe { Ok(FastStr::from_bytes_unchecked(bytes)) }
     |                  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

warning: question mark operator is useless here
    --> pilota/src/thrift/compact.rs:1123:9
     |
1123 |         Ok(self.read_varint_async::<i64>().await?)
     |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try removing question mark and `Ok()`: `self.read_varint_async::<i64>().await`
     |
     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_question_mark

warning: redundant closure
   --> pilota-build/src/codegen/workspace.rs:120:18
    |
120 |             .map(|s| FastStr::new(s))
    |                  ^^^^^^^^^^^^^^^^^^^ help: replace the closure with the function itself: `FastStr::new`
    |
    = 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

warning: question mark operator is useless here
    --> pilota/src/thrift/compact.rs:1118:9
     |
1118 |         Ok(self.read_varint_async::<i32>().await?)
     |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try removing question mark and `Ok()`: `self.read_varint_async::<i32>().await`
     |
     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_question_mark

warning: question mark operator is useless here
    --> pilota/src/thrift/compact.rs:1113:9
     |
1113 |         Ok(self.read_varint_async::<i16>().await?)
     |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try removing question mark and `Ok()`: `self.read_varint_async::<i16>().await`
     |
     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_question_mark
     = note: `#[warn(clippy::needless_question_mark)]` on by default

warning: use Vec::sort_by_key here instead
 --> pilota-build/src/codegen/toml.rs:9:13
  |
9 |             a.sort_by(|a, b| a.to_string().cmp(&b.to_string()));
  |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `a.sort_by_key(|a| a.to_string())`
  |
  = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_sort_by
  = note: `#[warn(clippy::unnecessary_sort_by)]` on by default

warning: explicit call to `.into_iter()` in function argument accepting `IntoIterator`
  --> pilota-build/src/codegen/pkg_tree.rs:32:20
   |
32 |             .chain(Some(k).into_iter())
   |                    ^^^^^^^^^^^^^^^^^^^ help: consider removing the `.into_iter()`: `Some(k)`
   |
note: this parameter accepts any `IntoIterator`, so you don't need to call `.into_iter()`
  --> /rustc/9fad6859925b34f9a0b1af592a7839ccb1e71f60/library/core/src/iter/traits/iterator.rs:524:12
   = 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

warning: this expression creates a reference which is immediately dereferenced by the compiler
    --> pilota/src/thrift/binary_unsafe.rs:1169:9
     |
1169 |         &mut self.trans
     |         ^^^^^^^^^^^^^^^ help: change this to: `self.trans`
     |
     = 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

warning: unneeded `return` statement
    --> pilota/src/thrift/binary_unsafe.rs:1107:13
     |
1107 |             return Ok(FastStr::from_bytes_unchecked(bytes));
     |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
     |
     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
help: remove `return`
     |
1107 -             return Ok(FastStr::from_bytes_unchecked(bytes));
1107 +             Ok(FastStr::from_bytes_unchecked(bytes))
     |

warning: use of `offset` with a `usize` casted to an `isize`
   --> pilota/src/thrift/binary_unsafe.rs:738:17
    |
738 |                 self.buf.as_mut_ptr().offset(self.index as isize),
    |                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `self.buf.as_mut_ptr().add(self.index)`
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#ptr_offset_with_cast

warning: use of `offset` with a `usize` casted to an `isize`
   --> pilota/src/thrift/binary_unsafe.rs:683:17
    |
683 |                 self.buf.as_mut_ptr().offset(self.index as isize),
    |                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `self.buf.as_mut_ptr().add(self.index)`
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#ptr_offset_with_cast

warning: use of `offset` with a `usize` casted to an `isize`
   --> pilota/src/thrift/binary_unsafe.rs:674:21
    |
674 |                     self.trans.bytes_mut().as_mut_ptr().offset(l as isize),
    |                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `self.trans.bytes_mut().as_mut_ptr().add(l)`
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#ptr_offset_with_cast