]> git.donarmstrong.com Git - lilypond.git/blobdiff - lily/side-position-interface.cc
* lily/spacing-engraver.cc (finalize): robustifications.
[lilypond.git] / lily / side-position-interface.cc
index 592d53834f02176a937e67e28a57c2c6b2a6b6fa..94c3f5d3493c8824aff51162fb217dc85a37e312 100644 (file)
@@ -15,6 +15,7 @@
 #include "staff-symbol-referencer.hh"
 #include "group-interface.hh"
 #include "directional-element-interface.hh"
+#include "staff-symbol-referencer.hh"
 
 void
 Side_position_interface::add_support (Grob*me, Grob*e)
@@ -22,8 +23,6 @@ Side_position_interface::add_support (Grob*me, Grob*e)
   Pointer_group_interface::add_grob (me, ly_symbol2scm ("side-support-elements"), e);
 }
 
-
-
 Direction
 Side_position_interface::get_direction (Grob*me)
 {
@@ -67,17 +66,7 @@ Side_position_interface::aligned_on_support_extents (SCM element_smob, SCM axis)
 SCM
 Side_position_interface::general_side_position (Grob * me, Axis a, bool use_extents)
 {
-
-
-  /*
-    As this is only used as a callback, this is called only once. We
-    could wipe SIDE-SUPPORT-ELEMENTS after we retrieve it to conserve
-    memory; however -- we should look more into benefits of such actions?
-
-    The benefit is small, it seems: total GC times taken don't
-    differ. Would this also hamper Generational GC ?
-    
-  */
+  Real ss = Staff_symbol_referencer::staff_space (me);
   SCM support = me->get_grob_property ("side-support-elements");
   Grob *common = common_refpoint_of_list (support, me->get_parent (a), a);
   
@@ -95,7 +84,7 @@ Side_position_interface::general_side_position (Grob * me, Axis a, bool use_exte
          }
     }
 
-  if (dim.empty_b ())
+  if (dim.is_empty ())
     {
       dim = Interval (0,0);
     }
@@ -103,25 +92,24 @@ Side_position_interface::general_side_position (Grob * me, Axis a, bool use_exte
   Direction dir = Side_position_interface::get_direction (me);
     
   Real off =  me->get_parent (a)->relative_coordinate (common, a);
-  SCM minimum = me->get_grob_property ("minimum-space");
+  Real  minimum_space = ss * robust_scm2double (me->get_grob_property ("minimum-space"),  -1);
 
   Real total_off = dim.linear_combination (dir) - off;
-  SCM padding = me->get_grob_property ("padding");
-  if (gh_number_p (padding))
-    {
-      total_off += gh_scm2double (padding) * dir;
-    }
+  total_off += dir * ss * robust_scm2double (me->get_grob_property ("padding"), 0);
 
-  if (gh_number_p (minimum) 
+  if (minimum_space >= 0
       && dir
-      && total_off * dir < gh_scm2double (minimum))
+      && total_off * dir < minimum_space)
     {
-      total_off = gh_scm2double (minimum) * dir;
+      total_off = minimum_space * dir;
     }
 
   if (fabs (total_off) > 100 CM)
     programming_error ("Huh ? Improbable staff side dim.");
 
+
+  
+  
   return gh_double2scm (total_off);
 }
 
@@ -135,7 +123,7 @@ Side_position_interface::aligned_on_support_refpoints (SCM smob, SCM axis)
   Grob *me = unsmob_grob (smob);
   Axis a = (Axis) gh_scm2int (axis);
 
-  return  general_side_position (me, a, false); 
+  return general_side_position (me, a, false); 
 }
 
 
@@ -199,7 +187,7 @@ Side_position_interface::aligned_side (SCM element_smob, SCM axis)
 
   Interval iv =  me->extent (me, a);
 
-  if (!iv.empty_b ())
+  if (!iv.is_empty ())
     {
       if (!d)
        {
@@ -208,50 +196,32 @@ Side_position_interface::aligned_side (SCM element_smob, SCM axis)
        }
       o += - iv[-d];
     }
-  return gh_double2scm (o);
-}
 
-/*
+  /*
   Maintain a minimum distance to the staff. This is similar to side
   position with padding, but it will put adjoining objects on a row if
   stuff sticks out of the staff a little.
  */
-MAKE_SCHEME_CALLBACK (Side_position_interface,out_of_staff,2);
-SCM
-Side_position_interface::out_of_staff (SCM element_smob, SCM axis)
-{
-  Grob *me = unsmob_grob (element_smob);
-  Axis a = (Axis) gh_scm2int (axis);
-
   Grob * st = Staff_symbol_referencer::get_staff_symbol (me);
-
-  if (!st)
-    return gh_int2scm (0);
-
-  Real padding=0.0;
-  SCM spad = me->get_grob_property ("staff-padding");
-
-  if (gh_number_p (spad))
-    padding = gh_scm2double (spad);
-  
-  Grob *common = me->common_refpoint (st, Y_AXIS);
-  Direction d = Side_position_interface::get_direction (me);
-  Interval staff_size = st->extent (common, Y_AXIS);
-  Interval me_ext = me->extent (common, a);
-  Real diff =  d*staff_size[d] + padding - d*me_ext[-d];
-  return gh_double2scm (diff >? 0);
-}
-
-void
-Side_position_interface::add_staff_support (Grob*me)
-{
-  Grob* st = Staff_symbol_referencer::get_staff_symbol (me);
-  if (st && get_axis (me) == Y_AXIS)
+  if (st && a == Y_AXIS
+      && gh_number_p (me->get_grob_property ("staff-padding")))
     {
-      add_support (me,st);
+      Real padding=
+      Staff_symbol_referencer::staff_space (me)
+      * gh_scm2double (me->get_grob_property ("staff-padding"));
+  
+      Grob *common = me->common_refpoint (st, Y_AXIS);
+      
+      Interval staff_size = st->extent (common, Y_AXIS);
+      Interval me_ext = me->extent (common, a);
+      Real diff =  d*staff_size[d] + padding - d*(o + iv[-d]);
+      o += (d*  (diff >? 0));
     }
+      
+  return gh_double2scm (o);
 }
 
+
 void
 Side_position_interface::set_axis (Grob*me, Axis a)
 {
@@ -272,38 +242,15 @@ Side_position_interface::get_axis (Grob*me)
   return Y_AXIS;
 }
 
-void
-Side_position_interface::set_direction (Grob*me, Direction d)
-{
-  me->set_grob_property ("direction", gh_int2scm (d));
-}
-
-void
-Side_position_interface::set_minimum_space (Grob*me, Real m)
-{
-  me->set_grob_property ("minimum-space", gh_double2scm (m));
-}
-
-void
-Side_position_interface::set_padding (Grob*me, Real p)
-{
-  me->set_grob_property ("padding", gh_double2scm (p));
-}
-
-
-bool
-Side_position_interface::supported_b (Grob*me) 
-{
-  SCM s = me->get_grob_property ("side-support-elements"); 
-  return gh_pair_p (s);
-}
-
 
 
 
 ADD_INTERFACE (Side_position_interface,"side-position-interface",
-  "Position a victim object (this one) next to other objects (the "
-"support).  In this case, the property @code{direction} signifies where to put the  "
-"victim object relative to the support (left or right, up or down?) "
-,
-  "staff-padding side-support-elements direction-source direction side-relative-direction minimum-space padding");
+              "Position a victim object (this one) next to other objects (the "
+              "support).  In this case, the property @code{direction} signifies where to put the  "
+              "victim object relative to the support (left or right, up or down?)\n\n "
+              "The routine puts objects at a distance of the staff if the property "
+              "@code{staff-padding} is defined. If undefined, the staff symbol is ignored." 
+              ,
+              "staff-padding side-support-elements direction-source "
+              "direction side-relative-direction minimum-space padding");