X-Git-Url: https://git.donarmstrong.com/?a=blobdiff_plain;f=lily%2Fbar-engraver.cc;h=e3fcdfefd533344b88e4d618ddd348fba8aeaf30;hb=4ebb61bb6f16135ecbe329c5acb2f88969981a91;hp=630d4ce14e3c311ee9beb3070755b76b1ec61cb5;hpb=2804f8f79e01dc606ce067bffda2d966524e6e36;p=lilypond.git

diff --git a/lily/bar-engraver.cc b/lily/bar-engraver.cc
index 630d4ce14e..e3fcdfefd5 100644
--- a/lily/bar-engraver.cc
+++ b/lily/bar-engraver.cc
@@ -3,17 +3,14 @@
 
   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>
   Jan Nieuwenhuizen <janneke@gnu.org>
 */
 
-#include "bar.hh"
+#include "bar-line.hh"
 #include "score-engraver.hh"
-#include "musical-request.hh"
-#include "engraver-group-engraver.hh"
 #include "warn.hh"
 #include "item.hh"
-#include "engraver.hh"
 
 /*
   generate bars. Either user ("|:"), or default (new measure)
@@ -22,38 +19,35 @@
 class Bar_engraver : public Engraver
 {
 public:
-  TRANSLATOR_DECLARATIONS(  Bar_engraver );
-  void request_bar (String type_str);
+  TRANSLATOR_DECLARATIONS (Bar_engraver);
+  void request_bar (String type_string);
     
 protected:
   virtual void finalize ();
   virtual void stop_translation_timestep ();
-  virtual void create_grobs ();
+  virtual void process_acknowledged_grobs ();
 
 private:
   void typeset_bar ();
   void create_bar ();
 
-  Item * bar_p_;
+  Item * bar_;
 };
 
 Bar_engraver::Bar_engraver ()
 {
-  bar_p_ =0;
+  bar_ = 0;
 }
 
 void
 Bar_engraver::create_bar ()
 {
-  if (!bar_p_)
+  if (!bar_)
     {
-      bar_p_ = new Item (get_property ("BarLine"));
-
+      bar_ = make_item ("BarLine", SCM_EOL);
       SCM gl = get_property ("whichBar");
-      if (scm_equal_p (gl, bar_p_->get_grob_property ("glyph")) != SCM_BOOL_T)
-	  bar_p_->set_grob_property ("glyph", gl);
-      
-      announce_grob(bar_p_, SCM_EOL);
+      if (scm_equal_p (gl, bar_->get_property ("glyph")) != SCM_BOOL_T)
+	  bar_->set_property ("glyph", gl);
     }
 }
 
@@ -66,24 +60,26 @@ Bar_engraver::finalize ()
 /*
   Bar_engraver should come *after* any engravers that  
   modify whichBar
+
+  This is a little hairy : whichBar may be set by
+  Repeat_acknowledge_engraver::process_music, which is at score
+  context. This means that grobs could should be created after
+  process_music. We do stuff process_acknowledged_grobs (), just to be
+  on the safe side.
+     
 */
+
 void
-Bar_engraver::create_grobs ()
+Bar_engraver::process_acknowledged_grobs ()
 {
-  if (!bar_p_ && gh_string_p (get_property ("whichBar")))
-    {
-      create_bar ();
-    }
+  if (!bar_ && scm_is_string (get_property ("whichBar")))
+    create_bar ();
 }
 
 void
 Bar_engraver::typeset_bar ()
 {
-  if (bar_p_) 
-    {
-      typeset_grob (bar_p_);
-      bar_p_ =0;
-    }
+  bar_ = 0;
 }
 
 /*
@@ -92,19 +88,19 @@ Bar_engraver::typeset_bar ()
 void 
 Bar_engraver::stop_translation_timestep ()
 {
-  if (!bar_p_)
-    {
-      top_engraver ()->forbid_breaks ();	// guh. Use properties!
-    }
+  if (!bar_)
+    /* guh. Use properties! */
+    get_score_engraver ()->forbid_breaks ();
   else
     typeset_bar ();
 }
 
 
-ENTER_DESCRIPTION(Bar_engraver,
-/* descr */       "Create barlines. This engraver is controlled through the
-@code{whichBar} property. If it has no bar line to create, it will forbid a linebreak at this point",
+ADD_TRANSLATOR (Bar_engraver,
+/* descr */       "Create barlines. This engraver is controlled through the "
+"@code{whichBar} property. If it has no bar line to create, it will forbid a linebreak at this point",
 /* creats*/       "BarLine",
-/* acks  */       "",
+/* accepts */     "",
+/* acks  */      "",
 /* reads */       "whichBar",
 /* write */       "");