Skip to content

Conversation

@arvidn
Copy link
Contributor

@arvidn arvidn commented Nov 5, 2025

Simple generators cannot make use of any arguments passed to them, so we're also banning block references. We also no longer need to pass the CLVM deserializer program as an argument after this soft-fork.

Don't pass the CLVM deserializer or any block references to simple generators. i.e. when the SIMPLE_GENERATOR flag is set. This is enabled when hard fork 2 activates.

…nerators. i.e. when the SIMPLE_GENERATOR flag is set. This is enabled when hard fork 2 activates
@arvidn arvidn requested a review from matt-o-how November 5, 2025 13:54
@arvidn arvidn changed the title extend SIMPLE_GENERATOR flag [CHIA-3793] extend SIMPLE_GENERATOR flag Nov 5, 2025
@coveralls-official
Copy link

coveralls-official bot commented Nov 5, 2025

Pull Request Test Coverage Report for Build 19120126391

Details

  • 15 of 15 (100.0%) changed or added relevant lines in 4 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.007%) to 84.635%

Totals Coverage Status
Change from base Build 19099057450: 0.007%
Covered Lines: 17004
Relevant Lines: 20091

💛 - Coveralls

@arvidn arvidn merged commit 8e406fc into main Nov 6, 2025
81 checks passed
@arvidn arvidn deleted the simple-generator branch November 6, 2025 16:11
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.

3 participants