Reviewers: ,
Message:
Please review, fixes issue 4979.
-Paul
Description:
Don't merge non-overlapping ledger lines
Issue 4979/2: Add regression test for non-merging ledger lines
Issue 4979/1: Don't merge non-overlapping ledger lines
Please review this at https://codereview.appspot.com/308
On 2016/10/02 20:46:47, pwm wrote:
Please review, fixes issue 4979.
-Paul
I can't review C++, but I applied your patch and tested it with my code
from
http://lists.gnu.org/archive/html/lilypond-user/2012-07/msg00336.html
(A far more extreme example than your regtest)
Works nicely.
So far: LGT
On 10/03/2016 06:10 AM, thomasmorle...@gmail.com wrote:
I can't review C++, but I applied your patch and tested it with my code
from
http://lists.gnu.org/archive/html/lilypond-user/2012-07/msg00336.html
(A far more extreme example than your regtest)
Works nicely.
So far: LGTM
Now that's a go