* lily/break-substitution.cc (substitute_mutable_property_alist):

only put property back into value if != SCM_UNDEFINED

* lily/lyric-extender.cc (print): don't take common refpoint of
null object if right_text isn't there.

diff --git a/ChangeLog b/ChangeLog
index 26f1950f65ad94a7332e0741ec0e597ab7a0ab45..c388998457412b29bde2c87aee762a6ca2e19794 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,11 @@
 2004-10-03  Han-Wen Nienhuys  <hanwen@xs4all.nl>
 
+       * lily/break-substitution.cc (substitute_mutable_property_alist):
+       only put property back into value if != SCM_UNDEFINED
+
+       * lily/lyric-extender.cc (print): don't take common refpoint of
+       null object if right_text isn't there.
+ 
        * lily/slur-scoring.cc (struct Slur_score_state): new
        struct. Collect scoring function arguments.
        (struct Slur_score_state): add musical_dy_

diff --git a/lily/break-substitution.cc b/lily/break-substitution.cc
index 2238f4495751a7f87a3946502670258d52dbfd3e..3ccc16c09063bbc80beeb2b0dd8059280673ed32 100644 (file)
--- a/lily/break-substitution.cc
+++ b/lily/break-substitution.cc
@@ -478,10 +478,17 @@ substitute_mutable_property_alist (SCM alist)
       else
        val = do_break_substitution (val);
 
-      *tail = scm_cons (scm_cons (sym, val), SCM_EOL);
-      tail = SCM_CDRLOC (*tail);
-    }
 
+      if (val != SCM_UNDEFINED)
+       {
+         /*
+           for ly:grob? properties, SCM_UNDEFINED could leak out
+           through ly:grob-property
+          */
+         *tail = scm_cons (scm_cons (sym, val), SCM_EOL);
+         tail = SCM_CDRLOC (*tail);
+       }
+    }
   return l;
 }
 

diff --git a/lily/lyric-extender.cc b/lily/lyric-extender.cc
index 6d1c037c003ecc4ae2601fa47367f37f6507e059..bae1d9bbfe25c17fb8a37c31b609e71aa796c3fa 100644 (file)
--- a/lily/lyric-extender.cc
+++ b/lily/lyric-extender.cc
@@ -25,10 +25,14 @@ SCM
 Lyric_extender::print (SCM smob) 
 {
   Spanner *me = unsmob_spanner (smob);
-  Item *le = me->get_bound (LEFT);
+  Item *left_edge = me->get_bound (LEFT);
   Item *right_text = unsmob_item (me->get_property ("next"));
   
-  Grob *common = le->common_refpoint (right_text, X_AXIS);
+  Grob *common = left_edge;
+
+  if (right_text)
+    common = common->common_refpoint (right_text, X_AXIS);
+  
   common = common->common_refpoint (me->get_bound (RIGHT), X_AXIS);
   Real sl = me->get_paper ()->get_dimension (ly_symbol2scm ("linethickness"));  
   Link_array<Grob> heads (Pointer_group_interface__extract_grobs (me, (Grob*)0,
@@ -40,12 +44,12 @@ Lyric_extender::print (SCM smob)
   common = common_refpoint_of_array (heads, common, X_AXIS);
   
   Real left_point = 0.0;
-  if (le->internal_has_interface (ly_symbol2scm ("lyric-syllable-interface")))
-    left_point = le->extent (common, X_AXIS)[RIGHT];
+  if (left_edge->internal_has_interface (ly_symbol2scm ("lyric-syllable-interface")))
+    left_point = left_edge->extent (common, X_AXIS)[RIGHT];
   else if (heads.size ())
     left_point = heads[0]->extent (common, X_AXIS)[LEFT];
   else
-    left_point = le->extent (common, X_AXIS)[RIGHT];
+    left_point = left_edge->extent (common, X_AXIS)[RIGHT];
 
   if (isinf (left_point))
     return SCM_EOL;
@@ -66,7 +70,7 @@ Lyric_extender::print (SCM smob)
     right_point = right_point <? (robust_relative_extent (right_text, common, X_AXIS)[LEFT] - pad);
 
   /* run to end of line. */
-  right_point = right_point >? (me->get_bound (RIGHT)->extent (common, X_AXIS)[LEFT] - pad);
+  right_point = right_point >? (robust_relative_extent (me->get_bound (RIGHT), common, X_AXIS)[LEFT] - pad);
   
   left_point += pad;