Skip to content

Finalise KIAV splitting - #680

Open
AbdullinAM wants to merge 8 commits into
abdullin/extend-expression-formatterfrom
abdullin/control-flow-expression-formatter
Open

Finalise KIAV splitting#680
AbdullinAM wants to merge 8 commits into
abdullin/extend-expression-formatterfrom
abdullin/control-flow-expression-formatter

Conversation

@AbdullinAM

@AbdullinAM AbdullinAM commented Aug 24, 2026

Copy link
Copy Markdown
Member

This is a refactor that finishes the splitting of KotlinInputAstVisitor into a set of smaller, focused sub-visitors:

  • Extract ControlFlowExpressionFormatter
  • Extract DeclarationFormatter
  • Small drive-by fixes

@AbdullinAM
AbdullinAM force-pushed the abdullin/extend-expression-formatter branch from f819326 to 31efa8e Compare August 27, 2026 14:21
@AbdullinAM
AbdullinAM force-pushed the abdullin/control-flow-expression-formatter branch from e367476 to 6677b90 Compare August 27, 2026 14:22
@AbdullinAM
AbdullinAM force-pushed the abdullin/extend-expression-formatter branch from 31efa8e to f6571df Compare August 27, 2026 14:43
@AbdullinAM
AbdullinAM force-pushed the abdullin/control-flow-expression-formatter branch from 6677b90 to 34b88a2 Compare August 27, 2026 14:44
…trolFlowExpressionFormatter

Purely mechanical change that extracts all `visit*` functions related to control flow basic  expressions into corresponding `format*` analogs
…arationFormatter

Purely mechanical change that extracts all `visit*` functions related to declarations into corresponding `format*` analogs
* Drop the per-function `Example` KDoc that the surrounding code no
  longer needs now that each formatter is named after what it formats
* Dispatch directly to `formatTypeParameterList`/`formatTypeConstraintList`
  instead of going through `format`
* One `block(ZERO) {}` becomes `block {}`
@AbdullinAM
AbdullinAM force-pushed the abdullin/extend-expression-formatter branch from f6571df to f0a781e Compare August 27, 2026 16:28
@AbdullinAM
AbdullinAM force-pushed the abdullin/control-flow-expression-formatter branch from 34b88a2 to 0486ef2 Compare August 27, 2026 16:28
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.

1 participant