subject:"Issue 5627\: clean up the breaking of breakable Items \(issue 571190043 by nine.fierce.ball...@gmail.com\)"

Re: Issue 5627: clean up the breaking of breakable Items (issue 571190043 by nine.fierce.ball...@gmail.com)

2019-12-04 Thread nine . fierce . ballads

https://codereview.appspot.com/571190043/diff/559290043/lily/system.cc File lily/system.cc (right): https://codereview.appspot.com/571190043/diff/559290043/lily/system.cc#newcode525 lily/system.cc:525: { On 2019/12/04 08:08:09, lemzwerg wrote: Is there a reason for grouping the code into

Re: Issue 5627: clean up the breaking of breakable Items (issue 571190043 by nine.fierce.ball...@gmail.com)

2019-12-04 Thread lemzwerg--- via Discussions on LilyPond development

LGTM, thanks! https://codereview.appspot.com/571190043/diff/559290043/lily/system.cc File lily/system.cc (right): https://codereview.appspot.com/571190043/diff/559290043/lily/system.cc#newcode525 lily/system.cc:525: { Is there a reason for grouping the code into additional braces?