2003 -> 2004

[lilypond.git] / lily / axis-group-engraver.cc

diff --git a/lily/axis-group-engraver.cc b/lily/axis-group-engraver.cc
index 9bca1ce3c7ee01006e3217fc9f0d665cb732a105..a825805d789ce2ad6d277faadf3b71600d4432c8 100644 (file)
--- a/lily/axis-group-engraver.cc
+++ b/lily/axis-group-engraver.cc
@@ -3,7 +3,7 @@
   
   source file of the GNU LilyPond music typesetter
   
-  (c) 1999--2002 Han-Wen Nienhuys <hanwen@cs.uu.nl>
+  (c) 1999--2004 Han-Wen Nienhuys <hanwen@cs.uu.nl>
  */
 
 #include "spanner.hh"
@@ -11,6 +11,7 @@
 #include "axis-group-interface.hh"
 #include "engraver.hh"
 #include "engraver-group-engraver.hh"
+#include "warn.hh"
 
 /**
    Put stuff in a Spanner with an Axis_group_interface.
@@ -21,63 +22,71 @@ class Axis_group_engraver : public Engraver
 protected:
   Spanner *staffline_;
   Link_array<Grob> elts_;
-  virtual void initialize ();
+  virtual void process_music ();
   virtual void finalize ();
   virtual void acknowledge_grob (Grob_info);
   virtual void process_acknowledged_grobs ();
-  virtual Spanner* get_spanner () const;
+  virtual Spanner* get_spanner () ;
   virtual void add_element (Grob*) ;
 public:  
-TRANSLATOR_DECLARATIONS(Axis_group_engraver );
+  TRANSLATOR_DECLARATIONS(Axis_group_engraver);
 };
 
-
-
 Axis_group_engraver::Axis_group_engraver ()
 {
   staffline_ = 0;
 }
 
 void
-Axis_group_engraver::initialize ()
+Axis_group_engraver::process_music ()
 {
-  staffline_ = get_spanner ();
+  if (!staffline_)
+    {
+      staffline_ = get_spanner ();
 
-  Grob *  it = unsmob_grob (get_property ("currentCommandColumn"));
+      Grob *  it = unsmob_grob (get_property ("currentCommandColumn"));
 
-  staffline_->set_bound (LEFT,it);
+      staffline_->set_bound (LEFT,it);
 
-  announce_grob(staffline_, SCM_EOL);
-}
+      announce_grob(staffline_, SCM_EOL);
+    }
+} 
 
 Spanner*
-Axis_group_engraver::get_spanner () const
+Axis_group_engraver::get_spanner () 
 {
-  return new Spanner (get_property ("VerticalAxisGroup"));
+  return make_spanner ("VerticalAxisGroup");
 }
 
+/*
+  TODO: should we junk minimumVerticalExtent/extraVerticalExtent ?  
+ */
+
 void
 Axis_group_engraver::finalize ()
 {
-  String type = get_daddy_grav ()->type_string_ ;
+  if (!staffline_)
+    return ;
+  
+  String type = get_daddy_grav ()->context_name ();
   SCM dims = get_property ("verticalExtent");
   
   if (gh_pair_p (dims) && gh_number_p (ly_car (dims))
       && gh_number_p (ly_cdr (dims)))
     {
       staffline_->set_extent (Grob::preset_extent_proc, Y_AXIS);
-      staffline_->set_grob_property ("extent-Y", dims);
+      staffline_->set_grob_property ("Y-extent", dims);
     }
 
   dims = get_property ("minimumVerticalExtent");
   if (gh_pair_p (dims) && gh_number_p (ly_car (dims))
       && gh_number_p (ly_cdr (dims)))
-    staffline_->set_grob_property ("minimum-extent-Y", dims);
+    staffline_->set_grob_property ("minimum-Y-extent", dims);
 
   dims = get_property ("extraVerticalExtent");
   if (gh_pair_p (dims) && gh_number_p (ly_car (dims))
       && gh_number_p (ly_cdr (dims)))
-    staffline_->set_grob_property ("extra-extent-Y", dims);
+    staffline_->set_grob_property ("extra-Y-extent", dims);
 
   Grob *  it = unsmob_grob (get_property ("currentCommandColumn"));
 
@@ -106,9 +115,18 @@ Axis_group_engraver::process_acknowledged_grobs ()
     {
       Grob *par = elts_[i]->get_parent (Y_AXIS);
 
-      if ((!par || !Axis_group_interface::has_interface (par))
-         && ! elts_[i]->empty_b (Y_AXIS))
-       add_element (elts_[i]);
+      if (!par || !Axis_group_interface::has_interface (par))
+       if (elts_[i]->is_empty (Y_AXIS))
+         {
+           /*
+             We have to do _something_, otherwise staff objects will
+             end up with System as parent.  
+             
+            */
+           elts_[i]->set_parent (staffline_, Y_AXIS);
+         }
+       else
+         add_element (elts_[i]);
     }
   elts_.clear ();
 }
@@ -119,8 +137,14 @@ Axis_group_engraver::add_element (Grob*e)
   Axis_group_interface::add_element (staffline_, e);
 }
 
-////////////////////////////////////////////////////////
-// maybenot such a good idea after all., to put classes in .cc
+/****************************************************************/
+
+/*
+  
+ maybenot such a good idea after all., to put class declarations in
+ .cc
+ 
+*/
 
 #include "hara-kiri-group-spanner.hh"
 #include "rhythmic-head.hh"
@@ -128,7 +152,7 @@ Axis_group_engraver::add_element (Grob*e)
 class Hara_kiri_engraver : public Axis_group_engraver
 {
 protected:
-  virtual Spanner*get_spanner ()const;
+  virtual Spanner*get_spanner ();
   virtual void acknowledge_grob (Grob_info);
   virtual void add_element (Grob *e);
 public:
@@ -143,10 +167,10 @@ Hara_kiri_engraver::add_element (Grob*e)
 
 
 Spanner*
-Hara_kiri_engraver::get_spanner () const
+Hara_kiri_engraver::get_spanner () 
 {
-  Spanner * sp = new Spanner (get_property ("HaraKiriVerticalGroup"));
-
+  Spanner * sp = make_spanner ("RemoveEmptyVerticalGroup");
+  
   return sp;
 }
 
@@ -154,27 +178,32 @@ void
 Hara_kiri_engraver::acknowledge_grob (Grob_info i)
 {
   Axis_group_engraver::acknowledge_grob (i);
-  if (Rhythmic_head::has_interface (i.grob_)
-      || i.grob_->internal_has_interface (ly_symbol2scm ("lyric-syllable-interface")))
+  if (i.grob_->internal_has_interface (ly_symbol2scm ("rhythmic-grob-interface"))
+      || i.grob_->internal_has_interface (ly_symbol2scm ("lyric-interface"))
+      )
     {
       Hara_kiri_group_spanner::add_interesting_item (staffline_, i.grob_);
     }
 }
 
-Hara_kiri_engraver::Hara_kiri_engraver(){}
+
+Hara_kiri_engraver::Hara_kiri_engraver()
+{
+}
 
 ENTER_DESCRIPTION(Hara_kiri_engraver,
-/* descr */       "Like Axis_group_engraver, but make a hara kiri spanner, and add
-interesting items (ie. note heads, lyric syllables and normal rests)
-",
-/* creats*/       "HaraKiriVerticalGroup",
-/* acks  */       "grob-interface",
+/* descr */       "Like Axis_group_engraver, but make a hara kiri spanner, and add "
+"interesting items (ie. note heads, lyric syllables and normal rests) ",
+/* creats*/       "RemoveEmptyVerticalGroup",
+/* accepts */     "",
+/* acks  */      "grob-interface",
 /* reads */       "",
 /* write */       "");
 
 ENTER_DESCRIPTION(Axis_group_engraver,
 /* descr */       "Group all objects created in this context in a VerticalAxisGroup spanner.",
 /* creats*/       "VerticalAxisGroup",
-/* acks  */       "grob-interface",
+/* accepts */     "",
+/* acks  */      "grob-interface",
 /* reads */       "verticalExtent minimumVerticalExtent extraVerticalExtent",
 /* write */       "");