[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: add \shape (issue 6255056)
From: |
david . nalesnik |
Subject: |
Re: add \shape (issue 6255056) |
Date: |
Thu, 31 May 2012 17:40:40 +0000 |
Thank you for reviewing this, Neil!
On 2012/05/31 16:22:53, Neil Puttock wrote:
http://codereview.appspot.com/6255056/diff/5001/input/regression/shape-slurs.ly
File input/regression/shape-slurs.ly (right):
http://codereview.appspot.com/6255056/diff/5001/input/regression/shape-slurs.ly#newcode5
input/regression/shape-slurs.ly:5: @code{\\shape}. The blue slurs are
modified
from the default slurs shown first."
two spaces after full-stop/period
Done.
http://codereview.appspot.com/6255056/diff/5001/ly/music-functions-init.ly
File ly/music-functions-init.ly (right):
http://codereview.appspot.com/6255056/diff/5001/ly/music-functions-init.ly#newcode1041
ly/music-functions-init.ly:1041: (_i "Offset control-points of
@var{grob} by
@var{offsets}. The argument
two spaces follow full stop/period
Done. (This is actually what I do instinctively. Here I was fighting
the impulse--with inconsistent results!)
http://codereview.appspot.com/6255056/diff/5001/ly/music-functions-init.ly#newcode1042
ly/music-functions-init.ly:1042: is a Scheme alist or list of alists.
Each
element of an alist represents
`offsets' isn't an alist: there's no (key . value) relationship in the
list of
number pairs
Done. This certainly makes more sense.
http://codereview.appspot.com/6255056/diff/5001/ly/music-functions-init.ly#newcode1046
ly/music-functions-init.ly:1046: (siblings (if (and (ly:grob? orig)
ly:grob-original always returns a grob, so this is unnecessary
Done.
http://codereview.appspot.com/6255056/diff/5001/ly/music-functions-init.ly#newcode1048
ly/music-functions-init.ly:1048: (ly:spanner-broken-into orig) '() ))
remove spurious whitespace
Done.
http://codereview.appspot.com/6255056/diff/5001/ly/music-functions-init.ly#newcode1051
ly/music-functions-init.ly:1051: (reverse (ly:grob-basic-properties
grob))))
fix indent (aligns with 'control-points)
Done.
http://codereview.appspot.com/6255056/
- add \shape (issue 6255056), david . nalesnik, 2012/05/28
- Re: add \shape (issue 6255056), janek . lilypond, 2012/05/28
- Re: add \shape (issue 6255056), david . nalesnik, 2012/05/28
- Re: add \shape (issue 6255056), janek . lilypond, 2012/05/30
- Re: add \shape (issue 6255056), graham, 2012/05/30
- Re: add \shape (issue 6255056), n . puttock, 2012/05/31
- Re: add \shape (issue 6255056),
david . nalesnik <=