*** empty log message ***

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

diff --git a/lily/bar-engraver.cc b/lily/bar-engraver.cc
index 3e4c5f58d4b3743a8f81870f7061037db7348067..78008a625ffd20e4f4502e5130e390f8028a66fe 100644 (file)
--- a/lily/bar-engraver.cc
+++ b/lily/bar-engraver.cc
@@ -3,13 +3,13 @@
 
   source file of the GNU LilyPond music typesetter
 
-  (c)  1997--2001 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 "event.hh"
 #include "engraver-group-engraver.hh"
 #include "warn.hh"
 #include "item.hh"
@@ -22,38 +22,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_, 0);
+      if (scm_equal_p (gl, bar_->get_property ("glyph")) != SCM_BOOL_T)
+         bar_->set_property ("glyph", gl);
     }
 }
 
@@ -66,11 +63,19 @@ 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")))
+  if (!bar_ && ly_c_string_p (get_property ("whichBar")))
     {
       create_bar ();
     }
@@ -79,11 +84,7 @@ Bar_engraver::create_grobs ()
 void
 Bar_engraver::typeset_bar ()
 {
-  if (bar_p_) 
-    {
-      typeset_grob (bar_p_);
-      bar_p_ =0;
-    }
+      bar_ =0;
 }
 
 /*
@@ -92,19 +93,20 @@ Bar_engraver::typeset_bar ()
 void 
 Bar_engraver::stop_translation_timestep ()
 {
-  if (!bar_p_)
+  if (!bar_)
     {
-      top_engraver ()->forbid_breaks ();       // guh. Use properties!
+      get_score_engraver ()->forbid_breaks (); // guh. Use properties!
     }
   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",
+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",
 /* creats*/       "BarLine",
-/* acks  */       "",
+/* accepts */     "",
+/* acks  */      "",
 /* reads */       "whichBar",
 /* write */       "");