Re: regression introduced with issue 5122 "Fix not scaling stem in note-by-number-markup"

[Thomas Morley]

2017-06-13 11:24 GMT+02:00 David Kastrup <address@hidden>:
> Thomas Morley <address@hidden> writes:
>
>> As of
>> https://sourceforge.net/p/testlilyissues/issues/5122/#99dc
>>
>> commit eee677c480c78d58a5215e246575aa94ba2d1897
>>
>> introduced a regression.
>>
>> Not sure when I have the time to look deeper into it.
>> At first sight I've no good idea for a proper fix.
>
> Do you have an idea about the underlying problem?
>
>> So I suggest to revert it.
>> At least it should not go into 2.20 in the current state.
>
> Noted.
>
> --
> David Kastrup

I just uploaded a patch
https://sourceforge.net/p/testlilyissues/issues/5231/
https://codereview.appspot.com/337880043/

If it turns out to be a reasonable fix, I don't see any reason anymore to revert
commit eee677c480c78d58a5215e246575aa94ba2d1897
for 2.20.

Cheers,
  Harm