X-Git-Url: https://git.donarmstrong.com/?a=blobdiff_plain;f=lily%2Fmulti-measure-rest-engraver.cc;h=4f1115a98a3727c41fea67705aad8fe5d815a69c;hb=f9efd2c54b174591fa42889557686c7aea356547;hp=460ccec319827a46c47331f59ca653c3c52c4f32;hpb=89bf299597169eeaa5ed7a215b6e808fa849c8cd;p=lilypond.git

diff --git a/lily/multi-measure-rest-engraver.cc b/lily/multi-measure-rest-engraver.cc
index 460ccec319..4f1115a98a 100644
--- a/lily/multi-measure-rest-engraver.cc
+++ b/lily/multi-measure-rest-engraver.cc
@@ -1,7 +1,7 @@
 /*
   multi_measure_rest-engraver.cc -- implement Multi_measure_rest_engraver
 
-  (c) 1998--2000 Jan Nieuwenhuizen <janneke@gnu.org>
+  (c) 1998--2002 Jan Nieuwenhuizen <janneke@gnu.org>
        Han-Wen Nienhuys <hanwen@cs.uu.nl>
 */
 
@@ -9,7 +9,7 @@
 #include "multi-measure-rest.hh"
 #include "paper-column.hh"
 #include "engraver-group-engraver.hh"
-#include "bar.hh"
+
 #include "staff-symbol-referencer.hh"
 #include "engraver.hh"
 #include "moment.hh"
@@ -18,23 +18,18 @@
 /**
    The name says it all: make multi measure rests 
 
-FIXME? The MM rest engraver must be able to see bar lines, so it won't
-work at Voice level. Not a problem in practice, but aesthetically pleasing? 
-
 */
 class Multi_measure_rest_engraver : public Engraver
 {
 public:
-  VIRTUAL_COPY_CONS(Translator);
-  Multi_measure_rest_engraver ();
+  TRANSLATOR_DECLARATIONS(Multi_measure_rest_engraver);
 
 protected:
-  virtual void acknowledge_grob (Grob_info i);
   virtual bool try_music (Music*);
+  virtual void process_music ();
   virtual void stop_translation_timestep ();
   virtual void start_translation_timestep ();
   virtual void finalize ();
-  virtual void create_grobs ();
 
 private:
   Span_req * new_req_l_;
@@ -47,7 +42,7 @@ private:
   Spanner *lastrest_p_;
 };
 
-ADD_THIS_TRANSLATOR (Multi_measure_rest_engraver);
+
 
 Multi_measure_rest_engraver::Multi_measure_rest_engraver ()
 {
@@ -56,18 +51,6 @@ Multi_measure_rest_engraver::Multi_measure_rest_engraver ()
   new_req_l_ = busy_span_req_l_ = stop_req_l_ =0;
 }
 
-void
-Multi_measure_rest_engraver::acknowledge_grob (Grob_info i)
-{
-  Item * item = dynamic_cast<Item*> (i.elem_l_); 
-  if (item && Bar::has_interface (item))
-    {
-      if (mmrest_p_)
-	Multi_measure_rest::add_column (mmrest_p_,item);
-      if (lastrest_p_)
-	Multi_measure_rest::add_column (lastrest_p_,item);
-    }
-}
 
 bool
 Multi_measure_rest_engraver::try_music (Music* req_l)
@@ -78,11 +61,11 @@ Multi_measure_rest_engraver::try_music (Music* req_l)
       if (scm_equal_p (sp->get_mus_property ("span-type"),
 		       ly_str02scm ("rest")) == SCM_BOOL_T)
 	{
-	  if (sp->get_span_dir() == STOP)
+	  if (sp->get_span_dir () == STOP)
 	    {
 	      stop_req_l_ = sp;
 	    }
-	  else if (sp->get_span_dir() == START && !new_req_l_)
+	  else if (sp->get_span_dir () == START && !new_req_l_)
 	    {
 	      new_req_l_ = sp;
 	    }
@@ -93,7 +76,7 @@ Multi_measure_rest_engraver::try_music (Music* req_l)
 }
 
 void
-Multi_measure_rest_engraver::create_grobs ()
+Multi_measure_rest_engraver::process_music ()
 {
   if (new_req_l_ && stop_req_l_)
     stop_req_l_ = 0;
@@ -117,33 +100,45 @@ Multi_measure_rest_engraver::create_grobs ()
     {
       mmrest_p_ = new Spanner (get_property ("MultiMeasureRest"));
 
-      Multi_measure_rest::set_interface (mmrest_p_);
-      Staff_symbol_referencer::set_interface (mmrest_p_);
-
-      announce_grob (mmrest_p_, busy_span_req_l_);
+      announce_grob(mmrest_p_, busy_span_req_l_->self_scm());
       start_measure_i_
 	= gh_scm2int (get_property ("currentBarNumber"));
     }
+
+  if (gh_string_p (get_property ("whichBar")))
+    {
+      Grob *cmc = unsmob_grob (get_property( "currentCommandColumn"));
+      Item *it = dynamic_cast<Item*> (cmc);
+      if (mmrest_p_)
+	add_bound_item (mmrest_p_, it);
+      if (lastrest_p_)
+	add_bound_item (lastrest_p_,it);
+    }
 }
 
 void
 Multi_measure_rest_engraver::stop_translation_timestep ()
 {
   SCM smp = get_property ("measurePosition");
-  Moment mp =  (unsmob_moment (smp)) ? *unsmob_moment (smp) : Moment (0);
+  Moment mp = (unsmob_moment (smp)) ? *unsmob_moment (smp) : Moment (0);
 
   if (mmrest_p_ && (now_mom () >= start_moment_) 
-    && !mp
-    && (scm_ilength (mmrest_p_->get_grob_property ("columns")) >= 2))
+      && !mp.to_bool ()
+      && mmrest_p_->get_bound (LEFT) && mmrest_p_->get_bound (RIGHT))
     {
       typeset_grob (mmrest_p_);
       /*
-	must keep mmrest_p_ around to set measures_i_
+	we must keep mmrest_p_ around to set measure-count, so
+	no mmrest_p_ = 0 here. 
        */
     }
+
   if (lastrest_p_)
     {
-      typeset_grob (lastrest_p_);
+      /* sanity check */
+      if (lastrest_p_->get_bound (LEFT) && lastrest_p_->get_bound (RIGHT)
+	  && lastrest_p_->get_bound (LEFT) != lastrest_p_->get_bound (RIGHT))
+	typeset_grob (lastrest_p_);
       lastrest_p_ = 0;
     }
 
@@ -159,9 +154,9 @@ void
 Multi_measure_rest_engraver::start_translation_timestep ()
 {
   SCM smp = get_property ("measurePosition");
-  Moment mp =  (unsmob_moment (smp)) ? *unsmob_moment (smp) : Moment (0);
+  Moment mp = (unsmob_moment (smp)) ? *unsmob_moment (smp) : Moment (0);
   
-  if (mmrest_p_ && !mp)
+  if (mmrest_p_ && !mp.to_bool ())
     {
       lastrest_p_ = mmrest_p_;
       int cur = gh_scm2int (get_property ("currentBarNumber"));
@@ -180,3 +175,12 @@ Multi_measure_rest_engraver::finalize ()
   if (lastrest_p_)
     typeset_grob (lastrest_p_);
 }
+
+ENTER_DESCRIPTION(Multi_measure_rest_engraver,
+/* descr */       "Engraves multi-measure rests that are produced with @code{R}.  Reads
+measurePosition and currentBarNumber to determine what number to print over the MultiMeasureRest
+",
+/* creats*/       "MultiMeasureRest",
+/* acks  */       "",
+/* reads */       "currentBarNumber currentCommandColumn measurePosition",
+/* write */       "");