read114: Pedal/lineWidth falsely dependent on DPI_F
Compiling 3.1 with
DPI_F set to 1 (instead of 5) due to https://musescore.org/en/node/284218 fails in the testsuite because the 114 file’s…
<Pedal> <endHookHeight>-1.5</endHookHeight> <lineWidth>0.15</lineWidth>
… is supposed (perhaps wrongly?) to be read, by
<Pedal> <beginText><sym>keyboardPedalPed</sym></beginText> <endHookHeight>-1.5</endHookHeight> <lineWidth>0.006</lineWidth> </Pedal>
… but instead reads as
0.3 (which, incidentally, is a factor of 5 from
I believe the reference file (and the souce code) bogus, but I do not know enough about the 1.x file format to say which value would be correct in the 3.x file format.
The code is a bit questionable… volta are read like…
volta->setLineWidth(e.readDouble() * volta->spatium());
… but pedals use…
// and, further up, even
… and ottava use yet another thing.
Even if the current
pedal-ref.mscx is correct, the code is wrong (and must add
* 5.0 / DPI_F or something equivalent).
Related to (but not a duplicate of) #290149: more DPI_F-related bugs?