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.