[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Add a \voicify command (issue 320820043 by address@hidden)
From: |
thomasmorley65 |
Subject: |
Re: Add a \voicify command (issue 320820043 by address@hidden) |
Date: |
Sun, 09 Apr 2017 11:36:00 -0700 |
On 2017/04/09 15:15:47, dak wrote:
mailto:address@hidden writes:
>
https://codereview.appspot.com/320820043/diff/40001/scm/music-functions.scm
> File scm/music-functions.scm (right):
>
>
https://codereview.appspot.com/320820043/diff/40001/scm/music-functions.scm#newcode968
> scm/music-functions.scm:968:
> On 2017/04/08 12:02:45, thomasmorley651 wrote:
>> Why not throw an error if lengths of `lst' and `id' doesn't match?
>
> Other possibility would be to extend the id-list with defaults, if
it is
> too short and disregard supernumerous entries.
> In both cases a warning would be appropriate.
Uh, effectively the id-list _is_ extended with defaults (1, 2, 3...)
if
I remember correctly.
> https://codereview.appspot.com/320820043/
--
David Kastrup
I then would have expected the following to work (with a warning):
{ \voicify 1 << a' \\ c' >> }
instead I get:
$ lilypond-git atest-54.ly
GNU LilyPond 2.19.59
Processing `atest-54.ly'
Parsing.../home/hermann/lilypond-git/build/out/share/lilypond/current/scm/music-functions.scm:978:31:
In expression ("No more voicification ids"):
/home/hermann/lilypond-git/build/out/share/lilypond/current/scm/music-functions.scm:978:31:
Wrong type to apply: "No more voicification ids"
Which is the full log.
-Harm
https://codereview.appspot.com/320820043/
- Re: Add a \voicify command (issue 320820043 by address@hidden), (continued)
- Re: Add a \voicify command (issue 320820043 by address@hidden), Carl Sorensen, 2017/04/07
- Re: Add a \voicify command (issue 320820043 by address@hidden), David Kastrup, 2017/04/09
- Re: Add a \voicify command (issue 320820043 by address@hidden), David Kastrup, 2017/04/09
- Re: Add a \voicify command (issue 320820043 by address@hidden), Trevor Daniels, 2017/04/09
- Re: Add a \voicify command (issue 320820043 by address@hidden), Thomas Morley, 2017/04/09
- Re: Add a \voicify command (issue 320820043 by address@hidden), Werner LEMBERG, 2017/04/09
Re: Add a \voicify command (issue 320820043 by address@hidden), thomasmorley65, 2017/04/08
Re: Add a \voicify command (issue 320820043 by address@hidden), thomasmorley65, 2017/04/08
Re: Add a \voicify command (issue 320820043 by address@hidden),
thomasmorley65 <=
Re: Add a \voicify command (issue 320820043 by address@hidden), thomasmorley65, 2017/04/09
Re: Add a \voicify command (issue 320820043 by address@hidden), thomasmorley65, 2017/04/10