Re: Bug in articulate.ly + TrillSpan + full-measure rests

[Peter Chubb]

>>>>> "Davide" == Davide Liessi <address@hidden> writes:

Davide> Dear Peter, Il 04/02/15 09.46, Peter Chubb ha scritto:
>> Looks like the code that was added to do agogic accents (aka swing)
>> expands MultiMeasureRestMusic and throws away any articulation
>> events (like the end of the trill spanner) on the way.
>> 
>> The fix is probably to add the ariculations in at the end of
>> ac:unFoldMusic when expanding MultiMeasureRestMusic.

Davide> How can I do this?


I had a brainstorm -- the problem is SkipMusic which isn't handled
anywhere.  But we can use a SkipEvent instead, and it all works!!!

diff --git a/ly/articulate.ly b/ly/articulate.ly
index bbfea19..882cf95 100644
--- a/ly/articulate.ly
+++ b/ly/articulate.ly
@@ -483,11 +483,11 @@
         (make-sequential-music
          (list-tabulate eff-nrep (lambda (i) (ly:music-deep-copy m))))))))
      ((MultiMeasureRestMusic)
-      (make-sequential-music
-       (list
+       (event-chord-wrap! (make-sequential-music (list
        (make-music 'BarCheck)
-       (make-music 'SkipMusic 'duration (ly:music-property m 'duration))
-       (make-music 'BarCheck))))
+       (make-music 'SkipEvent 'duration (ly:music-property m 'duration)
+        'articulations (ly:music-property m 'articulations))
+       (make-music 'BarCheck)))))
      (else
       m)))
    (unfold-repeats music)))