ossia / score / 31800453194

19%

Build Type

push

github

Committed by jcelerier

Commit Message

lv2: restore control port values on document reload

Two combined regressions reset control ports to defaults on reload
(reported on DrumSlam, Bode frequency shifter; B.Shapr appeared fine only
because its audible state rides in state:interface):

- Since 03ef54526, Port::loadData keeps the current value unless
  PortLoadDataFlags::ReloadValue is passed; restore_ports called it with
  the default, so every reloaded LV2 control kept its just-created default.
- Documents saved before port properties were honored store every control
  as a FloatSlider; when the concrete widget differs (IntSlider, Toggle,
  ComboBox, LogFloatSlider), copy value + address across the type change
  instead of dropping them.

Fixture bundle grew integer + toggled ports; tests cover the same-version
JSON round-trip and a synthesized legacy all-FloatSlider document.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

Coverage Stats

2 of 13 new or added lines in 1 file covered. (15.38%)

3283 existing lines in 37 files now uncovered.

31837 of 204847 relevant lines covered (15.54%)

1581.52 hits per line