Skip to content

[gas] optimize dependency gas by avoid allocating new ModuleId

Codecov / codecov/project succeeded Feb 24, 2024 in 0s

64.0% (-7.4%) compared to d4fdb8f

View this Pull Request on Codecov

64.0% (-7.4%) compared to d4fdb8f

Details

Codecov Report

Attention: Patch coverage is 36.61670% with 296 lines in your changes are missing coverage. Please review.

Project coverage is 64.0%. Comparing base (d4fdb8f) to head (25f4925).
Report is 41 commits behind head on main.

❗ Current head 25f4925 differs from pull request most recent head 6b451ac. Consider uploading reports for the commit 6b451ac to get more accurate results

Files Patch % Lines
third_party/move/move-vm/runtime/src/loader/mod.rs 31.0% 69 Missing ⚠️
aptos-move/aptos-vm/src/aptos_vm.rs 0.0% 57 Missing ⚠️
third_party/move/move-vm/runtime/src/session.rs 8.1% 45 Missing ⚠️
third_party/move/move-binary-format/src/access.rs 0.0% 30 Missing ⚠️
...aptos-gas-schedule/src/gas_schedule/transaction.rs 0.0% 20 Missing ⚠️
aptos-move/aptos-gas-meter/src/algebra.rs 0.0% 18 Missing ⚠️
aptos-move/aptos-gas-meter/src/meter.rs 0.0% 14 Missing ⚠️
third_party/move/move-vm/runtime/src/move_vm.rs 66.6% 8 Missing ⚠️
..._party/move/move-vm/test-utils/src/gas_schedule.rs 0.0% 8 Missing ⚠️
third_party/move/move-vm/types/src/gas.rs 0.0% 8 Missing ⚠️
... and 5 more
Additional details and impacted files 
@@            Coverage Diff             @@
##             main   #12166      +/-   ##
==========================================
- Coverage    71.4%    64.0%    -7.4%     
==========================================
  Files         810      793      -17     
  Lines      184821   176326    -8495     
==========================================
- Hits       132047   113016   -19031     
- Misses      52774    63310   +10536

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

View more details on Codecov