Fix some bugs in the dynamic engraver and PostScript backend

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

diff --git a/lily/stem-engraver.cc b/lily/stem-engraver.cc
index 36e7661242c5c0edb3be981ef39f9c54d820b1be..797406b4fbc062d28aa0cc9af6f152d3435073ce 100644 (file)
--- a/lily/stem-engraver.cc
+++ b/lily/stem-engraver.cc
@@ -3,12 +3,15 @@
 
   source file of the GNU LilyPond music typesetter
 
-  (c) 1997--2005 Han-Wen Nienhuys <hanwen@cs.uu.nl>
+  (c) 1997--2006 Han-Wen Nienhuys <hanwen@xs4all.nl>
 */
 
+#include "engraver.hh"
+
 #include "context.hh"
 #include "directional-element-interface.hh"
-#include "engraver.hh"
+#include "duration.hh"
+#include "international.hh"
 #include "item.hh"
 #include "misc.hh"
 #include "rhythmic-head.hh"
@@ -16,7 +19,6 @@
 #include "staff-symbol-referencer.hh"
 #include "stem-tremolo.hh"
 #include "stem.hh"
-#include "duration.hh"
 
 /**
    Make stems upon receiving noteheads.
@@ -33,8 +35,8 @@ class Stem_engraver : public Engraver
 protected:
   void make_stem (Grob_info);
 
-  virtual void acknowledge_grob (Grob_info);
-  virtual void stop_translation_timestep ();
+  DECLARE_ACKNOWLEDGER (rhythmic_head);
+  void stop_translation_timestep ();
   virtual bool try_music (Music *);
 };
 
@@ -51,7 +53,7 @@ Stem_engraver::make_stem (Grob_info gi)
 {
   /* Announce the cause of the head as cause of the stem.  The
      stem needs a rhythmic structure to fit it into a beam.  */
-  stem_ = make_item ("Stem", gi.music_cause ()->self_scm ());
+  stem_ = make_item ("Stem", gi.grob ()->self_scm ());
 
   /*
     we take the duration log from the Event, since the duration-log
@@ -59,8 +61,8 @@ Stem_engraver::make_stem (Grob_info gi)
   */
   Music *music = gi.music_cause ();
   Duration *dur = unsmob_duration (music->get_property ("duration"));
-  
-  stem_->set_property ("duration-log", dur ? scm_int2num (dur->duration_log ()) : 0);
+
+  stem_->set_property ("duration-log", dur ? scm_from_int (dur->duration_log ()) : 0);
 
   if (tremolo_ev_)
     {
@@ -82,7 +84,7 @@ Stem_engraver::make_stem (Grob_info gi)
            requested_type = 8;
        }
       else
-       context ()->set_property ("tremoloFlags", scm_int2num (requested_type));
+       context ()->set_property ("tremoloFlags", scm_from_int (requested_type));
 
       int tremolo_flags = intlog2 (requested_type) - 2
        - (dur->duration_log () > 2 ? dur->duration_log () - 2 : 0);
@@ -98,7 +100,7 @@ Stem_engraver::make_stem (Grob_info gi)
 
          /* The number of tremolo flags is the number of flags of the
             tremolo-type minus the number of flags of the note itself.  */
-         tremolo_->set_property ("flag-count", scm_int2num (tremolo_flags));
+         tremolo_->set_property ("flag-count", scm_from_int (tremolo_flags));
          tremolo_->set_parent (stem_, X_AXIS);
          stem_->set_object ("tremolo-flag", tremolo_->self_scm ());
          tremolo_->set_object ("stem", stem_->self_scm ());
@@ -107,33 +109,30 @@ Stem_engraver::make_stem (Grob_info gi)
 }
 
 void
-Stem_engraver::acknowledge_grob (Grob_info gi)
+Stem_engraver::acknowledge_rhythmic_head (Grob_info gi)
 {
-  if (Rhythmic_head::has_interface (gi.grob ()))
-    {
-      if (Rhythmic_head::get_stem (gi.grob ()))
-       return;
-
-      Music *cause = gi.music_cause ();
-      if (!cause)
-       return;
-      Duration *d = unsmob_duration (cause->get_property ("duration"));
-      if (!d)
-       return ;
-      
-      if (!stem_)
-       make_stem (gi);
-      
-      if (Stem::duration_log (stem_) != d->duration_log ())
-       {
-         // FIXME: 
-         gi.music_cause ()->origin ()->warning (_f ("adding note head to incompatible stem (type = %d)",
-                                                    1 << Stem::duration_log (stem_)));
-         gi.music_cause ()->origin ()->warning (_f ("maybe input should specify polyphonic voices"));
-       }
+  if (Rhythmic_head::get_stem (gi.grob ()))
+    return;
+
+  Music *cause = gi.music_cause ();
+  if (!cause)
+    return;
+  Duration *d = unsmob_duration (cause->get_property ("duration"));
+  if (!d)
+    return;
 
-      Stem::add_head (stem_, gi.grob ());
+  if (!stem_)
+    make_stem (gi);
+
+  if (Stem::duration_log (stem_) != d->duration_log ())
+    {
+      // FIXME: 
+      gi.music_cause ()->origin ()->warning (_f ("adding note head to incompatible stem (type = %d)",
+                                                1 << Stem::duration_log (stem_)));
+      gi.music_cause ()->origin ()->warning (_f ("maybe input should specify polyphonic voices"));
     }
+
+  Stem::add_head (stem_, gi.grob ());
 }
 
 void
@@ -171,11 +170,19 @@ Stem_engraver::try_music (Music *m)
   return false;
 }
 
+#include "translator.icc"
+
+ADD_ACKNOWLEDGER (Stem_engraver, rhythmic_head);
+
 ADD_TRANSLATOR (Stem_engraver,
-               /* descr */ "Create stems and single-stem tremolos.  It also works together with "
+               /* doc */ "Create stems and single-stem tremolos.  It also works together with "
                "the beam engraver for overriding beaming.",
-               /* creats*/ "Stem StemTremolo",
-               /* accepts */ "tremolo-event",
-               /* acks  */ "rhythmic-head-interface",
-               /* reads */ "tremoloFlags stemLeftBeamCount stemRightBeamCount",
+               /* create */
+               "Stem "
+               "StemTremolo ",
+               /* accept */ "tremolo-event",
+               /* read */
+               "tremoloFlags "
+               "stemLeftBeamCount "
+               "stemRightBeamCount ",
                /* write */ "");