1: Reduce unnecessary access to line-positions property
In some places, Staff_symbol had already checked line-positions before calling line_count (), which checked it again.
2: Remove unused Staff_symbol::line_count () and Staff_symbol_referencer::line_count ()
3: Use ly_scm2floatvector () in Staff_symbol::line_positions ()
4: Make height of a no-line staff [0, 0] rather than empty
This fixes a bug in the output of input/regression/rest-positioning.ly, in which a staff-group bracket failed to extend to a no-line staff.
Here's the regtest difference. Previously, the bracket did not extend to the first staff.
Passes make, make check and a full make doc.
My reg test diffs attached
Patch on countdown for Dec 11th
Patch counted down - please push.