Re: Clef-BarLine crash with changed break-align-orders

[Thomas Morley]

2015-12-13 14:02 GMT+01:00 Thomas Morley <address@hidden>:
> 2015-12-13 13:39 GMT+01:00 Thomas Morley <address@hidden>:
>> 2015-12-11 22:24 GMT+01:00 Robin Bannister <address@hidden>:
>>>  Noeck wrote:
>>>
>>>> This puts the clef after the bar line. However, there is not enough
>>>> space between the bar line and the clef.
>>>
>>>
>>> This looks like an old weakness.
>>> The BarLine rework left it much the same.
>>
>>
>> Indeed.
>> In the code for the attached image I tested BarLines successively
>> incresing hair-thickness (the value is displayed), ref-points are
>> displayed as well.
>> In the first place BarLine/Clef-space is consumed by the increasing
>> extent of the BarLine. Only if no space left, Clef moves.
>> Same can be observed if in BarLine.space-alist the setting of
>> key-signature is changed to 'minimum-space.
>> The proper fix seems to be to use 'extra-space for
>> BarLine/Clef-spacing, I'll likely provide a patch. For now:
>>
>> \override Staff.BarLine.space-alist.clef = #'(extra-space . 1)
>>
>> I don't know the reason why 'minimum-space is used. It's in the source
>> at least since release 1.2.0
>> Checkout:
>> abc2e31b608746a1ac924085a87244cc32ee7178
>>
>>
>> Cheers,
>>   Harm
>
> It's now issue 4694 with patch
> https://sourceforge.net/p/testlilyissues/issues/4694/
> https://codereview.appspot.com/277320043/



pushed now, available next release

Cheers,
  Harm