[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Improvements to \offset (issue 15850044)
From: |
david . nalesnik |
Subject: |
Re: Improvements to \offset (issue 15850044) |
Date: |
Wed, 23 Oct 2013 14:24:33 +0000 |
https://codereview.appspot.com/15850044/diff/120001/ly/music-functions-init.ly
File ly/music-functions-init.ly (right):
https://codereview.appspot.com/15850044/diff/120001/ly/music-functions-init.ly#newcode722
ly/music-functions-init.ly:722: (make-music 'Music)))
Hmm...
Shouldn't the return here be `item' since it is guaranteed to be music?
The following code
\relative c'' {
<c e g>-\offset foo #'(-1 . 1) \arpeggio
}
results in a failed file.
The way the code is now, the snippet returns an error about bad grob
property path (`foo') and "post-event expected." No arpeggio appears.
With a return of item, we only get the warning about property path, and
the arpeggio is typeset. This seems more sensible.
https://codereview.appspot.com/15850044/
- Improvements to \offset (issue 15850044), david . nalesnik, 2013/10/22
- Re: Improvements to \offset (issue 15850044), david . nalesnik, 2013/10/22
- Re: Improvements to \offset (issue 15850044), david . nalesnik, 2013/10/23
- Re: Improvements to \offset (issue 15850044),
david . nalesnik <=
- Re: Improvements to \offset (issue 15850044), dak, 2013/10/23
- Re: Improvements to \offset (issue 15850044), dak, 2013/10/23
- Re: Improvements to \offset (issue 15850044), david . nalesnik, 2013/10/23
- Re: Improvements to \offset (issue 15850044), janek . lilypond, 2013/10/23