Skip to content

Conversation

@phrwlk
Copy link
Contributor

@phrwlk phrwlk commented Jan 12, 2026

CallStack::precall_size was written in a few places but never read anywhere in the formatter. This effectively made it dead state and caused unnecessary writes on every formatted assignment or multi-var initialization. The field and its helper methods have been removed together with the corresponding calls in sol.rs.

The change keeps the existing formatting behavior intact by preserving the neverbreak logic and all indentation decisions. Only the unused tracking state was dropped to simplify the code and avoid future confusion when maintaining the call stack logic.

@phrwlk phrwlk changed the title Refactor/fmt remove dead callstack precall refactor(fmt): remove dead CallStack precall state Jan 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

1 participant