X-Git-Url: https://git.donarmstrong.com/?a=blobdiff_plain;f=lily%2Fbar-engraver.cc;h=a070e4f411ac161a2fb7b99c8b4bb9541217fb86;hb=0817e0513d1016ff22a633b6fee20ddba2a062f2;hp=d835edbd09765ef3a809ab807ad9338f494218f9;hpb=ad7defd5f5a3d0151a711555c2d680c56ef559c7;p=lilypond.git

diff --git a/lily/bar-engraver.cc b/lily/bar-engraver.cc
index d835edbd09..a070e4f411 100644
--- a/lily/bar-engraver.cc
+++ b/lily/bar-engraver.cc
@@ -3,123 +3,95 @@
 
   source file of the GNU LilyPond music typesetter
 
-  (c)  1997--2000 Han-Wen Nienhuys <hanwen@cs.uu.nl>
+  (c)  1997--2001 Han-Wen Nienhuys <hanwen@cs.uu.nl>
   Jan Nieuwenhuizen <janneke@gnu.org>
 */
 
+#include "bar.hh"
 #include "score-engraver.hh"
-#include "bar-engraver.hh"
-#include "staff-bar.hh"
 #include "musical-request.hh"
-#include "multi-measure-rest.hh"
-#include "command-request.hh"
-#include "timing-engraver.hh"
 #include "engraver-group-engraver.hh"
 #include "warn.hh"
+#include "item.hh"
+#include "engraver.hh"
 
-Bar_engraver::Bar_engraver()
+/*
+  generate bars. Either user ("|:"), or default (new measure)
+
+  */
+class Bar_engraver : public Engraver
+{
+public:
+  TRANSLATOR_DECLARATIONS(  Bar_engraver );
+  void request_bar (String type_str);
+    
+protected:
+  virtual void finalize ();
+  virtual void stop_translation_timestep ();
+  virtual void create_grobs ();
+
+private:
+  void typeset_bar ();
+  void create_bar ();
+
+  Item * bar_p_;
+};
+
+Bar_engraver::Bar_engraver ()
 {
   bar_p_ =0;
-  do_post_move_processing();
 }
 
-
-
-
 void
 Bar_engraver::create_bar ()
 {
   if (!bar_p_)
     {
-      bar_p_ = new Staff_bar;
-      bar_p_->set_elt_property ("break-aligned", SCM_BOOL_T);
+      bar_p_ = new Item (get_property ("BarLine"));
 
-      // urg: "" != empty...
-      SCM default_type = get_property ("defaultBarType");
-      if (gh_string_p (default_type))
-	{
-	  bar_p_->set_elt_property ("glyph", default_type); // gu.h
-	}
-
-#if 0
-      /*
-	urg.  Why did I implement this? And did I implement this so
-	clumsily?
-
-	input/test/just-friends.ly
-	Maybe a staffgroup of just one staff would be a better solution.
-      */
-      SCM prop = get_property ("barAtLineStart");
-      if (to_boolean (prop))
-	{
-	  bar_p_->set_elt_property ("at-line-start", SCM_BOOL_T);
-	}
-#endif
-      announce_element (Score_element_info (bar_p_, 0));
+      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);
     }
 }
 
-/**
-   Make a barline.  If there are both |: and :| requested, merge them
-   to :|:.
+void
+Bar_engraver::finalize ()
+{
+  typeset_bar ();
+}
 
+/*
+  Bar_engraver should come *after* any engravers that  
+  modify whichBar
 */
 void
-Bar_engraver::request_bar (String requested_type)
+Bar_engraver::create_grobs ()
 {
-  if (!now_mom ())
-    {
-      SCM prop = get_property ("barAtLineStart");
-      if (!to_boolean (prop))
-	return;
-    }
-  bool  bar_existed = bar_p_;
-  create_bar ();
-  if (bar_existed && requested_type == "")
+  if (!bar_p_ && gh_string_p (get_property ("whichBar")))
     {
-      return;
+      create_bar ();
     }
-
-  String current = ly_scm2string (bar_p_->get_elt_property ("glyph"));
-  
-  if ((requested_type == "|:" && current== ":|")
-    || (requested_type == ":|" && current == "|:"))
-    requested_type = ":|:";
-
-  
-  bar_p_->set_elt_property ("glyph",
-			    ly_str02scm (requested_type.ch_C ()));
-}
-
-void 
-Bar_engraver::do_creation_processing ()
-{
 }
 
 void
-Bar_engraver::do_removal_processing ()
+Bar_engraver::typeset_bar ()
 {
   if (bar_p_) 
     {
-      typeset_element (bar_p_);
+      typeset_grob (bar_p_);
       bar_p_ =0;
     }
 }
 
-void
-Bar_engraver::do_process_music()
-{  
-  Translator * t = daddy_grav_l  ()->get_simple_translator ("Timing_engraver");
-
-  Timing_engraver * te = dynamic_cast<Timing_engraver*>(t);
-  String which = (te) ? te->which_bar () : "";
-
-  if (which.length_i ())
-    {
-      create_bar();
-      bar_p_->set_elt_property ("glyph",  ly_str02scm (which.ch_C ()));
-    }
-  
+/*
+  lines may only be broken if there is a barline in all staves 
+*/
+void 
+Bar_engraver::stop_translation_timestep ()
+{
   if (!bar_p_)
     {
       Score_engraver * e = 0;
@@ -128,26 +100,21 @@ Bar_engraver::do_process_music()
 	{
 	  e = dynamic_cast<Score_engraver*> (t);
 	}
-      
+
       if (!e)
 	programming_error ("No score engraver!");
       else
-	e->forbid_breaks ();
+	e->forbid_breaks ();	// guh. Use properties!
     }
+  else
+    typeset_bar ();
 }
 
 
-void 
-Bar_engraver::do_pre_move_processing()
-{
-  if (bar_p_) 
-    {
-      typeset_element (bar_p_);
-      bar_p_ =0;
-    }
-}
-
-ADD_THIS_TRANSLATOR(Bar_engraver);
-
-
-
+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  */       "",
+/* reads */       "whichBar stavesFound",
+/* write */       "");