LGTM, with one comment. I do wonder if David is right, and we should be using spanner_id instead of slur_id.
Carl http://codereview.appspot.com/4643067/diff/3001/lily/slur-engraver.cc File lily/slur-engraver.cc (right): http://codereview.appspot.com/4643067/diff/3001/lily/slur-engraver.cc#newcode201 lily/slur-engraver.cc:201: ev->origin ()->warning(_ ("Already have slur")); Should this be "already have slur"? http://codereview.appspot.com/4643067/ _______________________________________________ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel