]> git.donarmstrong.com Git - lilypond.git/commitdiff
Issue 4903/4: Fold Slur_proto_engraver into Slur_engraver
authorDavid Kastrup <dak@gnu.org>
Mon, 20 Jun 2016 21:48:14 +0000 (23:48 +0200)
committerDavid Kastrup <dak@gnu.org>
Fri, 24 Jun 2016 20:50:09 +0000 (22:50 +0200)
A symmetrical common base class to both Slur_engraver and
Phrasing_slur_engraver seems like an unnecessary complication.  Instead,
Phrasing_slur_engraver can just be derived from Slur_engraver .

lily/include/slur-engraver.hh [new file with mode: 0644]
lily/include/slur-proto-engraver.hh [deleted file]
lily/phrasing-slur-engraver.cc
lily/slur-engraver.cc
lily/slur-proto-engraver.cc [deleted file]

diff --git a/lily/include/slur-engraver.hh b/lily/include/slur-engraver.hh
new file mode 100644 (file)
index 0000000..24e715d
--- /dev/null
@@ -0,0 +1,81 @@
+/*
+  This file is part of LilyPond, the GNU music typesetter.
+
+  Copyright (C) 2013--2015 Mike Solomon <mike@mikesolomon.org>
+  Copyright (C) 2016 David Kastrup <dak@gnu.org>
+
+  LilyPond is free software: you can redistribute it and/or modify
+  it under the terms of the GNU General Public License as published by
+  the Free Software Foundation, either version 3 of the License, or
+  (at your option) any later version.
+
+  LilyPond is distributed in the hope that it will be useful,
+  but WITHOUT ANY WARRANTY; without even the implied warranty of
+  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+  GNU General Public License for more details.
+
+  You should have received a copy of the GNU General Public License
+  along with LilyPond.  If not, see <http://www.gnu.org/licenses/>.
+*/
+
+#ifndef SLUR_ENGRAVER_HH
+#define SLUR_ENGRAVER_HH
+
+#include "engraver.hh"
+#include <map>
+
+class Slur_engraver : public Engraver
+{
+protected:
+  struct Event_info {
+    Stream_event *slur_, *note_;
+    Event_info (Stream_event *slur, Stream_event *note)
+      : slur_ (slur), note_ (note)
+    { }
+  };
+  // protected so that subclasses can see them
+  vector<Event_info> start_events_;
+  vector<Event_info> stop_events_;
+
+  typedef std::multimap<Stream_event *, Spanner *> Note_slurs;
+  Drul_array<Note_slurs> note_slurs_;
+  vector<Grob *> slurs_;
+  vector<Grob *> end_slurs_;
+  vector<Grob_info> objects_to_acknowledge_;
+
+  virtual SCM event_symbol () const;
+  virtual bool double_property () const;
+  virtual SCM grob_symbol () const;
+  virtual const char* object_name () const;
+
+  void acknowledge_inline_accidental (Grob_info);
+  void acknowledge_fingering (Grob_info);
+  void acknowledge_note_column (Grob_info);
+  void acknowledge_script (Grob_info);
+  void acknowledge_dots (Grob_info);
+  void acknowledge_text_script (Grob_info);
+  void acknowledge_end_tie (Grob_info);
+  void acknowledge_tuplet_number (Grob_info);
+
+  void listen_note (Stream_event *ev);
+  void listen_slur (Stream_event *ev, Stream_event *note);
+  // You'd think the following is the same as defaulting `note' to 0,
+  // but template resolution for trampolines disagrees.  Huh.
+  void listen_slur (Stream_event *ev) { listen_slur (ev, 0); }
+  void acknowledge_extra_object (Grob_info);
+  void stop_translation_timestep ();
+  void process_music ();
+
+  bool can_create_slur (const string&, vsize, vsize *, Stream_event *);
+  void create_slur (const string &spanner_id, Event_info evi, Grob *g_cause, Direction dir, bool left_broken);
+  bool try_to_end (Event_info evi);
+
+  virtual void set_melisma (bool);
+  virtual void finalize ();
+  virtual void derived_mark () const;
+
+public:
+  TRANSLATOR_DECLARATIONS (Slur_engraver);
+};
+
+#endif // SLUR_ENGRAVER_HH
diff --git a/lily/include/slur-proto-engraver.hh b/lily/include/slur-proto-engraver.hh
deleted file mode 100644 (file)
index af4cacf..0000000
+++ /dev/null
@@ -1,83 +0,0 @@
-/*
-  This file is part of LilyPond, the GNU music typesetter.
-
-  Copyright (C) 2013--2015 Mike Solomon <mike@mikesolomon.org>
-
-  LilyPond is free software: you can redistribute it and/or modify
-  it under the terms of the GNU General Public License as published by
-  the Free Software Foundation, either version 3 of the License, or
-  (at your option) any later version.
-
-  LilyPond is distributed in the hope that it will be useful,
-  but WITHOUT ANY WARRANTY; without even the implied warranty of
-  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-  GNU General Public License for more details.
-
-  You should have received a copy of the GNU General Public License
-  along with LilyPond.  If not, see <http://www.gnu.org/licenses/>.
-*/
-
-#ifndef SLUR_PROTO_ENGRAVER_HH
-#define SLUR_PROTO_ENGRAVER_HH
-
-#include "engraver.hh"
-#include "moment.hh"
-#include <map>
-
-class Slur_proto_engraver : public Engraver
-{
-protected:
-  struct Event_info {
-    Stream_event *slur_, *note_;
-    Event_info (Stream_event *slur, Stream_event *note)
-      : slur_ (slur), note_ (note)
-    { }
-  };
-  // protected so that subclasses can see them
-  vector<Event_info> start_events_;
-  vector<Event_info> stop_events_;
-
-  typedef std::multimap<Stream_event *, Spanner *> Note_slurs;
-  Drul_array<Note_slurs> note_slurs_;
-  vector<Grob *> slurs_;
-  vector<Grob *> end_slurs_;
-  vector<Grob_info> objects_to_acknowledge_;
-
-  virtual SCM event_symbol () const = 0;
-  virtual bool double_property () const = 0;
-  virtual SCM grob_symbol () const = 0;
-  virtual const char* object_name () const = 0;
-
-  void acknowledge_inline_accidental (Grob_info);
-  void acknowledge_fingering (Grob_info);
-  void acknowledge_note_column (Grob_info);
-  void acknowledge_script (Grob_info);
-  void acknowledge_dots (Grob_info);
-  void acknowledge_text_script (Grob_info);
-  void acknowledge_end_tie (Grob_info);
-  void acknowledge_tuplet_number (Grob_info);
-
-  void listen_note (Stream_event *ev);
-  void listen_slur (Stream_event *ev, Stream_event *note);
-  // You'd think the following is the same as defaulting `note' to 0,
-  // but template resolution for trampolines disagrees.  Huh.
-  void listen_slur (Stream_event *ev) { listen_slur (ev, 0); }
-  void acknowledge_extra_object (Grob_info);
-  void stop_translation_timestep ();
-  void process_music ();
-
-  bool can_create_slur (const string&, vsize, vsize *, Stream_event *);
-  void create_slur (const string &spanner_id, Event_info evi, Grob *g_cause, Direction dir, bool left_broken);
-  bool try_to_end (Event_info evi);
-
-  virtual void set_melisma (bool);
-  virtual void finalize ();
-  virtual void derived_mark () const;
-
-public:
-  // no TRANSLATOR_DECLARATIONS (Slur_proto_engraver) needed since this
-  // class is abstract
-  DECLARE_TRANSLATOR_CALLBACKS (Slur_proto_engraver);
-};
-
-#endif // SLUR_PROTO_ENGRAVER_HH
index 70b6dc764b922de91e24e5a8c73d45ed227849ab..cdb50cd9c4f51a3ad20d3c48c72f00d1d33ba434 100644 (file)
   along with LilyPond.  If not, see <http://www.gnu.org/licenses/>.
 */
 
-#include "slur-proto-engraver.hh"
+#include "slur-engraver.hh"
 
 #include "translator.icc"
 
-class Phrasing_slur_engraver : public Slur_proto_engraver
+class Phrasing_slur_engraver : public Slur_engraver
 {
   virtual SCM event_symbol () const;
   virtual bool double_property () const;
   virtual SCM grob_symbol () const;
   virtual const char* object_name () const;
+  virtual void set_melisma (bool);
 
 protected:
   void listen_phrasing_slur (Stream_event *);
@@ -34,7 +35,7 @@ protected:
 
 public:
   TRANSLATOR_DECLARATIONS (Phrasing_slur_engraver);
-  TRANSLATOR_INHERIT (Slur_proto_engraver);
+  TRANSLATOR_INHERIT (Slur_engraver);
 };
 
 SCM
@@ -65,10 +66,15 @@ Phrasing_slur_engraver::Phrasing_slur_engraver ()
 {
 }
 
+void
+Phrasing_slur_engraver::set_melisma (bool)
+{
+}
+
 void
 Phrasing_slur_engraver::listen_phrasing_slur (Stream_event *ev)
 {
-  Slur_proto_engraver::listen_slur (ev);
+  Slur_engraver::listen_slur (ev);
 }
 
 void
index 88495f4de2a1d5bc5e4ca808229f0bac7a838716..c0dedcf0d44c93585c4a2ae3d6f1b01051f5e3c2 100644 (file)
 */
 
 #include "context.hh"
-#include "slur-proto-engraver.hh"
+#include "directional-element-interface.hh"
+#include "international.hh"
+#include "note-column.hh"
+#include "pointer-group-interface.hh"
+#include "slur-engraver.hh"
+#include "slur.hh"
+#include "spanner.hh"
+#include "warn.hh"
 
 #include "translator.icc"
 
-class Slur_engraver : public Slur_proto_engraver
-{
-  virtual SCM event_symbol () const;
-  virtual bool double_property () const;
-  virtual SCM grob_symbol () const;
-  virtual const char * object_name () const;
-  virtual void set_melisma (bool);
-
-public:
-  TRANSLATOR_DECLARATIONS (Slur_engraver);
-  TRANSLATOR_INHERIT (Slur_proto_engraver);
-};
-
 SCM
 Slur_engraver::event_symbol () const
 {
@@ -98,3 +92,314 @@ ADD_TRANSLATOR (Slur_engraver,
                 /* write */
                 ""
                );
+
+void
+Slur_engraver::derived_mark () const
+{
+  for (vsize i = start_events_.size (); i--;)
+    {
+      scm_gc_mark (start_events_[i].slur_->self_scm ());
+      if (start_events_[i].note_)
+        scm_gc_mark (start_events_[i].note_->self_scm ());
+    }
+  for (vsize i = stop_events_.size (); i--;)
+    {
+      scm_gc_mark (stop_events_[i].slur_->self_scm ());
+      if (stop_events_[i].note_)
+        scm_gc_mark (stop_events_[i].note_->self_scm ());
+    }
+}
+
+void
+Slur_engraver::listen_slur (Stream_event *ev, Stream_event *note)
+{
+  Direction d = to_dir (ev->get_property ("span-direction"));
+  if (d == START)
+    start_events_.push_back (Event_info (ev, note));
+  else if (d == STOP)
+    stop_events_.push_back (Event_info (ev, note));
+  else ev->origin ()->warning (_f ("direction of %s invalid: %d",
+                                   ev->name ().c_str (),
+                                   int (d)));
+}
+
+void
+Slur_engraver::listen_note (Stream_event *ev)
+{
+  for (SCM arts = ev->get_property ("articulations");
+       scm_is_pair (arts); arts = scm_cdr (arts))
+    {
+      Stream_event *art = unsmob<Stream_event> (scm_car (arts));
+      if (art->in_event_class (event_symbol ()))
+        listen_slur (art, ev);
+    }
+}
+
+void
+Slur_engraver::acknowledge_note_column (Grob_info info)
+{
+  Grob *e = info.grob ();
+  for (vsize i = slurs_.size (); i--;)
+    Slur::add_column (slurs_[i], e);
+  for (vsize i = end_slurs_.size (); i--;)
+    Slur::add_column (end_slurs_[i], e);
+  // Now cater for slurs starting/ending at a notehead: those override
+  // the column bounds
+  if (note_slurs_[START].empty () && note_slurs_[STOP].empty ())
+    return;
+  extract_grob_set (e, "note-heads", heads);
+  for (vsize i = heads.size (); i--;)
+    {
+      if (Stream_event *ev =
+          unsmob<Stream_event> (heads[i]->get_property ("cause")))
+        for (LEFT_and_RIGHT (d))
+          {
+            std::pair<Note_slurs::const_iterator, Note_slurs::const_iterator> its
+              = note_slurs_[d].equal_range (ev);
+            for (Note_slurs::const_iterator it = its.first;
+                 it != its.second;
+                 ++it)
+              it->second->set_bound (d, heads[i]);
+          }
+    }
+}
+
+void
+Slur_engraver::acknowledge_extra_object (Grob_info info)
+{
+  objects_to_acknowledge_.push_back (info);
+}
+
+void
+Slur_engraver::acknowledge_inline_accidental (Grob_info info)
+{
+  acknowledge_extra_object (info);
+}
+
+void
+Slur_engraver::acknowledge_dots (Grob_info info)
+{
+  acknowledge_extra_object (info);
+}
+
+void
+Slur_engraver::acknowledge_fingering (Grob_info info)
+{
+  acknowledge_extra_object (info);
+}
+
+void
+Slur_engraver::acknowledge_tuplet_number (Grob_info info)
+{
+  acknowledge_extra_object (info);
+}
+
+void
+Slur_engraver::acknowledge_script (Grob_info info)
+{
+  if (!info.grob ()->internal_has_interface (ly_symbol2scm ("dynamic-interface")))
+    acknowledge_extra_object (info);
+}
+
+void
+Slur_engraver::acknowledge_text_script (Grob_info info)
+{
+  acknowledge_extra_object (info);
+}
+
+void
+Slur_engraver::acknowledge_end_tie (Grob_info info)
+{
+  acknowledge_extra_object (info);
+}
+
+void
+Slur_engraver::finalize ()
+{
+  for (vsize i = 0; i < slurs_.size (); i++)
+    {
+      slurs_[i]->warning (_f ("unterminated %s", object_name ()));
+      slurs_[i]->suicide ();
+    }
+  slurs_.clear ();
+}
+
+void
+Slur_engraver::create_slur (const string &spanner_id, Event_info evi, Grob *g_cause, Direction dir, bool left_broken)
+{
+  Grob *ccc = left_broken
+    ? unsmob<Grob> (get_property ("currentCommandColumn"))
+    : 0; // efficiency
+  SCM cause = evi.slur_ ? evi.slur_->self_scm () : g_cause->self_scm ();
+  Spanner *slur = make_spanner (grob_symbol (), cause);
+  slur->set_property ("spanner-id", ly_string2scm (spanner_id));
+  if (dir)
+    set_grob_direction (slur, dir);
+  if (left_broken)
+    slur->set_bound (LEFT, ccc);
+  slurs_.push_back (slur);
+  if (evi.note_)
+    note_slurs_[START].insert (Note_slurs::value_type (evi.note_, slur));
+
+  if (double_property ())
+  {
+    set_grob_direction (slur, DOWN);
+    slur = make_spanner (grob_symbol (), cause);
+    slur->set_property ("spanner-id", ly_string2scm (spanner_id));
+    set_grob_direction (slur, UP);
+    if (left_broken)
+      slur->set_bound (LEFT, ccc);
+    slurs_.push_back (slur);
+    if (evi.note_)
+      note_slurs_[START].insert(Note_slurs::value_type (evi.note_, slur));
+  }
+
+}
+
+bool
+Slur_engraver::can_create_slur (const string &id, vsize old_slurs, vsize *event_idx, Stream_event *ev)
+{
+  for (vsize j = slurs_.size (); j--;)
+    {
+      Grob *slur = slurs_[j];
+      Direction updown = to_dir (ev->get_property ("direction"));
+
+      // Check if we already have a slur with the same spanner-id.
+      if (id == robust_scm2string (slur->get_property ("spanner-id"), ""))
+        {
+          if (j < old_slurs)
+            {
+              // We already have an old slur, so give a warning
+              // and completely ignore the new slur.
+              ev->origin ()->warning (_f ("already have %s", object_name ()));
+              if (event_idx)
+                start_events_.erase (start_events_.begin () + (*event_idx));
+              return false;
+            }
+
+          // If this slur event has no direction, it will not
+          // contribute anything new to the existing slur(s), so
+          // we can ignore it.
+
+          if (!updown)
+            return false;
+
+          Stream_event *c = unsmob<Stream_event> (slur->get_property ("cause"));
+
+          if (!c)
+            {
+              slur->programming_error (_f ("%s without a cause", object_name ()));
+              return true;
+            }
+
+          Direction slur_dir = to_dir (c->get_property ("direction"));
+
+          // If the existing slur does not have a direction yet,
+          // we'd rather take the new one.
+
+          if (!slur_dir)
+            {
+              slur->suicide ();
+              slurs_.erase (slurs_.begin () + j);
+              return true;
+            }
+
+          // If the existing slur has the same direction as ours, drop ours
+
+          if (slur_dir == updown)
+            return false;
+        }
+    }
+  return true;
+}
+
+bool
+Slur_engraver::try_to_end (Event_info evi)
+{
+  string id = robust_scm2string (evi.slur_->get_property ("spanner-id"), "");
+
+  // Find the slurs that are ended with this event (by checking the spanner-id)
+  bool ended = false;
+  for (vsize j = slurs_.size (); j--;)
+    {
+      if (id == robust_scm2string (slurs_[j]->get_property ("spanner-id"), ""))
+        {
+          ended = true;
+          end_slurs_.push_back (slurs_[j]);
+          if (evi.note_)
+            note_slurs_[STOP].insert
+              (Note_slurs::value_type
+               (evi.note_, dynamic_cast <Spanner *> (slurs_[j])));
+          slurs_.erase (slurs_.begin () + j);
+        }
+    }
+  return ended;
+}
+
+void
+Slur_engraver::process_music ()
+{
+  for (vsize i = 0; i < stop_events_.size (); i++)
+    {
+      string id = robust_scm2string
+        (stop_events_[i].slur_->get_property ("spanner-id"), "");
+      bool ended = try_to_end (stop_events_[i]);
+      if (ended)
+        {
+          // Ignore redundant stop events for this id
+          for (vsize j = stop_events_.size (); --j > i;)
+            {
+              if (id == robust_scm2string
+                  (stop_events_[j].slur_->get_property ("spanner-id"), ""))
+                stop_events_.erase (stop_events_.begin () + j);
+            }
+        }
+      else
+        stop_events_[i].slur_->origin ()->warning (_f ("cannot end %s", object_name ()));
+    }
+
+  vsize old_slurs = slurs_.size ();
+  for (vsize i = start_events_.size (); i--;)
+    {
+      Stream_event *ev = start_events_[i].slur_;
+      string id = robust_scm2string (ev->get_property ("spanner-id"), "");
+      Direction updown = to_dir (ev->get_property ("direction"));
+
+      if (can_create_slur (id, old_slurs, &i, ev))
+        create_slur (id, start_events_[i], 0, updown, false);
+    }
+
+  set_melisma (slurs_.size ());
+}
+
+void
+Slur_engraver::stop_translation_timestep ()
+{
+  if (Grob *g = unsmob<Grob> (get_property ("currentCommandColumn")))
+    {
+      for (vsize i = 0; i < end_slurs_.size (); i++)
+        Slur::add_extra_encompass (end_slurs_[i], g);
+
+      if (!start_events_.size ())
+        for (vsize i = 0; i < slurs_.size (); i++)
+          Slur::add_extra_encompass (slurs_[i], g);
+    }
+
+  for (vsize i = 0; i < end_slurs_.size (); i++)
+    {
+      Spanner *s = dynamic_cast<Spanner *> (end_slurs_[i]);
+      if (!s->get_bound (RIGHT))
+        s->set_bound (RIGHT, unsmob<Grob> (get_property ("currentMusicalColumn")));
+      announce_end_grob (s, SCM_EOL);
+    }
+
+  for (vsize i = 0; i < objects_to_acknowledge_.size (); i++)
+    Slur::auxiliary_acknowledge_extra_object (objects_to_acknowledge_[i], slurs_, end_slurs_);
+
+  note_slurs_[LEFT].clear ();
+  note_slurs_[RIGHT].clear ();
+  objects_to_acknowledge_.clear ();
+  end_slurs_.clear ();
+  start_events_.clear ();
+  stop_events_.clear ();
+}
diff --git a/lily/slur-proto-engraver.cc b/lily/slur-proto-engraver.cc
deleted file mode 100644 (file)
index 6ba0df6..0000000
+++ /dev/null
@@ -1,352 +0,0 @@
-/*
-  This file is part of LilyPond, the GNU music typesetter.
-
-  Copyright (C) 2013--2015 Mike Solomon <mike@mikesolomon.org>
-
-  LilyPond is free software: you can redistribute it and/or modify
-  it under the terms of the GNU General Public License as published by
-  the Free Software Foundation, either version 3 of the License, or
-  (at your option) any later version.
-
-  LilyPond is distributed in the hope that it will be useful,
-  but WITHOUT ANY WARRANTY; without even the implied warranty of
-  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-  GNU General Public License for more details.
-
-  You should have received a copy of the GNU General Public License
-  along with LilyPond.  If not, see <http://www.gnu.org/licenses/>.
-*/
-
-#include "engraver.hh"
-
-#include "context.hh"
-#include "directional-element-interface.hh"
-#include "international.hh"
-#include "note-column.hh"
-#include "pointer-group-interface.hh"
-#include "slur.hh"
-#include "slur-proto-engraver.hh"
-#include "spanner.hh"
-#include "stream-event.hh"
-#include "warn.hh"
-
-#include "translator.icc"
-
-void
-Slur_proto_engraver::derived_mark () const
-{
-  for (vsize i = start_events_.size (); i--;)
-    {
-      scm_gc_mark (start_events_[i].slur_->self_scm ());
-      if (start_events_[i].note_)
-        scm_gc_mark (start_events_[i].note_->self_scm ());
-    }
-  for (vsize i = stop_events_.size (); i--;)
-    {
-      scm_gc_mark (stop_events_[i].slur_->self_scm ());
-      if (stop_events_[i].note_)
-        scm_gc_mark (stop_events_[i].note_->self_scm ());
-    }
-}
-
-void
-Slur_proto_engraver::listen_slur (Stream_event *ev, Stream_event *note)
-{
-  Direction d = to_dir (ev->get_property ("span-direction"));
-  if (d == START)
-    start_events_.push_back (Event_info (ev, note));
-  else if (d == STOP)
-    stop_events_.push_back (Event_info (ev, note));
-  else ev->origin ()->warning (_f ("direction of %s invalid: %d",
-                                   ev->name ().c_str (),
-                                   int (d)));
-}
-
-void
-Slur_proto_engraver::listen_note (Stream_event *ev)
-{
-  for (SCM arts = ev->get_property ("articulations");
-       scm_is_pair (arts); arts = scm_cdr (arts))
-    {
-      Stream_event *art = unsmob<Stream_event> (scm_car (arts));
-      if (art->in_event_class (event_symbol ()))
-        listen_slur (art, ev);
-    }
-}
-
-void
-Slur_proto_engraver::acknowledge_note_column (Grob_info info)
-{
-  Grob *e = info.grob ();
-  for (vsize i = slurs_.size (); i--;)
-    Slur::add_column (slurs_[i], e);
-  for (vsize i = end_slurs_.size (); i--;)
-    Slur::add_column (end_slurs_[i], e);
-  // Now cater for slurs starting/ending at a notehead: those override
-  // the column bounds
-  if (note_slurs_[START].empty () && note_slurs_[STOP].empty ())
-    return;
-  extract_grob_set (e, "note-heads", heads);
-  for (vsize i = heads.size (); i--;)
-    {
-      if (Stream_event *ev =
-          unsmob<Stream_event> (heads[i]->get_property ("cause")))
-        for (LEFT_and_RIGHT (d))
-          {
-            std::pair<Note_slurs::const_iterator, Note_slurs::const_iterator> its
-              = note_slurs_[d].equal_range (ev);
-            for (Note_slurs::const_iterator it = its.first;
-                 it != its.second;
-                 ++it)
-              it->second->set_bound (d, heads[i]);
-          }
-    }
-}
-
-void
-Slur_proto_engraver::acknowledge_extra_object (Grob_info info)
-{
-  objects_to_acknowledge_.push_back (info);
-}
-
-void
-Slur_proto_engraver::acknowledge_inline_accidental (Grob_info info)
-{
-  acknowledge_extra_object (info);
-}
-
-void
-Slur_proto_engraver::acknowledge_dots (Grob_info info)
-{
-  acknowledge_extra_object (info);
-}
-
-void
-Slur_proto_engraver::acknowledge_fingering (Grob_info info)
-{
-  acknowledge_extra_object (info);
-}
-
-void
-Slur_proto_engraver::acknowledge_tuplet_number (Grob_info info)
-{
-  acknowledge_extra_object (info);
-}
-
-void
-Slur_proto_engraver::acknowledge_script (Grob_info info)
-{
-  if (!info.grob ()->internal_has_interface (ly_symbol2scm ("dynamic-interface")))
-    acknowledge_extra_object (info);
-}
-
-void
-Slur_proto_engraver::acknowledge_text_script (Grob_info info)
-{
-  acknowledge_extra_object (info);
-}
-
-void
-Slur_proto_engraver::acknowledge_end_tie (Grob_info info)
-{
-  acknowledge_extra_object (info);
-}
-
-void
-Slur_proto_engraver::finalize ()
-{
-  for (vsize i = 0; i < slurs_.size (); i++)
-    {
-      slurs_[i]->warning (_f ("unterminated %s", object_name ()));
-      slurs_[i]->suicide ();
-    }
-  slurs_.clear ();
-}
-
-void
-Slur_proto_engraver::create_slur (const string &spanner_id, Event_info evi, Grob *g_cause, Direction dir, bool left_broken)
-{
-  Grob *ccc = left_broken
-    ? unsmob<Grob> (get_property ("currentCommandColumn"))
-    : 0; // efficiency
-  SCM cause = evi.slur_ ? evi.slur_->self_scm () : g_cause->self_scm ();
-  Spanner *slur = make_spanner (grob_symbol (), cause);
-  slur->set_property ("spanner-id", ly_string2scm (spanner_id));
-  if (dir)
-    set_grob_direction (slur, dir);
-  if (left_broken)
-    slur->set_bound (LEFT, ccc);
-  slurs_.push_back (slur);
-  if (evi.note_)
-    note_slurs_[START].insert (Note_slurs::value_type (evi.note_, slur));
-
-  if (double_property ())
-  {
-    set_grob_direction (slur, DOWN);
-    slur = make_spanner (grob_symbol (), cause);
-    slur->set_property ("spanner-id", ly_string2scm (spanner_id));
-    set_grob_direction (slur, UP);
-    if (left_broken)
-      slur->set_bound (LEFT, ccc);
-    slurs_.push_back (slur);
-    if (evi.note_)
-      note_slurs_[START].insert(Note_slurs::value_type (evi.note_, slur));
-  }
-
-}
-
-bool
-Slur_proto_engraver::can_create_slur (const string &id, vsize old_slurs, vsize *event_idx, Stream_event *ev)
-{
-  for (vsize j = slurs_.size (); j--;)
-    {
-      Grob *slur = slurs_[j];
-      Direction updown = to_dir (ev->get_property ("direction"));
-
-      // Check if we already have a slur with the same spanner-id.
-      if (id == robust_scm2string (slur->get_property ("spanner-id"), ""))
-        {
-          if (j < old_slurs)
-            {
-              // We already have an old slur, so give a warning
-              // and completely ignore the new slur.
-              ev->origin ()->warning (_f ("already have %s", object_name ()));
-              if (event_idx)
-                start_events_.erase (start_events_.begin () + (*event_idx));
-              return false;
-            }
-
-          // If this slur event has no direction, it will not
-          // contribute anything new to the existing slur(s), so
-          // we can ignore it.
-
-          if (!updown)
-            return false;
-
-          Stream_event *c = unsmob<Stream_event> (slur->get_property ("cause"));
-
-          if (!c)
-            {
-              slur->programming_error (_f ("%s without a cause", object_name ()));
-              return true;
-            }
-
-          Direction slur_dir = to_dir (c->get_property ("direction"));
-
-          // If the existing slur does not have a direction yet,
-          // we'd rather take the new one.
-
-          if (!slur_dir)
-            {
-              slur->suicide ();
-              slurs_.erase (slurs_.begin () + j);
-              return true;
-            }
-
-          // If the existing slur has the same direction as ours, drop ours
-
-          if (slur_dir == updown)
-            return false;
-        }
-    }
-  return true;
-}
-
-bool
-Slur_proto_engraver::try_to_end (Event_info evi)
-{
-  string id = robust_scm2string (evi.slur_->get_property ("spanner-id"), "");
-
-  // Find the slurs that are ended with this event (by checking the spanner-id)
-  bool ended = false;
-  for (vsize j = slurs_.size (); j--;)
-    {
-      if (id == robust_scm2string (slurs_[j]->get_property ("spanner-id"), ""))
-        {
-          ended = true;
-          end_slurs_.push_back (slurs_[j]);
-          if (evi.note_)
-            note_slurs_[STOP].insert
-              (Note_slurs::value_type
-               (evi.note_, dynamic_cast <Spanner *> (slurs_[j])));
-          slurs_.erase (slurs_.begin () + j);
-        }
-    }
-  return ended;
-}
-
-void
-Slur_proto_engraver::process_music ()
-{
-  for (vsize i = 0; i < stop_events_.size (); i++)
-    {
-      string id = robust_scm2string
-        (stop_events_[i].slur_->get_property ("spanner-id"), "");
-      bool ended = try_to_end (stop_events_[i]);
-      if (ended)
-        {
-          // Ignore redundant stop events for this id
-          for (vsize j = stop_events_.size (); --j > i;)
-            {
-              if (id == robust_scm2string
-                  (stop_events_[j].slur_->get_property ("spanner-id"), ""))
-                stop_events_.erase (stop_events_.begin () + j);
-            }
-        }
-      else
-        stop_events_[i].slur_->origin ()->warning (_f ("cannot end %s", object_name ()));
-    }
-
-  vsize old_slurs = slurs_.size ();
-  for (vsize i = start_events_.size (); i--;)
-    {
-      Stream_event *ev = start_events_[i].slur_;
-      string id = robust_scm2string (ev->get_property ("spanner-id"), "");
-      Direction updown = to_dir (ev->get_property ("direction"));
-
-      if (can_create_slur (id, old_slurs, &i, ev))
-        create_slur (id, start_events_[i], 0, updown, false);
-    }
-
-  set_melisma (slurs_.size ());
-}
-
-void
-Slur_proto_engraver::set_melisma (bool)
-{
-}
-
-void
-Slur_proto_engraver::stop_translation_timestep ()
-{
-  if (Grob *g = unsmob<Grob> (get_property ("currentCommandColumn")))
-    {
-      for (vsize i = 0; i < end_slurs_.size (); i++)
-        Slur::add_extra_encompass (end_slurs_[i], g);
-
-      if (!start_events_.size ())
-        for (vsize i = 0; i < slurs_.size (); i++)
-          Slur::add_extra_encompass (slurs_[i], g);
-    }
-
-  for (vsize i = 0; i < end_slurs_.size (); i++)
-    {
-      Spanner *s = dynamic_cast<Spanner *> (end_slurs_[i]);
-      if (!s->get_bound (RIGHT))
-        s->set_bound (RIGHT, unsmob<Grob> (get_property ("currentMusicalColumn")));
-      announce_end_grob (s, SCM_EOL);
-    }
-
-  for (vsize i = 0; i < objects_to_acknowledge_.size (); i++)
-    Slur::auxiliary_acknowledge_extra_object (objects_to_acknowledge_[i], slurs_, end_slurs_);
-
-  note_slurs_[LEFT].clear ();
-  note_slurs_[RIGHT].clear ();
-  objects_to_acknowledge_.clear ();
-  end_slurs_.clear ();
-  start_events_.clear ();
-  stop_events_.clear ();
-}
-
-// no ADD_ACKNOWLEDGER / ADD_TRANSLATOR macro calls
-// since this class is abstract