X-Git-Url: https://git.donarmstrong.com/?a=blobdiff_plain;f=lily%2Fmidi-walker.cc;h=d9765c681450931fe85acca5c81213a90a153001;hb=5d84bfad4626892bcffd05adcced53c8a2329047;hp=6fbb780a1ba4c05b98ac9af1c2d40ffa825e6835;hpb=2f320e9722ec20d6f13ec567d007763cc912c5da;p=lilypond.git

diff --git a/lily/midi-walker.cc b/lily/midi-walker.cc
index 6fbb780a1b..d9765c6814 100644
--- a/lily/midi-walker.cc
+++ b/lily/midi-walker.cc
@@ -1,7 +1,7 @@
 /*
   This file is part of LilyPond, the GNU music typesetter.
 
-  Copyright (C) 1997--2011 Han-Wen Nienhuys <hanwen@xs4all.nl>
+  Copyright (C) 1997--2015 Han-Wen Nienhuys <hanwen@xs4all.nl>
   Jan Nieuwenhuizen <janneke@gnu.org>
 
   LilyPond is free software: you can redistribute it and/or modify
@@ -46,19 +46,21 @@ compare (Midi_note_event const &left, Midi_note_event const &right)
 }
 
 bool
-audio_item_less (Audio_item * const a,
-		 Audio_item * const b)
+audio_item_less (Audio_item *const a,
+                 Audio_item *const b)
 {
-  return a->get_column ()->when_ <  b->get_column ()->when_;
+  return a->get_column ()->when_ < b->get_column ()->when_;
 }
 
-Midi_walker::Midi_walker (Audio_staff *audio_staff, Midi_track *track)
+Midi_walker::Midi_walker (Audio_staff *audio_staff, Midi_track *track, int start_tick)
 {
   track_ = track;
   index_ = 0;
   items_ = audio_staff->audio_items_;
   vector_sort (items_, audio_item_less);
-  last_tick_ = 0;
+  //Scores that begin with grace notes start at negative times. This
+  //is OK - MIDI output doesn't use absolute ticks, only differences.
+  last_tick_ = start_tick;
   percussion_ = audio_staff->percussion_;
   merge_unisons_ = audio_staff->merge_unisons_;
 }
@@ -87,41 +89,42 @@ Midi_walker::do_start_note (Midi_note *note)
                         Real (384 * 4)) + now_ticks;
   for (vsize i = 0; i < stop_note_queue.size (); i++)
     {
-      /* if this pitch already in queue */
-      if (stop_note_queue[i].val->get_semitone_pitch ()
-	  == note->get_semitone_pitch ())
-	{
-	  int queued_ticks
-	    = stop_note_queue[i].val->audio_->audio_column_->ticks ();
-	  // If the two notes started at the same time, or option is set,
-	  if (now_ticks == queued_ticks || merge_unisons_)
-	    {
-	      // merge them.
-	      if (stop_note_queue[i].key < stop_ticks)
-		{
-		  Midi_note_event e;
-		  e.val = stop_note_queue[i].val;
-		  e.key = stop_ticks;
-		  stop_note_queue[i].ignore_ = true;
-		  stop_note_queue.insert (e);
-		}
-	      note = 0;
-	      break;
-	    }
-	  else
-	    {
-	      // A note was played that interruped a played note.
-	      // Stop the old note, and continue to the greatest moment
-	      // between the two.
-	      if (stop_note_queue[i].key > stop_ticks)
-		{
-		  stop_ticks = stop_note_queue[i].key;
-		}
-	      output_event (now_ticks, stop_note_queue[i].val);
-	      stop_note_queue[i].ignore_ = true;
-	      break;
-	    }
-	}
+      /* if this pitch already in queue, and is not already ignored */
+      if (!stop_note_queue[i].ignore_ &&
+          stop_note_queue[i].val->get_semitone_pitch ()
+          == note->get_semitone_pitch ())
+        {
+          int queued_ticks
+            = stop_note_queue[i].val->audio_->audio_column_->ticks ();
+          // If the two notes started at the same time, or option is set,
+          if (now_ticks == queued_ticks || merge_unisons_)
+            {
+              // merge them.
+              if (stop_note_queue[i].key < stop_ticks)
+                {
+                  Midi_note_event e;
+                  e.val = stop_note_queue[i].val;
+                  e.key = stop_ticks;
+                  stop_note_queue[i].ignore_ = true;
+                  stop_note_queue.insert (e);
+                }
+              note = 0;
+              break;
+            }
+          else
+            {
+              // A note was played that interruped a played note.
+              // Stop the old note, and continue to the greatest moment
+              // between the two.
+              if (stop_note_queue[i].key > stop_ticks)
+                {
+                  stop_ticks = stop_note_queue[i].key;
+                }
+              output_event (now_ticks, stop_note_queue[i].val);
+              stop_note_queue[i].ignore_ = true;
+              break;
+            }
+        }
     }
 
   if (note)
@@ -144,9 +147,9 @@ Midi_walker::do_stop_notes (int max_ticks)
     {
       Midi_note_event e = stop_note_queue.get ();
       if (e.ignore_)
-	{
-	  continue;
-	}
+        {
+          continue;
+        }
 
       int stop_ticks = e.key;
       Midi_note *note = e.val;
@@ -178,27 +181,28 @@ void
 Midi_walker::process ()
 {
   Audio_item *audio = items_[index_];
-  do_stop_notes (audio->audio_column_->ticks ());
+  Audio_column *col = audio->get_column ();
+  do_stop_notes (col->ticks ());
 
   if (Midi_item *midi = get_midi (audio))
     {
       if (Midi_note *note = dynamic_cast<Midi_note *> (midi))
-	{
-	  if (note->audio_->length_mom_.to_bool ())
-	    do_start_note (note);
-	}
+        {
+          if (note->audio_->length_mom_.to_bool ())
+            do_start_note (note);
+        }
       else
-	output_event (audio->audio_column_->ticks (), midi);
+        output_event (audio->audio_column_->ticks (), midi);
     }
 }
 
-Midi_item*
+Midi_item *
 Midi_walker::get_midi (Audio_item *i)
 {
   Midi_item *mi = Midi_item::get_midi (i);
 
   if (percussion_)
-    if (Midi_channel_item *mci = dynamic_cast<Midi_channel_item*> (mi))
+    if (Midi_channel_item *mci = dynamic_cast<Midi_channel_item *> (mi))
       mci->channel_ = 9;
 
   midi_events_.push_back (mi);
@@ -212,7 +216,7 @@ Midi_walker::ok () const
 }
 
 void
-Midi_walker::operator ++ (int)
+Midi_walker::operator ++(int)
 {
   assert (ok ());
   index_++;