Skip to content

add build feature 'allocator-debug' to add additional checks in NodePtr - #588

Merged
arvidn merged 1 commit into
mainfrom
allocator-debug
Jun 18, 2025
Merged

add build feature 'allocator-debug' to add additional checks in NodePtr#588
arvidn merged 1 commit into
mainfrom
allocator-debug

Conversation

@arvidn

@arvidn arvidn commented Jun 18, 2025

Copy link
Copy Markdown
Contributor

The allocator-debug feature is disabled by default. When enabled, NodePtrs are validated when passed into the various Allocator functions. It detects mixing NodePtr from one Allocator with another, as well as using NodePtrs that were invalidated by a restore_checkpoint().

Comment thread src/test_ops.rs
@arvidn
arvidn requested a review from Rigidity June 18, 2025 13:16
@arvidn

arvidn commented Jun 18, 2025

Copy link
Copy Markdown
Contributor Author

tests are run with the new build feature here: https://github.com/Chia-Network/clvm_rs/actions/runs/15733862561/job/44341292477?pr=588

@coveralls-official

Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 15733862561

Details

  • 78 of 80 (97.5%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.04%) to 94.364%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/test_ops.rs 10 12 83.33%
Totals Coverage Status
Change from base Build 15712997395: 0.04%
Covered Lines: 6982
Relevant Lines: 7399

💛 - Coveralls

Comment thread src/allocator.rs
@arvidn
arvidn merged commit 0570735 into main Jun 18, 2025
37 of 38 checks passed
@arvidn
arvidn deleted the allocator-debug branch June 18, 2025 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants