Re: Add a \voicify command (issue 320820043 by address@hidden)

[thomasmorley65]

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/