Commit a4c1c9b
chore: update rust to 1.88.0 (#6045)
# Changes
1. Removed long deprecated rls component, since it's been [removed from
rust in 1.87.0](rust-lang/rust#126856)
2. disabled
[uninlined_format_args](https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args)
check - otherwise I would have to change essentially every file in the
repo in this PR... - note that the lint[ has been moved back to
`pedantic` in the cargo
clippy](rust-lang/rust-clippy#15287), so the
problem will eventually go away, but it is still an issue in this
version of rust (`1.88.0`). See also
rust-lang/rust-clippy#15151
3. Explicitly ignored
[result_large_err](https://rust-lang.github.io/rust-clippy/master/index.html#result_large_err)
warnings - I'll leave it up to the code owners whether/how to fix the
warnings
4. Modified the backtrace message in
`rs/execution_environment/tests/backtraces.rs`
5. Bumped max compressed wasm size of some nns canisters from 1.5MB to
1.6MB in `file_size_check` tests
6. Fixed
[io_other_error](https://rust-lang.github.io/rust-clippy/master/index.html#io_other_error)
warnings
7. Fixed
[iter_kv_map](https://rust-lang.github.io/rust-clippy/master/index.html#iter_kv_map)
warnings
8. Fixed
[manual_contains](https://rust-lang.github.io/rust-clippy/master/index.html#manual_contains)
warnings
9. Fixed some other, rarely occurring, clippy warnings
## Benchmarks updates
### //rs/ledger_suite/icrc1/ledger:canbench_u256
Only significant changes:
| status | name | calls | ins | ins Δ% | HI | HI Δ% | SMI | SMI Δ% |
|--------|-------------------------------------|-------|---------|---------|-----|--------|-----|---------|
| + | bench_icrc1_transfers::post_upgrade | 1 | 365.33M | +3.75% | 71 |
0.00% | 0 | 0.00% |
| + | bench_icrc1_transfers::upgrade | 1 | 516.12M | +3.17% | 200 |
0.00% | 128 | 0.00% |
ins = instructions, HI = heap_increase, SMI = stable_memory_increase, Δ%
= percent change
### //rs/ledger_suite/icrc1/ledger:canbench_u64
Only significant changes:
| status | name | calls | ins | ins Δ% | HI | HI Δ% | SMI | SMI Δ% |
|--------|--------------------------------------------|-------|--------|---------|----|---------|-----|---------|
| + | bench_icrc1_transfers::icrc2_approve | 1 | 18.44B | -0.22% | 29 |
+16.00% | 128 | 0.00% |
| - | bench_icrc1_transfers::icrc1_transfer | 1 | 12.24B | -0.50% | 31 |
-8.82% | 0 | 0.00% |
| - | bench_icrc1_transfers::icrc2_transfer_from | 1 | 20.57B | -0.54% |
3 | -25.00% | 0 | 0.00% |
ins = instructions, HI = heap_increase, SMI = stable_memory_increase, Δ%
= percent change
### //rs/nns/governance:governance-canbench
Only significant changes:
| status | name | calls | ins | ins Δ% | HI | HI Δ% | SMI | SMI Δ% |
|--------|-----------------------------------|-------|--------|---------|----|--------|-----|---------|
| + | add_neuron_inactive_maximum | | 64.86M | +5.79% | 0 | 0.00% | 0 |
0.00% |
| + | add_neuron_active_maximum_stable | | 64.86M | +5.79% | 0 | 0.00% |
0 | 0.00% |
| - | list_neurons_stable | | 63.04M | -5.65% | 9 | 0.00% | 0 | 0.00% |
| - | list_neurons_by_subaccount_stable | | 62.12M | -5.67% | 5 | 0.00%
| 0 | 0.00% |
| - | range_neurons_performance | | 51.56M | -6.28% | 0 | 0.00% | 0 |
0.00% |
ins = instructions, HI = heap_increase, SMI = stable_memory_increase, Δ%
= percent change
---------
Co-authored-by: IDX GitHub Automation <>
Co-authored-by: IDX GitHub Automation <[email protected]>1 parent 040a532 commit a4c1c9b
File tree
106 files changed
+275
-280
lines changed- .devcontainer
- .github
- workflows-source
- workflows
- bazel/conf
- ci
- container
- scripts
- rs
- artifact_pool/src
- bitcoin/adapter/test_utils/src
- boundary_node/rate_limits/canister
- canister_client/src
- canister_sandbox/src
- consensus
- dkg/src
- idkg/src
- payload_builder
- src
- consensus
- tests/framework
- utils/src
- crypto
- internal/crypto_service_provider/src
- threshold/ni_dkg/tests
- vault
- local_csp_vault
- basic_sig
- idkg
- multi_sig
- ni_dkg
- tls
- remote_csp_vault
- test_utils
- src/sign/canister_threshold_sig/idkg
- embedders/src
- ethereum/ledger-suite-orchestrator/src/state
- execution_environment
- src
- execution
- query_handler
- tests
- http_utils/src
- https_outcalls
- adapter/src
- client/src
- consensus/src/payload_builder
- ic_os
- device/src
- grub/src
- vsock/vsock_lib/src/host
- interfaces/src
- crypto/sign
- ledger_suite
- icp/ledger/src
- icrc1/ledger
- canbench_results
- src
- nervous_system/common/validation/src
- nns
- governance
- canbench
- src/neurons_fund
- tests
- integration_tests
- orchestrator/src
- pocket_ic_server/src
- state_api
- prep/src/bin
- protobuf/src
- query_stats/src
- registry
- canister/tests
- local_store/src
- replica/src
- rosetta-api/icp/src
- rust_canisters
- canister_test/src
- dfn_core/src
- stable_reader/src
- sns
- governance
- canbench
- tests/fixtures
- state_machine_tests/src
- state_manager/src
- sys/src
- test_utilities
- embedders/src
- src
- tests/driver/src/driver
- tla_instrumentation/tla_instrumentation/src
- types/types/src/consensus
- idkg
- xnet/payload_builder/tests
Some content is hidden
Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.
106 files changed
+275
-280
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | | - | |
| 2 | + | |
3 | 3 | | |
4 | 4 | | |
5 | 5 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
34 | 34 | | |
35 | 35 | | |
36 | 36 | | |
37 | | - | |
| 37 | + | |
38 | 38 | | |
39 | 39 | | |
40 | 40 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
17 | 17 | | |
18 | 18 | | |
19 | 19 | | |
20 | | - | |
| 20 | + | |
21 | 21 | | |
22 | 22 | | |
23 | 23 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
25 | 25 | | |
26 | 26 | | |
27 | 27 | | |
28 | | - | |
| 28 | + | |
29 | 29 | | |
30 | 30 | | |
31 | 31 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
14 | 14 | | |
15 | 15 | | |
16 | 16 | | |
17 | | - | |
| 17 | + | |
18 | 18 | | |
19 | 19 | | |
20 | 20 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
109 | 109 | | |
110 | 110 | | |
111 | 111 | | |
112 | | - | |
| 112 | + | |
113 | 113 | | |
114 | 114 | | |
115 | 115 | | |
| |||
298 | 298 | | |
299 | 299 | | |
300 | 300 | | |
301 | | - | |
| 301 | + | |
302 | 302 | | |
303 | 303 | | |
304 | 304 | | |
| |||
342 | 342 | | |
343 | 343 | | |
344 | 344 | | |
345 | | - | |
| 345 | + | |
346 | 346 | | |
347 | 347 | | |
348 | 348 | | |
| |||
364 | 364 | | |
365 | 365 | | |
366 | 366 | | |
367 | | - | |
| 367 | + | |
368 | 368 | | |
369 | 369 | | |
370 | 370 | | |
| |||
396 | 396 | | |
397 | 397 | | |
398 | 398 | | |
399 | | - | |
| 399 | + | |
400 | 400 | | |
401 | 401 | | |
402 | 402 | | |
| |||
478 | 478 | | |
479 | 479 | | |
480 | 480 | | |
481 | | - | |
| 481 | + | |
482 | 482 | | |
483 | 483 | | |
484 | 484 | | |
| |||
497 | 497 | | |
498 | 498 | | |
499 | 499 | | |
500 | | - | |
| 500 | + | |
501 | 501 | | |
502 | 502 | | |
503 | 503 | | |
| |||
534 | 534 | | |
535 | 535 | | |
536 | 536 | | |
537 | | - | |
| 537 | + | |
538 | 538 | | |
539 | 539 | | |
540 | 540 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
20 | 20 | | |
21 | 21 | | |
22 | 22 | | |
23 | | - | |
| 23 | + | |
24 | 24 | | |
25 | 25 | | |
26 | 26 | | |
| |||
38 | 38 | | |
39 | 39 | | |
40 | 40 | | |
41 | | - | |
| 41 | + | |
42 | 42 | | |
43 | 43 | | |
44 | 44 | | |
| |||
68 | 68 | | |
69 | 69 | | |
70 | 70 | | |
71 | | - | |
| 71 | + | |
72 | 72 | | |
73 | 73 | | |
74 | 74 | | |
| |||
110 | 110 | | |
111 | 111 | | |
112 | 112 | | |
113 | | - | |
| 113 | + | |
114 | 114 | | |
115 | 115 | | |
116 | 116 | | |
| |||
146 | 146 | | |
147 | 147 | | |
148 | 148 | | |
149 | | - | |
| 149 | + | |
150 | 150 | | |
151 | 151 | | |
152 | 152 | | |
| |||
207 | 207 | | |
208 | 208 | | |
209 | 209 | | |
210 | | - | |
| 210 | + | |
211 | 211 | | |
212 | 212 | | |
213 | 213 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
17 | 17 | | |
18 | 18 | | |
19 | 19 | | |
20 | | - | |
| 20 | + | |
21 | 21 | | |
22 | 22 | | |
23 | 23 | | |
| |||
61 | 61 | | |
62 | 62 | | |
63 | 63 | | |
64 | | - | |
| 64 | + | |
65 | 65 | | |
66 | 66 | | |
67 | 67 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
32 | 32 | | |
33 | 33 | | |
34 | 34 | | |
35 | | - | |
| 35 | + | |
36 | 36 | | |
37 | 37 | | |
38 | 38 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
31 | 31 | | |
32 | 32 | | |
33 | 33 | | |
34 | | - | |
| 34 | + | |
35 | 35 | | |
36 | 36 | | |
37 | 37 | | |
| |||
56 | 56 | | |
57 | 57 | | |
58 | 58 | | |
59 | | - | |
| 59 | + | |
60 | 60 | | |
61 | 61 | | |
62 | 62 | | |
| |||
97 | 97 | | |
98 | 98 | | |
99 | 99 | | |
100 | | - | |
| 100 | + | |
101 | 101 | | |
102 | 102 | | |
103 | 103 | | |
| |||
124 | 124 | | |
125 | 125 | | |
126 | 126 | | |
127 | | - | |
| 127 | + | |
128 | 128 | | |
129 | 129 | | |
130 | 130 | | |
| |||
0 commit comments