Re: Fix issues 1259 and 1433 (\breakDynamicSpan and a spanner's style=#'none over a line break) (issue4630070)

[Neil Puttock]

On 3 July 2011 21:27,  <address@hidden> wrote:
>
> http://codereview.appspot.com/4630070/diff/3002/input/regression/dynamics-alignment-breaker-linebreak.ly
> File input/regression/dynamics-alignment-breaker-linebreak.ly (right):
>
> http://codereview.appspot.com/4630070/diff/3002/input/regression/dynamics-alignment-breaker-linebreak.ly#newcode12
> input/regression/dynamics-alignment-breaker-linebreak.ly:12:
> c1_\dim\breakDynamicSpan
> On 2011/07/03 20:14:56, Neil Puttock wrote:
>>
>> redundant \breakDynamicSpan
>
> I don't think so. This \breakDynamicSpan breaks the \dim, while the
> previous on breaks only the \<. The dim will create a new
> DynamicLineSpanner...
>
> http://codereview.appspot.com/4630070/diff/3002/input/regression/dynamics-alignment-breaker-linebreak.ly#newcode15
> input/regression/dynamics-alignment-breaker-linebreak.ly:15:
> c,,1_\dim\breakDynamicSpan
> On 2011/07/03 20:14:56, Neil Puttock wrote:
>>
>> redundant \breakDynamicSpan
>
> See above.

Have you removed them and compared the output?  They're identical, and
both snippets create five DynamicLineSpanner grobs.