Merge branch 'master' into nested-bookparts

[lilypond.git] / lily / completion-note-heads-engraver.cc

diff --git a/lily/completion-note-heads-engraver.cc b/lily/completion-note-heads-engraver.cc
index b116cb50b4bc9e41c6031fdc89fc330f4f6f83c9..8734b861c2a2d15eadb9b7367b508da6200619fa 100644 (file)
--- a/lily/completion-note-heads-engraver.cc
+++ b/lily/completion-note-heads-engraver.cc
@@ -14,7 +14,6 @@ using namespace std;
 #include "item.hh"
 #include "output-def.hh"
 #include "pitch.hh"
-#include "pqueue.hh"
 #include "rhythmic-head.hh"
 #include "score-engraver.hh"
 #include "spanner.hh"
@@ -28,18 +27,6 @@ using namespace std;
 /*
   TODO: make matching rest engraver.
 */
-struct Pending_tie
-{
-  Moment when_;
-  Stream_event* tie_event_;
-  Pending_tie() { tie_event_ = 0; }
-};
-
-int compare(Pending_tie const &a, Pending_tie const &b)
-{
-  return compare(a.when_, b.when_);
-}
-
 
 /*
   How does this work?
@@ -60,15 +47,10 @@ class Completion_heads_engraver : public Engraver
 {
   vector<Item*> notes_;
   vector<Item*> prev_notes_;
-
-  // Must remember notes for explicit ties.
-  vector<Item*> tie_note_candidates_;
-  vector<Stream_event*> tie_note_candidate_events_;
   vector<Grob*> ties_;
-  PQueue<Pending_tie> pending_ties_;
+
   vector<Stream_event*> note_events_;
 
-  Stream_event *current_tie_event_;
   Moment note_end_mom_;
   bool is_first_;
   Rational left_to_do_;
@@ -82,19 +64,16 @@ public:
 
 protected:
   virtual void initialize ();
-  void make_tie (Grob *, Grob *);
   void start_translation_timestep ();
   void process_music ();
   void stop_translation_timestep ();
   DECLARE_TRANSLATOR_LISTENER (note);
-  DECLARE_TRANSLATOR_LISTENER (tie);
 };
 
 void
 Completion_heads_engraver::initialize ()
 {
   is_first_ = false;
-  current_tie_event_ = 0;
 }
 
 IMPLEMENT_TRANSLATOR_LISTENER (Completion_heads_engraver, note);
@@ -111,14 +90,6 @@ Completion_heads_engraver::listen_note (Stream_event *ev)
   do_nothing_until_ = Rational (0, 0);
 }
 
-IMPLEMENT_TRANSLATOR_LISTENER (Completion_heads_engraver, tie);
-void
-Completion_heads_engraver::listen_tie (Stream_event *ev)
-{
-  is_first_ = true;
-  current_tie_event_ = ev;
-}
-
 /*
   The duration _until_ the next barline.
 */
@@ -157,14 +128,6 @@ Completion_heads_engraver::process_music ()
   if (!is_first_ && !left_to_do_)
     return;
 
-  if (current_tie_event_)
-    {
-      Pending_tie pending;
-      pending.when_ = note_end_mom_;
-      pending.tie_event_ = current_tie_event_;
-      pending_ties_.insert (pending);
-    }
-  
   is_first_ = false;
 
   Moment now = now_mom ();
@@ -211,25 +174,17 @@ Completion_heads_engraver::process_music ()
        event->unprotect ();
       notes_.push_back (note);
     }
-  
-  if (pending_ties_.size ()
-      && pending_ties_.front().when_ == now_mom())
-    {
-      for (vsize i = 0; i < tie_note_candidate_events_.size(); i++)
-       for (vsize j = 0; j < note_events_.size(); j++)
-         {
-           Pitch *p =  unsmob_pitch (note_events_[j]->get_property ("pitch"));
-           Pitch *p_last
-             = unsmob_pitch (tie_note_candidate_events_[j]->get_property ("pitch"));
-           if (p && p_last && *p == *p_last)
-             make_tie (tie_note_candidates_[i], notes_[j]);
-         }
-    }
-      
+
   if (prev_notes_.size () == notes_.size ())
     {
       for (vsize i = 0; i < notes_.size (); i++)
-       make_tie (prev_notes_[i], notes_[i]);
+       {
+         Grob *p = make_spanner ("Tie", SCM_EOL);
+         Tie::set_head (p, LEFT, prev_notes_[i]);
+         Tie::set_head (p, RIGHT, notes_[i]);
+
+         ties_.push_back (p);
+       }
     }
 
   left_to_do_ -= note_dur.get_length ();
@@ -238,19 +193,11 @@ Completion_heads_engraver::process_music ()
   /*
     don't do complicated arithmetic with grace notes.
   */
-  if (orig && now_mom ().grace_part_)
+  if (orig
+      && now_mom ().grace_part_)
     left_to_do_ = Rational (0, 0);
 }
 
-void
-Completion_heads_engraver::make_tie (Grob *left, Grob *right)
-{
-  Grob *p = make_spanner ("Tie", SCM_EOL);
-  Tie::set_head (p, LEFT, left);
-  Tie::set_head (p, RIGHT, right);
-  ties_.push_back (p);
-}
-                                    
 void
 Completion_heads_engraver::stop_translation_timestep ()
 {
@@ -265,16 +212,8 @@ void
 Completion_heads_engraver::start_translation_timestep ()
 {
   Moment now = now_mom ();
-  while (pending_ties_.size() && pending_ties_.front().when_ < now)
-    {
-      pending_ties_.delmin();
-    }
-  current_tie_event_ = 0;
   if (note_end_mom_.main_part_ <= now.main_part_)
     {
-      tie_note_candidate_events_ = note_events_;
-      tie_note_candidates_ = prev_notes_;
-
       note_events_.clear ();
       prev_notes_.clear ();
     }