Updates:
        Labels: -Patch-needs_work Patch-review

Comment #35 on issue 2533 by [email protected]: Patch: line_count fixes
http://code.google.com/p/lilypond/issues/detail?id=2533

The changes to slurs are due to an intentional change that Pál pointed out explicitly in his original review. There is a place slur-scoring.cc seems to have used line-count slightly wrongly to determine if a slur need fit among staff lines.

It makes sense to review the entire patch together as presented.

If the patch is good, it makes sense to commit logically-related sections, such that each commit results in a working and consistent state of the code -- *not* individual .cc files. If the changes to repeat-bars from the original patch had been a separate commit, I would have been able to revert only the troublesome part.

Reply via email to