Fix some bugs in the dynamic engraver and PostScript backend

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

diff --git a/lily/lyric-engraver.cc b/lily/lyric-engraver.cc
index 6b4ac2309a92729cd65315a5abc14bd13e9ba2d1..176c9b7166157228e5eb82571d6685a5236bd671 100644 (file)
--- a/lily/lyric-engraver.cc
+++ b/lily/lyric-engraver.cc
@@ -3,51 +3,53 @@
 
   source file of the GNU LilyPond music typesetter
 
-  (c) 1997--2004 Han-Wen Nienhuys <hanwen@cs.uu.nl>
+  (c) 1997--2006 Han-Wen Nienhuys <hanwen@xs4all.nl>
   Jan Nieuwenhuizen <janneke@gnu.org>
 */
 
-#include "engraver.hh"
-#include "event.hh"
-#include "item.hh"
 #include "context.hh"
+#include "engraver.hh"
 #include "font-metric.hh"
+#include "item.hh"
+#include "multi-measure-rest.hh"
 #include "note-head.hh"
+#include "rest.hh"
 
 /**
-   Generate texts for lyric syllables.  We only do one lyric at a time.  
+   Generate texts for lyric syllables.  We only do one lyric at a time.
    Multiple copies of this engraver should be used to do multiple voices.
- */
-class Lyric_engraver : public Engraver 
+*/
+class Lyric_engraver : public Engraver
 {
 protected:
-  virtual void stop_translation_timestep ();
+  void stop_translation_timestep ();
   virtual bool try_music (Music *);
-  virtual void process_music ();
-  
+  void process_music ();
+
 public:
   TRANSLATOR_DECLARATIONS (Lyric_engraver);
+
 private:
-  Music * event_;
-  Item* text_;
+  Music *event_;
+  Item *text_;
+  Item *last_text_;
 
-  Context* get_voice_context ();
+  Context *get_voice_context ();
 };
 
 Lyric_engraver::Lyric_engraver ()
 {
   text_ = 0;
+  last_text_ = 0;
   event_ = 0;
 }
 
 bool
-Lyric_engraver::try_music (Music*r)
+Lyric_engraver::try_music (Music *r)
 {
-  if (r->is_mus_type ("lyric-event"))
+  if (!event_)
     {
-      if (event_)
-       return false;
-      event_ =r;
+      event_ = r;
       return true;
     }
   return false;
@@ -58,34 +60,42 @@ Lyric_engraver::process_music ()
 {
   if (event_)
     {
-      text_=  make_item ("LyricText",event_->self_scm ());
-      
-      text_->set_property ("text", event_->get_property ("text"));
+      SCM text = event_->get_property ("text");
+
+      if (ly_is_equal (text, scm_makfrom0str (" ")))
+       {
+         if (last_text_)
+           last_text_->set_property ("self-alignment-X", scm_from_int (LEFT));
+       }
+      else
+       {
+         text_ = make_item ("LyricText", event_->self_scm ());
+         text_->set_property ("text", text);
+       }
     }
 }
 
-
-Context*
+Context *
 get_voice_to_lyrics (Context *lyrics)
 {
   SCM avc = lyrics->get_property ("associatedVoiceContext");
-  if  (Context *c = unsmob_context (avc))
+  if (Context *c = unsmob_context (avc))
     return c;
 
   SCM voice_name = lyrics->get_property ("associatedVoice");
-  String nm = lyrics->id_string ();
+  string nm = lyrics->id_string ();
 
-  if (ly_c_string_p (voice_name))
+  if (scm_is_string (voice_name))
     nm = ly_scm2string (voice_name);
   else
     {
-      int idx = nm.index_last ('-');
-      if (idx >= 0)
-       nm = nm.left_string (idx);
+      ssize idx = nm.rfind ('-');
+      if (idx != NPOS)
+       nm = nm.substr (0, idx);
     }
 
   Context *parent = lyrics;
-  Context *voice = 0; 
+  Context *voice = 0;
   while (parent && !voice)
     {
       voice = find_context_below (parent, ly_symbol2scm ("Voice"), nm);
@@ -96,7 +106,7 @@ get_voice_to_lyrics (Context *lyrics)
     return voice;
 
   parent = lyrics;
-  voice = 0; 
+  voice = 0;
   while (parent && !voice)
     {
       voice = find_context_below (parent, ly_symbol2scm ("Voice"), "");
@@ -107,19 +117,19 @@ get_voice_to_lyrics (Context *lyrics)
 }
 
 Grob *
-get_current_note_head (Context * voice)
+get_current_note_head (Context *voice)
 {
   for (SCM s = voice->get_property ("busyGrobs");
-       ly_c_pair_p (s); s = ly_cdr (s))
+       scm_is_pair (s); s = scm_cdr (s))
     {
-      Item*g = dynamic_cast<Item*> (unsmob_grob (ly_cdar (s)));
-         
+      Item *g = dynamic_cast<Item *> (unsmob_grob (scm_cdar (s)));
+
       if (g && !g->get_column ()
          && Note_head::has_interface (g))
        return g;
     }
-         
-  return 0;  
+
+  return 0;
 }
 
 void
@@ -127,7 +137,7 @@ Lyric_engraver::stop_translation_timestep ()
 {
   if (text_)
     {
-      Context * voice = get_voice_to_lyrics (context ());
+      Context *voice = get_voice_to_lyrics (context ());
 
       if (voice)
        {
@@ -137,20 +147,21 @@ Lyric_engraver::stop_translation_timestep ()
            {
              text_->set_parent (head, X_AXIS);
              if (melisma_busy (voice))
-               text_->set_property ("self-alignment-X", scm_int2num (LEFT)); 
+               text_->set_property ("self-alignment-X", scm_from_int (LEFT));
            }
        }
-      
-      text_ =0;
+
+      last_text_ = text_;
+      text_ = 0;
     }
-  event_ =0;
+  event_ = 0;
 }
 
+#include "translator.icc"
 
-ENTER_DESCRIPTION (Lyric_engraver,
-/* descr */       "",
-/* creats*/       "LyricText",
-/* accepts */     "lyric-event",
-/* acks  */      "",
-/* reads */       "",
-/* write */       "");
+ADD_TRANSLATOR (Lyric_engraver,
+               /* doc */ "",
+               /* create */ "LyricText",
+               /* accept */ "lyric-event",
+               /* read */ "",
+               /* write */ "");