Re: Don't merge non-overlapping ledger lines (issue 308560043 by paulwmor...@gmail.com)

2016-10-03 Thread Paul
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

Re: Don't merge non-overlapping ledger lines (issue 308560043 by paulwmor...@gmail.com)

2016-10-03 Thread thomasmorley65
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

Don't merge non-overlapping ledger lines (issue 308560043 by paulwmor...@gmail.com)

2016-10-02 Thread paulwmorris
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