Hi Ian, I found a rebasing issue that should be sorted out, as explained in my comment below.
Also, I think the subject line of this patch can be improved, since we're no longer removing `define-ly-syntax', just revising it. Thanks, Patrick http://codereview.appspot.com/2313044/diff/7001/scm/ly-syntax-constructors.scm File scm/ly-syntax-constructors.scm (right): http://codereview.appspot.com/2313044/diff/7001/scm/ly-syntax-constructors.scm#newcode251 scm/ly-syntax-constructors.scm:251: (define-ly-syntax-simple (partial dur) This part of the patch reverts Neil's changes as part of the fix for #372. Run the following command to see what I mean: git log -p scm/ly-syntax-constructors.scm In other words, there should be no changes to this procedure at all for your patch. http://codereview.appspot.com/2313044/ _______________________________________________ lilypond-devel mailing list lilypond-devel@gnu.org http://lists.gnu.org/mailman/listinfo/lilypond-devel