Re: DynamicTextSpanner is not fully contained in parent spanner

[Jay Anderson]

On Fri, Sep 10, 2010 at 12:52 AM, Urs Liska <address@hidden> wrote:
> Am 10.09.2010 06:33, schrieb Jay Anderson:
>>
>> \version "2.13.32"
>>
>> \score
>> {
>>   \new Staff \relative c'
>>   {
>>     %Works fine over break:
>>     c1\cresc
>>     \break
>>     c1\f
>>
>>     \override DynamicTextSpanner #'style = #'none
>>     c1\cresc
>>     \break
>>     c1\f
>>   }
>> }
>>
>> dts.ly:13:6: programming error: Spanner `DynamicTextSpanner' is not
>> fully contained in parent spanner. Ignoring orphaned part
>>     c1
>>       \cresc
>>
>
> I can't tell if there is some valid reason that this shouldn't be allowed.
> So I added is as http://code.google.com/p/lilypond/issues/detail?id=1259

This is definitely a hack, but it works for me:

diff --git a/lily/spanner.cc b/lily/spanner.cc
index 32e0d21..827f5d6 100644
--- a/lily/spanner.cc
+++ b/lily/spanner.cc
@@ -112,6 +112,8 @@ Spanner::do_break_processing ()

          bool ok = parent_rank_slice.contains
(bounds[LEFT]->get_column ()->get_rank ());
          ok = ok && parent_rank_slice.contains
(bounds[RIGHT]->get_column ()->get_rank ());
+
+          ok = ok || ly_symbol2scm("none") == get_property ("style");

          if (!ok)
            {

I haven't spent the time to understand how things work yet, but this
at least gets me past this problem for now.

-----Jay