Chia-Network / clvm_rs

Rust implementation of clvm
Apache License 2.0
66 stars 57 forks source link

Code cleanup to improve readability #324

Closed Rigidity closed 1 year ago

Rigidity commented 1 year ago

This PR aims to resolve a few issues within the code:

Non-code changes made:

This PR does not do any of the following:

coveralls-official[bot] commented 1 year ago

Pull Request Test Coverage Report for Build 5439039662


Changes Missing Coverage Covered Lines Changed/Added Lines %
src/serde/parse_atom.rs 126 127 99.21%
src/serde/tools.rs 158 159 99.37%
src/allocator.rs 300 307 97.72%
src/serde/object_cache.rs 132 140 94.29%
<!-- Total: 1323 1340 98.73% -->
Files with Coverage Reduction New Missed Lines %
src/allocator.rs 1 95.79%
src/serde/read_cache_lookup.rs 1 98.15%
src/serde/parse_atom.rs 2 98.24%
<!-- Total: 4 -->
Totals Coverage Status
Change from base Build 5332718148: 0.07%
Covered Lines: 5246
Relevant Lines: 5588

💛 - Coveralls