Re: T1249 - Remove (define define-ly-syntax define-public). (issue2313044)

[pnorcks]

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/