|
From: | Carl . D . Sorensen |
Subject: | Re: Fix issue 75: Allow multiple concurrent slurs (issue4643067) |
Date: | Tue, 05 Jul 2011 05:40:49 +0000 |
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/
[Prev in Thread] | Current Thread | [Next in Thread] |