Re: \divisioMaior too much largest than the older versions since 2.15.

[David Kastrup]

David Kastrup <address@hidden> writes:

> Manuel Burgos <address@hidden> writes:
>
>> divisioMaior = {
>>   \once \override BreathingSign  #'stencil = 
>> #ly:breathing-sign::divisio-maior
>>   \once \override BreathingSign  #'Y-offset = #0
>>   \breathe
>> }
>
>> \score {
>>   \relative c'' {
>>     g2 a4 g
>>     \divisioMaior
>>   }
>>   \layout {
>>     \context {
>>       \Staff
>>       \remove Bar_engraver
>>     }
>>   }
>> }
>> #############################################################################
>>
>> It's right working in  2.14 versions and previous, but since 2.15 it's wrong
>
>> 2.14 output (that's fine)
>> from http://lilypond.org/doc/v2.14/Documentation/notation/chants-psalms-and-
>> hymns
>>
>> 2.16 output (that's wrong)
>> from http://lilypond.org/doc/v2.16/Documentation/notation/chants-psalms-and-
>> hymns
>
> git blame lily/breathing-sign.cc
>
> shows
>
> d24b0b8c (Benkő Pál 2012-07-21 09:21:06 +0200 84) Interval ydim =
> Staff_symbol_referencer::staff_span (me);
> d24b0b8c (Benkő Pál 2012-07-21 09:21:06 +0200 85) ydim.widen (-0.25 *
> ydim.delta ());
>
> Widening ydim here by some factor involving ydim.delta () very much
> looks like the wrong thing to do since it would presumably scale
> quadratically with size.

That's not what "widen" does, so this part of what I write is wrong.
The problem should be in the vicinity of these lines, anyway.

-- 
David Kastrup