*** empty log message ***

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

diff --git a/lily/stem-engraver.cc b/lily/stem-engraver.cc
index 216e994bc5a2940d98571f9e5849457b07fb125c..69d47c70acb659f4ae58a05369334f9df50a8783 100644 (file)
--- a/lily/stem-engraver.cc
+++ b/lily/stem-engraver.cc
@@ -3,7 +3,7 @@
 
   source file of the GNU LilyPond music typesetter
 
-  (c)  1997--2002 Han-Wen Nienhuys <hanwen@cs.uu.nl>
+  (c) 1997--2004 Han-Wen Nienhuys <hanwen@cs.uu.nl>
 */
 
 #include "staff-symbol-referencer.hh"
@@ -13,7 +13,8 @@
 #include "misc.hh"
 #include "stem-tremolo.hh"
 #include "item.hh"
-#include "translator-group.hh"
+#include "context.hh"
+
 #include "engraver.hh"
 
 
@@ -23,7 +24,7 @@
  */
 class Stem_engraver : public Engraver
 {
-  TRANSLATOR_DECLARATIONS(Stem_engraver);
+  TRANSLATOR_DECLARATIONS (Stem_engraver);
 protected:
   virtual void acknowledge_grob (Grob_info);
   virtual void stop_translation_timestep ();
@@ -32,16 +33,16 @@ protected:
 private:
   Grob  *stem_;
   Grob *tremolo_;
-  Music *rhythmic_req_;
-  Music* tremolo_req_;
+  Music *rhythmic_ev_;
+  Music* tremolo_ev_;
 };
 
 Stem_engraver::Stem_engraver ()
 {
-  tremolo_req_ = 0;
+  tremolo_ev_ = 0;
   stem_ = 0;
   tremolo_ = 0;
-  rhythmic_req_ =0;
+  rhythmic_ev_ =0;
 }
 
 
@@ -60,65 +61,71 @@ Stem_engraver::acknowledge_grob (Grob_info i)
 
       Music * m = i.music_cause ();
       if (m->is_mus_type ("rhythmic-event"))
-       duration_log = unsmob_duration (m->get_mus_property ("duration"))-> duration_log (); 
+       duration_log = unsmob_duration (m->get_property ("duration"))-> duration_log (); 
       
       if (!stem_) 
        {
-         stem_ = new Item (get_property ("Stem"));
+         stem_ = make_item ("Stem");
 
-         stem_->set_grob_property ("duration-log", gh_int2scm (duration_log));
+         stem_->set_property ("duration-log", gh_int2scm (duration_log));
 
-         if (tremolo_req_)
+         if (tremolo_ev_)
            {
              /*
                Stem tremolo is never applied to a note by default,
-               is must me requested.  But there is a default for the
+               is must me evuested.  But there is a default for the
                tremolo value:
 
                   c4:8 c c:
 
                the first and last (quarter) note bothe get one tremolo flag.
               */
-             int requested_type = gh_scm2int (tremolo_req_->get_mus_property ("tremolo-type"));
+             int requested_type = gh_scm2int (tremolo_ev_->get_property ("tremolo-type"));
              SCM f = get_property ("tremoloFlags");
-             if (!requested_type && gh_number_p (f))
-               requested_type = gh_scm2int (f);
+             if (!requested_type)
+               if (gh_number_p (f))
+                 requested_type = gh_scm2int (f);
+               else
+                 requested_type = 8; 
              else
-               daddy_trans_->set_property ("tremoloFlags", gh_int2scm (requested_type));
+               daddy_context_->set_property ("tremoloFlags", gh_int2scm (requested_type));
 
              int tremolo_flags = intlog2 (requested_type) - 2
                - (duration_log > 2 ? duration_log - 2 : 0);
              if (tremolo_flags <= 0)
                {
-                 tremolo_req_->origin()->warning (_("tremolo duration is too long"));
+                 tremolo_ev_->origin ()->warning (_("tremolo duration is too long"));
                  tremolo_flags = 0;
                }
 
              if (tremolo_flags)
                {
-                 tremolo_ = new Item (get_property ("StemTremolo"));
-                 announce_grob(tremolo_, tremolo_req_->self_scm());
+                 tremolo_ = make_item ("StemTremolo");
+                 announce_grob (tremolo_, tremolo_ev_->self_scm ());
 
                  /*
                    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_grob_property ("flag-count",
-                                               gh_int2scm (tremolo_flags));
+                 tremolo_->set_property ("flag-count",
+                                              gh_int2scm (tremolo_flags));
                  tremolo_->set_parent (stem_, X_AXIS);
+                 stem_->set_property ("tremolo-flag", tremolo_->self_scm ());
                }
            }
 
          /*
            We announce the cause of the head as cause of the stem.
            The stem needs a rhythmic structure to fit it into a beam.  */
-         announce_grob(stem_, i.music_cause ()->self_scm());
+         announce_grob (stem_, i.music_cause ()->self_scm ());
        }
 
       if (Stem::duration_log (stem_) != duration_log)
        {
-         i.music_cause ()->origin ()->warning (_f ("Adding note head to incompatible stem (type = %d)", 1 <<  Stem::duration_log (stem_)));
+         i.music_cause ()->origin ()->warning (_f ("Adding note head to incompatible stem (type = %d)", 1 <<  Stem::duration_log (stem_))
+                                               + _f ("Don't you want polyphonic voices instead?")
+                                               );
        }
 
       Stem::add_head (stem_,h);
@@ -137,17 +144,20 @@ Stem_engraver::stop_translation_timestep ()
 
   if (stem_)
     {
+      /*
+       toDO: junk these properties.
+       */
       SCM prop = get_property ("stemLeftBeamCount");
       if (gh_number_p (prop))
        {
          Stem::set_beaming (stem_,gh_scm2int (prop),LEFT);
-         daddy_trans_->unset_property (ly_symbol2scm ("stemLeftBeamCount"));
+         daddy_context_->unset_property (ly_symbol2scm ("stemLeftBeamCount"));
        }
       prop = get_property ("stemRightBeamCount");
       if (gh_number_p (prop))
        {
          Stem::set_beaming (stem_,gh_scm2int (prop), RIGHT);
-         daddy_trans_->unset_property (ly_symbol2scm ("stemRightBeamCount"));
+         daddy_context_->unset_property (ly_symbol2scm ("stemRightBeamCount"));
        }
 
       typeset_grob (stem_);
@@ -155,7 +165,7 @@ Stem_engraver::stop_translation_timestep ()
     }
 
 
-  tremolo_req_ = 0;
+  tremolo_ev_ = 0;
 }
 
 bool
@@ -163,15 +173,15 @@ Stem_engraver::try_music (Music* r)
 {
   if (r->is_mus_type ("tremolo-event"))
     {
-      tremolo_req_ = r;
+      tremolo_ev_ = r;
       return true;
     }
   return false;
 }
 
-ENTER_DESCRIPTION(Stem_engraver,
-/* descr */       "Create stems and single-stem tremolos.  It also works together with
-the beam engraver for overriding beaming.",
+ENTER_DESCRIPTION (Stem_engraver,
+/* descr */       "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",