https://codereview.appspot.com/576090043/diff/557790043/lily/multi-measure-rest.cc File lily/multi-measure-rest.cc (right):
https://codereview.appspot.com/576090043/diff/557790043/lily/multi-measure-rest.cc#newcode268 lily/multi-measure-rest.cc:268: bool oneline = (!staff) || Staff_symbol::line_positions (staff).size() == 1; If there is no staff symbol, then there isn't even one line. Are you sure you don't want this? bool oneline = staff ? /*etc.*/ : false; Incidentally, I am surprised to see the amount of work involved in determining whether the staff has one line. It's not the kind of thing one expects to involve heap allocation. That's not a problem with your change, though. https://codereview.appspot.com/576090043/
