Skip to content

Comments

EvalState: Make some more fields const#14366

Merged
edolstra merged 2 commits intomasterfrom
const-fields
Oct 27, 2025
Merged

EvalState: Make some more fields const#14366
edolstra merged 2 commits intomasterfrom
const-fields

Conversation

@edolstra
Copy link
Member

Motivation

Always good in a multithreaded context to be clear about what fields are mutable.

Cherry-picked from DeterminateSystems#165.

Context


Add 👍 to pull requests you find important.

The Nix maintainer team uses a GitHub project board to schedule and track reviews.

@edolstra edolstra enabled auto-merge October 27, 2025 13:40
@edolstra edolstra added this pull request to the merge queue Oct 27, 2025
Merged via the queue into master with commit 126f30d Oct 27, 2025
20 checks passed
@edolstra edolstra deleted the const-fields branch October 27, 2025 18:27
@edolstra edolstra mentioned this pull request Dec 9, 2025
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