Issue 571210043: Issue 5629: Staff_symbol clean-up (Closed)
Descriptionhttps://sourceforge.net/p/testlilyissues/issues/5629/
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.
Patch Set 1 : Remove unnecessary access to line-positions property #Patch Set 2 : Remove unused Staff_symbol::line_count () #Patch Set 3 : Use ly_scm2floatvector () in Staff_symbol::line_positions () #Patch Set 4 : Make height of a no-line staff [0, 0] rather than empty #
MessagesTotal messages: 4
|
||||||||||||||||||||||||||||||||||||||||||||||
