David Kastrup <d...@gnu.org> writes:

> Here is the problem: \with-link declares its argument type wrong
> (declares symbol? and uses #"label", a string).  My patch has the
> side-effect that markup commands actually check their argument lists for
> correctness.  Previously it just let any Scheme argument through
> completely unchecked.

Worse.  All regtests employ symbols, the docstring gives an example with
a string, and autogenerated code uses a string.

Whatever.  I am not taking the blame and pressure for fixing everybody
else's bugs timely, so I am simply reverting the whole merge commit.

I'll see whether it is feasible to prepare a separate patch that just
does the typechecking and nothing else, then people can clean up the
current Lilypond code base without bothering about the rest.

> Now I could revert my branch merge, or I could prepare a patch that
> stops type-checking the argument lists to markup commands, but I think
> it is saner if we grit our teeth and just fix the previously unnoticed
> bugs as they get exposed.

Obviously not.

-- 
David Kastrup


_______________________________________________
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel

Reply via email to