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?

Not sure ...

I tried this, but it didn't work --- more investiagtion (that I don;t
have time for at present is needed:
diff --git a/ly/articulate.ly b/ly/articulate.ly
index bbfea19..48a8535 100644
--- a/ly/articulate.ly
+++ b/ly/articulate.ly
@@ -486,7 +486,8 @@
       (make-sequential-music
        (list
        (make-music 'BarCheck)
-       (make-music 'SkipMusic 'duration (ly:music-property m 'duration))
+       (make-music 'SkipMusic 'duration (ly:music-property m 'duration)
+        'articulations (ly:music-property m 'articulations))
        (make-music 'BarCheck))))
      (else
       m)))
@@ -538,7 +539,7 @@
       ((BeamEvent) ; throw away beam events, or they'll be duplicated by turn 
or trill
        (loop factor newelements tail actions))
 
-      ((LineBreakEvent FingeringEvent MarkEvent BreathingEvent TieEvent 
SkipEvent RestEvent) ; pass through some events.
+      ((SkipMusic LineBreakEvent FingeringEvent MarkEvent BreathingEvent 
TieEvent SkipEvent RestEvent) ; pass through some events.
        (loop (cons 1 1) (cons e newelements) tail actions))
 
       ((ArticulationEvent)
(END)