Re: SchemeTextSpanEvent in scheme-text-spanner.ly partly broken?

[Thomas Morley]

2016-08-08 12:37 GMT+02:00 David Kastrup <address@hidden>:
> Thomas Morley <address@hidden> writes:
>
>> Hi,
>>
>> in input/regression/scheme-text-spanner.ly `SchemeTextSpanEvent' is
>> surely meant to be a post-event, at least
>> http://lilypond.org/doc/v2.14/input/regression/collated-files.html
>> shows this behaviour.
>>
>> Adding 'post-event in (define scheme-event-spanner-types ...) fixes it.
>>
>> I'd prepare a patch for it, not sure it's the best fix, though.
>>
>> Any advice?
>
> The post-event property (to clue the parser in explicitly rather than
> relying on some other hard-coded stuff I don't quite remember) has been
> introduced in 2.15.28.
>
> commit 12eab82dd932613faa11d4c438ba48d5c5add40f
> Author: David Kastrup <address@hidden>
> Date:   Fri Jan 20 21:38:09 2012 +0100
>
>     Define a post-event music type and let post-event? and ly:event? use it.
>
>
> (there probably is an issue associated with it).  scheme-text-spanner
> probably has not been updated.
>
> --
> David Kastrup


Issue at
https://sourceforge.net/p/testlilyissues/issues/4953/
patch is up
http://codereview.appspot.com/301560043

Hope you enjoyed your accordionist meeting.

Thanks,
  Harm