X-Git-Url: https://git.donarmstrong.com/?a=blobdiff_plain;f=lily%2Ftext-spanner.cc;h=e98ae0f7dfa9afcd81c412e9c08ba504b87f5d07;hb=87eedcd59f4082cb0841528ad5bc82cb1d1191e3;hp=82c2475cba4a1102185926fece20ee616a44d61e;hpb=332da9f4d25f9f63a3222efc87fae06b0c7abbdd;p=lilypond.git

diff --git a/lily/text-spanner.cc b/lily/text-spanner.cc
index 82c2475cba..e98ae0f7df 100644
--- a/lily/text-spanner.cc
+++ b/lily/text-spanner.cc
@@ -3,14 +3,14 @@
 
   source file of the GNU LilyPond music typesetter
 
-  (c) 2000--2004 Jan Nieuwenhuizen <janneke@gnu.org>
+  (c) 2000--2007 Jan Nieuwenhuizen <janneke@gnu.org>
 
-  Revised over good by Han-Wen. 
+  Revised over good by Han-Wen.
 */
 
 #include "text-spanner.hh"
 
-#include "text-item.hh"
+#include "text-interface.hh"
 #include "line-spanner.hh"
 #include "spanner.hh"
 #include "font-interface.hh"
@@ -34,15 +34,15 @@ MAKE_SCHEME_CALLBACK (Text_spanner, print, 1);
   TODO: this function is too long
 */
 SCM
-Text_spanner::print (SCM smob) 
+Text_spanner::print (SCM smob)
 {
   Grob *me = unsmob_grob (smob);
-  Spanner *spanner = dynamic_cast<Spanner*> (me);
-  
+  Spanner *spanner = dynamic_cast<Spanner *> (me);
+
   /* Ugh, must be same as Hairpin::print.  */
 
   Grob *common = spanner->get_bound (LEFT)->common_refpoint (spanner->get_bound (RIGHT), X_AXIS);
-  Output_def * layout = me->get_layout ();
+  Output_def *layout = me->layout ();
 
   SCM flare = me->get_property ("bracket-flare");
   SCM shorten = me->get_property ("shorten-pair");
@@ -63,23 +63,22 @@ Text_spanner::print (SCM smob)
 	    span_points[d] = b->relative_coordinate (common, X_AXIS);
 	}
       else
-	  {
-	    Real encl = robust_scm2double (me->get_property ("enclose-bounds"), 0.0);
-	    Interval ext = b->extent (common, X_AXIS);
-	    
-	    span_points[d] =
-	      robust_relative_extent (b, common, X_AXIS).linear_combination (d * encl);
-
-	    if (is_number_pair (shorten))
-	      span_points -= d * scm_to_double (index_get_cell (shorten, d));
-	  }
-      
+	{
+	  Real encl = robust_scm2double (me->get_property ("enclose-bounds"), 0.0);
+	  Interval ext = b->extent (common, X_AXIS);
+
+	  span_points[d]
+	    = robust_relative_extent (b, common, X_AXIS).linear_combination (d * encl);
+
+	  if (is_number_pair (shorten))
+	    span_points -= d * scm_to_double (index_get_cell (shorten, d));
+	}
+
       if (is_number_pair (flare))
 	span_points -= d * scm_to_double (index_get_cell (flare, d));
     }
   while (flip (&d) != LEFT);
 
-
   SCM properties = Font_interface::text_font_alist_chain (me);
   SCM edge_text = me->get_property ("edge-text");
   Drul_array<Stencil> edge;
@@ -90,55 +89,57 @@ Text_spanner::print (SCM smob)
 	{
 	  if (broken[d])
 	    continue;
-	  
+
 	  SCM text = index_get_cell (edge_text, d);
 
-	  if (Text_interface::markup_p (text)) 
+	  if (Text_interface::is_markup (text))
 	    edge[d] = *unsmob_stencil (Text_interface::interpret_markup (layout->self_scm (), properties, text));
-	  
+
 	  if (!edge[d].is_empty ())
 	    edge[d].align_to (Y_AXIS, CENTER);
 	}
       while (flip (&d) != LEFT);
     }
-  
+
   Drul_array<Real> edge_height = robust_scm2interval (me->get_property ("edge-height"),
 						      Interval (0.0, 0.0));
   Drul_array<Stencil> edge_line;
-    {
-      Direction d = LEFT;
-      int dir = to_dir (me->get_property ("direction"));
-      do
-	{
-	  if (broken[d])
-	    continue;
-	  
-	  Real dx = 0.0;
-	  if (is_number_pair (flare))
-	    dx = scm_to_double (index_get_cell (flare, d)) * d;
-
-	  Real dy = - dir * edge_height[d] ;
-	  if (dy)
-	    edge_line[d] = Line_spanner::line_stencil (me, Offset (0,0), Offset (dx, dy));
-	}
-      while (flip (&d) != LEFT);
-    }
-  
+  {
+    Direction d = LEFT;
+    int dir = to_dir (me->get_property ("direction"));
+    do
+      {
+	if (broken[d])
+	  continue;
+
+	Real dx = 0.0;
+	if (is_number_pair (flare))
+	  dx = scm_to_double (index_get_cell (flare, d)) * d;
+
+	Real dy = -dir * edge_height[d];
+	if (dy)
+	  edge_line[d] = Line_spanner::line_stencil (me, Offset (0, 0), Offset (dx, dy));
+      }
+    while (flip (&d) != LEFT);
+  }
+
   Stencil m;
   do
     {
       Interval ext = edge[d].extent (X_AXIS);
       if (!ext.is_empty ())
 	{
+	  Real pad = robust_scm2double (me->get_property ("bound-padding"), 0.0);
 	  edge[d].translate_axis (span_points[d], X_AXIS);
 	  m.add_stencil (edge[d]);
-	  span_points[d] += -d *  ext[-d];
+	  span_points[d] += -d * (ext[-d] + pad);
 	}
     }
   while (flip (&d) != LEFT);
+
   do
     {
-      if (d* span_points[d] > d * edge[-d].extent (X_AXIS)[d])
+      if (d * span_points[d] > d * edge[-d].extent (X_AXIS)[d])
 	{
 	  edge_line[d].translate_axis (span_points[d], X_AXIS);
 	  m.add_stencil (edge_line[d]);
@@ -146,17 +147,32 @@ Text_spanner::print (SCM smob)
     }
   while (flip (&d) != LEFT);
 
-  if (!span_points.is_empty ())
+  
+  if (!span_points.is_empty ()
+      && span_points.length () > robust_scm2double (me->get_property ("dash-period"), 0.0))
     {
-      Stencil l = Line_spanner::line_stencil (me, Offset (span_points[LEFT], 0),
-					       Offset (span_points[RIGHT], 0));
+      Stencil l = Line_spanner::line_stencil (me,
+					      Offset (span_points[LEFT], 0),
+					      Offset (span_points[RIGHT], 0));
       m.add_stencil (l);
     }
   m.translate_axis (- me->relative_coordinate (common, X_AXIS), X_AXIS);
   return m.smobbed_copy ();
 }
 
-ADD_INTERFACE (Text_spanner,"text-spanner-interface",
+ADD_INTERFACE (Text_spanner,
+
 	       "generic text spanner",
-	       "dash-period dash-fraction edge-height bracket-flare edge-text shorten-pair style thickness enclose-bounds");
+
+	       "bound-padding "
+	       "bracket-flare "
+	       "dash-fraction "
+	       "dash-period "
+	       "edge-height "
+	       "edge-text "
+	       "enclose-bounds "
+	       "shorten-pair "
+	       "style "
+	       "thickness "
+	       );