ponder-lab / Hybridize-Functions-Refactoring / #1482

72%

main: 72%

Build Type

Pull #518

travis-pro

Committed by khatchad

Commit Message

Refactor `markParam` to an arrow-form `switch` (#516)

Replaces the chain of eight `paramName.equals(...)` branches with a
modern switch expression. The ten parameter-name constants on
`HybridizationParameters` are all `private static final String` (lines
140-158), so they're eligible as `case` labels directly.

The two deprecated-alias branches (`JIT_COMPILE`/`EXPERIMENTAL_COMPILE`
and `REDUCE_RETRACING`/`EXPERIMENTAL_RELAX_SHAPES`) collapse to
comma-separated cases. No `default` clause: Java permits non-exhaustive
switch statements on `String`, matching the existing "unknown names are
silently ignored" behavior documented on the method.

Closes #516

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

Pull Request Pull Request #518: Refactor `markParam` to an arrow-form `switch`

Coverage Stats

20 of 23 new or added lines in 2 files covered. (86.96%)

1 existing line in 1 file now uncovered.

1185 of 1645 relevant lines covered (72.04%)

0.72 hits per line