[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Doc: add snippet showing flared and constante hairpins (3235) (issue
From: |
Janek Warchoł |
Subject: |
Re: Doc: add snippet showing flared and constante hairpins (3235) (issue 8971043) |
Date: |
Thu, 9 May 2013 11:25:44 +0200 |
thanks! LGTM
2013/5/9 <address@hidden>:
> On 2013/05/09 04:33:17, janek wrote:
>>
>> one question.
>
>
>
> https://codereview.appspot.com/8971043/diff/2001/Documentation/snippets/printing-hairpins-in-various-styles.ly
>>
>> File Documentation/snippets/printing-hairpins-in-various-styles.ly
>
> (right):
>
>
> https://codereview.appspot.com/8971043/diff/2001/Documentation/snippets/printing-hairpins-in-various-styles.ly#newcode22
>>
>> Documentation/snippets/printing-hairpins-in-various-styles.ly:22:
>
> \override
>>
>> Hairpin #'stencil = #flared-hairpin
>> Shouldn't we use new
>> Hairpin.stencil
>> syntax, for uniformity's sake?
>
>
> Good call! Done.
>
> https://codereview.appspot.com/8971043/