Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(deps): bump tonlibjson-sys/ton-testnet from 4969176 to 0feaaf5 #736

Merged

fix(deps): bump tonlibjson-sys/ton-testnet from `4969176` to `0feaaf5`

38446f9
Select commit
Loading
Failed to load commit list.
Sign in for the full log view
Merged

fix(deps): bump tonlibjson-sys/ton-testnet from 4969176 to 0feaaf5 #736

fix(deps): bump tonlibjson-sys/ton-testnet from `4969176` to `0feaaf5`
38446f9
Select commit
Loading
Failed to load commit list.
GitHub Actions / clippy succeeded Mar 21, 2024 in 0s

clippy

14 warnings

Details

Results

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

Versions

  • rustc 1.76.0 (07dca489a 2024-02-04)
  • cargo 1.76.0 (c84b36747 2024-01-18)
  • clippy 0.1.76 (07dca48 2024-02-04)

Annotations

Check warning on line 22 in ton-grpc/src/message.rs

See this annotation in the file changed.

@github-actions 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

See this annotation in the file changed.

@github-actions 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

See this annotation in the file changed.

@github-actions 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

See this annotation in the file changed.

@github-actions 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

See this annotation in the file changed.

@github-actions 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

See this annotation in the file changed.

@github-actions 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

See this annotation in the file changed.

@github-actions 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

See this annotation in the file changed.

@github-actions 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

See this annotation in the file changed.

@github-actions 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

See this annotation in the file changed.

@github-actions 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

See this annotation in the file changed.

@github-actions 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 134 in tonlibjson-client/src/block.rs

See this annotation in the file changed.

@github-actions 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 70 in tvm-grpc/src/main.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

useless conversion to the same type: `std::time::Duration`

warning: useless conversion to the same type: `std::time::Duration`
  --> tvm-grpc/src/main.rs:70:18
   |
70 |         .timeout(args.timeout.into())
   |                  ^^^^^^^^^^^^^^^^^^^ help: consider removing `.into()`: `args.timeout`
   |
   = 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 220 in tonlibjson-client/build.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
   --> tonlibjson-client/build.rs:220:35
    |
220 |             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