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

[Thomas Morley]

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/

Cheers,
  Harm