feat: TON LiteServer ANDL TCP Client #647
clippy
33 warnings
Details
Results
Message level | Amount |
---|---|
Internal compiler error | 0 |
Error | 0 |
Warning | 33 |
Note | 0 |
Help | 0 |
Versions
- rustc 1.77.2 (25ef9e3d8 2024-04-09)
- cargo 1.77.2 (e52e36006 2024-03-26)
- clippy 0.1.77 (25ef9e3 2024-04-09)
Annotations
Check warning on line 22 in ton-grpc/src/message.rs
github-actions / clippy
in a `match` scrutinee, avoid complex blocks or closures with blocks; instead, move the block or closure higher and bind it with a `let`
warning: in a `match` scrutinee, avoid complex blocks or closures with blocks; instead, move the block or closure higher and bind it with a `let`
--> ton-grpc/src/message.rs:15:107
|
15 | async fn send_message(&self, request: Request<SendRequest>) -> Result<Response<SendResponse>, Status> {
| ___________________________________________________________________________________________________________^
16 | | let msg = request.into_inner();
17 | |
18 | | let hash = self.client.send_message_returning_hash(&msg.body).await
... |
21 | | Ok(Response::new(SendResponse { hash }))
22 | | }
| |_____^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#blocks_in_conditions
Check warning on line 90 in ton-grpc/src/block.rs
github-actions / clippy
in a `match` scrutinee, avoid complex blocks or closures with blocks; instead, move the block or closure higher and bind it with a `let`
warning: in a `match` scrutinee, avoid complex blocks or closures with blocks; instead, move the block or closure higher and bind it with a `let`
--> ton-grpc/src/block.rs:79:131
|
79 | async fn get_account_addresses(&self, request: Request<BlockId>) -> Result<Response<Self::GetAccountAddressesStream>, Status> {
| ___________________________________________________________________________________________________________________________________^
80 | | let msg = request.into_inner();
81 | | let block_id = extend_block_id(&self.client, &msg).await
82 | | .map_err(|e: anyhow::Error| Status::internal(e.to_string()))?;
... |
89 | | Ok(Response::new(stream))
90 | | }
| |_____^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#blocks_in_conditions
Check warning on line 74 in ton-grpc/src/block.rs
github-actions / clippy
in a `match` scrutinee, avoid complex blocks or closures with blocks; instead, move the block or closure higher and bind it with a `let`
warning: in a `match` scrutinee, avoid complex blocks or closures with blocks; instead, move the block or closure higher and bind it with a `let`
--> ton-grpc/src/block.rs:51:144
|
51 | async fn get_transaction_ids(&self, request: Request<GetTransactionIdsRequest>) -> Result<Response<Self::GetTransactionIdsStream>, Status> {
| ________________________________________________________________________________________________________________________________________________^
52 | | let msg = request.into_inner();
53 | |
54 | | let order = msg.order();
... |
73 | | Ok(Response::new(stream))
74 | | }
| |_____^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#blocks_in_conditions
Check warning on line 46 in ton-grpc/src/block.rs
github-actions / clippy
in a `match` scrutinee, avoid complex blocks or closures with blocks; instead, move the block or closure higher and bind it with a `let`
warning: in a `match` scrutinee, avoid complex blocks or closures with blocks; instead, move the block or closure higher and bind it with a `let`
--> ton-grpc/src/block.rs:36:106
|
36 | async fn get_shards(&self, request: Request<BlockId>) -> Result<Response<GetShardsResponse>, Status> {
| __________________________________________________________________________________________________________^
37 | | let block_id = extend_block_id(&self.client, &request.into_inner()).await
38 | | .map_err(|e: anyhow::Error| Status::internal(e.to_string()))?;
39 | |
... |
45 | | }))
46 | | }
| |_____^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#blocks_in_conditions
Check warning on line 33 in ton-grpc/src/block.rs
github-actions / clippy
in a `match` scrutinee, avoid complex blocks or closures with blocks; instead, move the block or closure higher and bind it with a `let`
warning: in a `match` scrutinee, avoid complex blocks or closures with blocks; instead, move the block or closure higher and bind it with a `let`
--> ton-grpc/src/block.rs:28:98
|
28 | async fn get_block(&self, request: Request<BlockId>) -> Result<Response<BlockIdExt>, Status> {
| __________________________________________________________________________________________________^
29 | | let block_id = extend_block_id(&self.client, &request.into_inner()).await
30 | | .map_err(|e: anyhow::Error| Status::internal(e.to_string()))?;
31 | |
32 | | Ok(Response::new(block_id.into()))
33 | | }
| |_____^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#blocks_in_conditions
Check warning on line 25 in ton-grpc/src/block.rs
github-actions / clippy
in a `match` scrutinee, avoid complex blocks or closures with blocks; instead, move the block or closure higher and bind it with a `let`
warning: in a `match` scrutinee, avoid complex blocks or closures with blocks; instead, move the block or closure higher and bind it with a `let`
--> ton-grpc/src/block.rs:20:116
|
20 | async fn get_last_block(&self, _request: Request<GetLastBlockRequest>) -> Result<Response<BlockIdExt>, Status> {
| ____________________________________________________________________________________________________________________^
21 | | let block = self.client.get_masterchain_info().await
22 | | .map_err(|e: anyhow::Error| Status::internal(e.to_string()))?.last;
23 | |
24 | | Ok(Response::new(block.into()))
25 | | }
| |_____^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#blocks_in_conditions
Check warning on line 105 in ton-grpc/src/account.rs
github-actions / clippy
in a `match` scrutinee, avoid complex blocks or closures with blocks; instead, move the block or closure higher and bind it with a `let`
warning: in a `match` scrutinee, avoid complex blocks or closures with blocks; instead, move the block or closure higher and bind it with a `let`
--> ton-grpc/src/account.rs:73:172
|
73 | async fn get_account_transactions(&self, request: Request<GetAccountTransactionsRequest>) -> std::result::Result<Response<Self::GetAccountTransactionsStream>, Status> {
| ____________________________________________________________________________________________________________________________________________________________________________^
74 | | let msg = request.into_inner();
75 | | let client = self.client.clone();
76 | |
... |
104 | | Ok(Response::new(stream))
105 | | }
| |_____^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#blocks_in_conditions
Check warning on line 68 in ton-grpc/src/account.rs
github-actions / clippy
in a `match` scrutinee, avoid complex blocks or closures with blocks; instead, move the block or closure higher and bind it with a `let`
warning: in a `match` scrutinee, avoid complex blocks or closures with blocks; instead, move the block or closure higher and bind it with a `let`
--> ton-grpc/src/account.rs:51:147
|
51 | async fn get_shard_account_cell(&self, request: Request<GetShardAccountCellRequest>) -> Result<Response<GetShardAccountCellResponse>, Status> {
| ___________________________________________________________________________________________________________________________________________________^
52 | | let msg = request.into_inner();
53 | |
54 | | let (block_id, cell) = self.fetch_shard_account_cell(&msg)
... |
67 | | Ok(Response::new(response))
68 | | }
| |_____^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#blocks_in_conditions
Check warning on line 48 in ton-grpc/src/account.rs
github-actions / clippy
in a `match` scrutinee, avoid complex blocks or closures with blocks; instead, move the block or closure higher and bind it with a `let`
warning: in a `match` scrutinee, avoid complex blocks or closures with blocks; instead, move the block or closure higher and bind it with a `let`
--> ton-grpc/src/account.rs:25:147
|
25 | async fn get_account_state(&self, request: Request<GetAccountStateRequest>) -> std::result::Result<Response<GetAccountStateResponse>, Status> {
| ___________________________________________________________________________________________________________________________________________________^
26 | | let msg = request.into_inner();
27 | |
28 | | let address = AccountAddressData::from_str(&msg.account_address)
... |
47 | | }))
48 | | }
| |_____^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#blocks_in_conditions
= note: `#[warn(clippy::blocks_in_conditions)]` on by default
Check warning on line 152 in tonlibjson-client/src/router.rs
github-actions / clippy
unneeded `return` statement
warning: unneeded `return` statement
--> tonlibjson-client/src/router.rs:152:17
|
152 | return Err(RouterError::RouteUnknown);
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
= note: `#[warn(clippy::needless_return)]` on by default
help: remove `return`
|
152 - return Err(RouterError::RouteUnknown);
152 + Err(RouterError::RouteUnknown)
|
Check warning on line 97 in tonlibjson-client/src/router.rs
github-actions / clippy
using `clone` on type `Route` which implements the `Copy` trait
warning: using `clone` on type `Route` which implements the `Copy` trait
--> tonlibjson-client/src/router.rs:97:31
|
97 | let req = req.clone();
| ^^^^^^^^^^^ help: try removing the `clone` call: `req`
|
= 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 80 in tonlibjson-client/src/cursor_client.rs
github-actions / clippy
this `let...else` may be rewritten with the `?` operator
warning: this `let...else` may be rewritten with the `?` operator
--> tonlibjson-client/src/cursor_client.rs:78:9
|
78 | / let Some(seqno) = self.right_end else {
79 | | return None;
80 | | };
| |__________^ help: replace it with: `let seqno = self.right_end?;`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#question_mark
= note: `#[warn(clippy::question_mark)]` on by default
Check warning on line 134 in tonlibjson-client/src/block.rs
github-actions / clippy
wildcard pattern covers any other pattern as it will match anyway
warning: wildcard pattern covers any other pattern as it will match anyway
--> tonlibjson-client/src/block.rs:134:13
|
134 | 1 | _ => BlockCriteria::Seqno { shard: self.id.shard, seqno: self.id.seqno }
| ^^^^^
|
= help: consider handling `_` separately
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#wildcard_in_or_patterns
= note: `#[warn(clippy::wildcard_in_or_patterns)]` on by default
Check warning on line 202 in tl_parser/src/lib.rs
github-actions / clippy
call to `.clone()` on a reference in this situation does nothing
warning: call to `.clone()` on a reference in this situation does nothing
--> tl_parser/src/lib.rs:202:25
|
202 | let prev = input.clone();
| ^^^^^^^^ help: remove this redundant call
|
= note: the type `str` does not implement `Clone`, so calling `clone` on `&str` copies the reference, which does not do anything and can be removed
= note: `#[warn(noop_method_call)]` on by default
Check warning on line 179 in tl_parser/src/lib.rs
github-actions / clippy
useless use of `format!`
warning: useless use of `format!`
--> tl_parser/src/lib.rs:179:23
|
179 | None => { format!("{}", self.r#type.constructor_number_form() )}
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using `.to_string()`: `self.r#type.constructor_number_form().to_string()`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_format
Check warning on line 86 in tl_parser/src/lib.rs
github-actions / clippy
useless use of `format!`
warning: useless use of `format!`
--> tl_parser/src/lib.rs:86:50
|
86 | Plain { name, condition: None } => { format!("{}", name) }
| ^^^^^^^^^^^^^^^^^^^ help: consider using `.to_string()`: `name.to_string()`
|
= 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
Check warning on line 156 in ton-liteserver-client/build.rs
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
--> ton-liteserver-client/build.rs:156:35
|
156 | if skip_list.contains(&&type_ident) {
| ^^^^^^^^^^^^ help: change this to: `(&type_ident)`
|
= 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 125 in ton-liteserver-client/build.rs
github-actions / clippy
methods `configure` and `configure_full` are never used
warning: methods `configure` and `configure_full` are never used
--> ton-liteserver-client/build.rs:125:8
|
117 | impl Generator {
| -------------- methods in this implementation
...
125 | fn configure(mut self, name: &str, derives: Vec<&str>) -> Self {
| ^^^^^^^^^
...
131 | fn configure_full(mut self, name: &str, configuration: TypeConfiguration) -> Self {
| ^^^^^^^^^^^^^^
Check warning on line 98 in ton-liteserver-client/build.rs
github-actions / clippy
methods `derives`, `field`, and `build` are never used
warning: methods `derives`, `field`, and `build` are never used
--> ton-liteserver-client/build.rs:98:8
|
97 | impl TypeConfigurationBuilder {
| ----------------------------- methods in this implementation
98 | fn derives(mut self, derives: Vec<&str>) -> Self {
| ^^^^^^^
...
105 | fn field(mut self, field: &str, configuration: FieldConfiguration) -> Self {
| ^^^^^
...
111 | fn build(self) -> TypeConfiguration {
| ^^^^^
Check warning on line 69 in ton-liteserver-client/build.rs
github-actions / clippy
methods `skip`, `optional`, `deserialize_with`, `serialize_with`, and `build` are never used
warning: methods `skip`, `optional`, `deserialize_with`, `serialize_with`, and `build` are never used
--> ton-liteserver-client/build.rs:69:8
|
68 | impl FieldConfigurationBuilder {
| ------------------------------ methods in this implementation
69 | fn skip(mut self) -> Self {
| ^^^^
...
74 | fn optional(mut self) -> Self {
| ^^^^^^^^
...
80 | fn deserialize_with(mut self, deserialize_with: &str) -> Self {
| ^^^^^^^^^^^^^^^^
...
86 | fn serialize_with(mut self, serialize_with: &str) -> Self {
| ^^^^^^^^^^^^^^
...
92 | fn build(self) -> FieldConfiguration {
| ^^^^^
Check warning on line 64 in ton-liteserver-client/build.rs
github-actions / clippy
fields `deserialize_with` and `serialize_with` are never read
warning: fields `deserialize_with` and `serialize_with` are never read
--> ton-liteserver-client/build.rs:64:9
|
61 | struct FieldConfiguration {
| ------------------ fields in this struct
...
64 | pub deserialize_with: Option<String>,
| ^^^^^^^^^^^^^^^^
65 | pub serialize_with: Option<String>
| ^^^^^^^^^^^^^^
Check warning on line 54 in ton-liteserver-client/build.rs
github-actions / clippy
fields `skip`, `optional`, `deserialize_with`, and `serialize_with` are never read
warning: fields `skip`, `optional`, `deserialize_with`, and `serialize_with` are never read
--> ton-liteserver-client/build.rs:54:5
|
53 | struct FieldConfigurationBuilder {
| ------------------------- fields in this struct
54 | skip: bool,
| ^^^^
55 | optional: bool,
| ^^^^^^^^
56 | deserialize_with: Option<String>,
| ^^^^^^^^^^^^^^^^
57 | serialize_with: Option<String>
| ^^^^^^^^^^^^^^
Check warning on line 37 in ton-liteserver-client/build.rs
github-actions / clippy
fields `derives` and `fields` are never read
warning: fields `derives` and `fields` are never read
--> ton-liteserver-client/build.rs:37:5
|
36 | struct TypeConfigurationBuilder {
| ------------------------ fields in this struct
37 | derives: Vec<String>,
| ^^^^^^^
38 | fields: HashMap<String, FieldConfiguration>
| ^^^^^^
Check warning on line 33 in ton-liteserver-client/build.rs
github-actions / clippy
function `configure_field` is never used
warning: function `configure_field` is never used
--> ton-liteserver-client/build.rs:33:4
|
33 | fn configure_field() -> FieldConfigurationBuilder { Default::default() }
| ^^^^^^^^^^^^^^^
Check warning on line 32 in ton-liteserver-client/build.rs
github-actions / clippy
function `configure_type` is never used
warning: function `configure_type` is never used
--> ton-liteserver-client/build.rs:32:4
|
32 | fn configure_type() -> TypeConfigurationBuilder { Default::default() }
| ^^^^^^^^^^^^^^
|
= note: `#[warn(dead_code)]` on by default