Fix some bugs in the dynamic engraver and PostScript backend

[lilypond.git] / lily / extender-engraver.cc

diff --git a/lily/extender-engraver.cc b/lily/extender-engraver.cc
index cc2939a030fd151c96b6da39fd58acb3cacc9e36..d3b2e90972facbdee6b1579468aedafb9900d345 100644 (file)
--- a/lily/extender-engraver.cc
+++ b/lily/extender-engraver.cc
@@ -2,40 +2,40 @@
   extender-engraver.cc -- implement Extender_engraver
 
   source file of the GNU LilyPond music typesetter
-  
-  (c) 1999--2004 Glen Prideaux <glenprideaux@iname.com>,
-                  Han-Wen Nienhuys <hanwen@cs.uu.nl>,
-                  Jan Nieuwenhuizen <janneke@gnu.org>
+
+  (c) 1999--2006 Glen Prideaux <glenprideaux@iname.com>,
+  Han-Wen Nienhuys <hanwen@xs4all.nl>,
+  Jan Nieuwenhuizen <janneke@gnu.org>
 */
 
-#include "warn.hh"
-#include "lyric-extender.hh"
-#include "item.hh"
-#include "engraver.hh"
 #include "context.hh"
-#include "group-interface.hh"
+#include "engraver.hh"
+#include "international.hh"
+#include "item.hh"
+#include "lyric-extender.hh"
+#include "note-head.hh"
+#include "pointer-group-interface.hh"
+#include "warn.hh"
+
+void completize_extender (Spanner *sp);
 
 class Extender_engraver : public Engraver
 {
-  Music* ev_;
-  Spanner* extender_;
-  Spanner * pending_extender_;  
+  Music *ev_;
+  Spanner *extender_;
+  Spanner *pending_extender_;
+
 public:
   TRANSLATOR_DECLARATIONS (Extender_engraver);
 
 protected:
-  virtual void acknowledge_grob (Grob_info);
+  DECLARE_ACKNOWLEDGER (lyric_syllable);
   virtual void finalize ();
-  virtual bool try_music (Music*);
-  virtual void stop_translation_timestep ();
-  virtual void process_music ();
-private:
-
+  virtual bool try_music (Music *);
+  void stop_translation_timestep ();
+  void process_music ();
 };
 
-
-
-
 Extender_engraver::Extender_engraver ()
 {
   extender_ = 0;
@@ -44,66 +44,60 @@ Extender_engraver::Extender_engraver ()
 }
 
 bool
-Extender_engraver::try_music (Music* r)
+Extender_engraver::try_music (Music *r)
 {
-  if (ev_)
-    return false;
-
-  ev_ = r;
-  return true;
+  if (!ev_)
+    {
+      ev_ = r;
+      return true;
+    }
+  return false;
 }
 
-
 void
 Extender_engraver::process_music ()
 {
   if (ev_)
-    {
-      extender_ = make_spanner ("LyricExtender");
-      announce_grob (extender_, ev_->self_scm ());
-    }
+    extender_ = make_spanner ("LyricExtender", ev_->self_scm ());
 }
 
-
 void
-Extender_engraver::acknowledge_grob (Grob_info i)
+Extender_engraver::acknowledge_lyric_syllable (Grob_info i)
 {
-  Item * item =  dynamic_cast<Item*> (i.grob_);
+  Item *item = i.item ();
+  if (extender_)
+    extender_->set_bound (LEFT, item);
 
-  if (item
-      && item->internal_has_interface (ly_symbol2scm ("lyric-syllable-interface")))
+  if (pending_extender_)
     {
-      if (extender_)
-       extender_->set_bound (LEFT, item);
-
-      if (pending_extender_)
-       pending_extender_->set_bound (RIGHT, item);
+      pending_extender_->set_object ("next", item->self_scm ());
+      completize_extender (pending_extender_);
+      pending_extender_ = 0;
     }
 }
 
 void
 Extender_engraver::stop_translation_timestep ()
 {
-  if (pending_extender_ && pending_extender_->get_bound (RIGHT))
-    {
-      typeset_grob (pending_extender_);
-      pending_extender_ = 0;
-    }
-
   if (extender_ || pending_extender_)
     {
-      Context *voice = get_voice_to_lyrics (daddy_context_);
-      Grob* h =  (voice) ? get_current_note_head (voice) : 0;
+      Context *voice = get_voice_to_lyrics (context ());
+      Grob *h = voice ? get_current_note_head (voice) : 0;
 
       if (h)
        {
          if (extender_)
-           Pointer_group_interface::add_grob (extender_,
-                                              ly_symbol2scm ("heads"), h);
+           {
+             Pointer_group_interface::add_grob (extender_,
+                                                ly_symbol2scm ("heads"), h);
+           }
+
          if (pending_extender_)
-           Pointer_group_interface::add_grob (pending_extender_,
-                                              ly_symbol2scm ("heads"), h);
-       }           
+           {
+             Pointer_group_interface::add_grob (pending_extender_,
+                                                ly_symbol2scm ("heads"), h);
+           }
+       }
 
       if (extender_)
        {
@@ -116,22 +110,16 @@ Extender_engraver::stop_translation_timestep ()
 }
 
 void
-completize_extender (Spanner* sp)
+completize_extender (Spanner *sp)
 {
   if (!sp->get_bound (RIGHT))
     {
-      SCM heads = sp->get_property ("heads");
-      if (ly_pair_p (heads))
-       {
-         Item* it = dynamic_cast<Item*> (unsmob_grob (ly_car (heads)));
-         if (it)
-           sp->set_bound (RIGHT, it);
-       }
+      extract_item_set (sp, "heads", heads);
+      if (heads.size ())
+       sp->set_bound (RIGHT, heads.back ());
     }
 }
 
-  
-
 void
 Extender_engraver::finalize ()
 {
@@ -141,7 +129,6 @@ Extender_engraver::finalize ()
 
       if (!extender_->get_bound (RIGHT))
        extender_->warning (_ ("unterminated extender"));
-      typeset_grob (extender_);
       extender_ = 0;
     }
 
@@ -150,20 +137,17 @@ Extender_engraver::finalize ()
       completize_extender (pending_extender_);
 
       if (!pending_extender_->get_bound (RIGHT))
-         pending_extender_->warning (_("unterminated extender"));
-      typeset_grob (pending_extender_);
-      pending_extender_ =0;
+       pending_extender_->warning (_ ("unterminated extender"));
+      pending_extender_ = 0;
     }
 }
 
+#include "translator.icc"
 
-
-
-
-ENTER_DESCRIPTION (Extender_engraver,
-/* descr */       "Create lyric extenders",
-/* creats*/       "LyricExtender",
-/* accepts */     "extender-event",
-/* acks  */       "lyric-syllable-interface",
-/* reads */       "",
-/* write */       "");
+ADD_ACKNOWLEDGER (Extender_engraver, lyric_syllable);
+ADD_TRANSLATOR (Extender_engraver,
+               /* doc */ "Create lyric extenders",
+               /* create */ "LyricExtender",
+               /* accept */ "extender-event",
+               /* read */ "",
+               /* write */ "");