X-Git-Url: https://git.donarmstrong.com/?a=blobdiff_plain;f=lily%2Fbar-engraver.cc;h=dad72a4fe4f7648f1125d65df46ea1e2d4569b36;hb=28240aceaab110b424d8e37cb8024ad5f815d57f;hp=e8b29882f86e45bb3d667a1941c8292adaa1c4d7;hpb=7ed4019033adf7db7a8f64c4eb7b7f6d0449402f;p=lilypond.git

diff --git a/lily/bar-engraver.cc b/lily/bar-engraver.cc
index e8b29882f8..dad72a4fe4 100644
--- a/lily/bar-engraver.cc
+++ b/lily/bar-engraver.cc
@@ -3,140 +3,101 @@
 
   source file of the GNU LilyPond music typesetter
 
-  (c)  1997, 1998, 1999 Han-Wen Nienhuys <hanwen@cs.uu.nl>
+  (c) 1997--2005 Han-Wen Nienhuys <hanwen@cs.uu.nl>
   Jan Nieuwenhuizen <janneke@gnu.org>
 */
 
+#include "bar-line.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"
 
-Bar_engraver::Bar_engraver()
+/*
+  generate bars. Either user ("|:"), or default (new measure)
+*/
+class Bar_engraver : public Engraver
 {
-  bar_p_ =0;
-  do_post_move_processing();
-}
+public:
+  TRANSLATOR_DECLARATIONS (Bar_engraver);
+  void request_bar (String type_string);
 
+protected:
+  virtual void finalize ();
+  virtual void stop_translation_timestep ();
+  virtual void process_acknowledged_grobs ();
 
+private:
+  void typeset_bar ();
+  void create_bar ();
 
+  Item *bar_;
+};
 
-void
-Bar_engraver::create_bar ()
+Bar_engraver::Bar_engraver ()
 {
-  if (!bar_p_)
-    {
-      bar_p_ = new Staff_bar;
-      bar_p_->set_elt_property ("break-aligned", SCM_BOOL_T);
-
-      // urg: "" != empty...
-      SCM default_type = get_property ("defaultBarType", 0);
-      if (gh_string_p (default_type))
-	{
-	  bar_p_->type_str_ = ly_scm2string (default_type);
-	}
-
-      /*
-	urg.  Why did I implement this?
-       */
-      SCM prop = get_property ("barAtLineStart", 0);
-      if (gh_boolean_p (prop) && gh_scm2bool (prop))
-	{
-	  bar_p_->set_elt_property ("at-line-start", SCM_BOOL_T);
-	}
-      announce_element (Score_element_info (bar_p_, 0));
-    }
+  bar_ = 0;
 }
 
-/**
-   Make a barline.  If there are both |: and :| requested, merge them
-   to :|:.
-
-*/
 void
-Bar_engraver::request_bar (String requested_type)
+Bar_engraver::create_bar ()
 {
-  if (!now_mom ())
-    {
-      SCM prop = get_property ("barAtLineStart", 0);
-      if (!gh_boolean_p (prop) && gh_scm2bool (prop))
-	return;
-    }
-  bool  bar_existed = bar_p_;
-  create_bar ();
-  if (bar_existed && requested_type == "")
+  if (!bar_)
     {
-      return;
+      bar_ = make_item ("BarLine", SCM_EOL);
+      SCM gl = get_property ("whichBar");
+      if (scm_equal_p (gl, bar_->get_property ("glyph")) != SCM_BOOL_T)
+	bar_->set_property ("glyph", gl);
     }
-  else if (((requested_type == "|:") && (bar_p_->type_str_ == ":|"))
-    || ((requested_type == ":|") && (bar_p_->type_str_ == "|:")))
-    bar_p_->type_str_ = ":|:";
-  else
-    bar_p_->type_str_ = requested_type;
 }
 
-void 
-Bar_engraver::do_creation_processing ()
+void
+Bar_engraver::finalize ()
 {
+  typeset_bar ();
 }
 
+/*
+  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::do_removal_processing ()
+Bar_engraver::process_acknowledged_grobs ()
 {
-  if (bar_p_) 
-    {
-      typeset_element (bar_p_);
-      bar_p_ =0;
-    }
+  if (!bar_ && scm_is_string (get_property ("whichBar")))
+    create_bar ();
 }
 
 void
-Bar_engraver::do_process_requests()
-{  
-  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_->type_str_ = which;
-    }
-  
-  if (!bar_p_)
-    {
-      Score_engraver * e = 0;
-      Translator * t  =  daddy_grav_l ();
-      for (; !e && t;  t = t->daddy_trans_l_)
-	{
-	  e = dynamic_cast<Score_engraver*> (t);
-	}
-      
-      if (!e)
-	programming_error ("No score engraver!");
-      else
-	e->forbid_breaks ();
-    }
+Bar_engraver::typeset_bar ()
+{
+  bar_ = 0;
 }
 
-
-void 
-Bar_engraver::do_pre_move_processing()
+/*
+  lines may only be broken if there is a barline in all staves
+*/
+void
+Bar_engraver::stop_translation_timestep ()
 {
-  if (bar_p_) 
-    {
-      typeset_element (bar_p_);
-      bar_p_ =0;
-    }
+  if (!bar_)
+    /* guh. Use properties! */
+    get_score_engraver ()->forbid_breaks ();
+  else
+    typeset_bar ();
 }
 
-ADD_THIS_TRANSLATOR(Bar_engraver);
-
-
-
+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",
+		/* accepts */ "",
+		/* acks  */ "",
+		/* reads */ "whichBar",
+		/* write */ "");