Again limited to the scm/ly-files:
LGTM
One nit:
https://codereview.appspot.com/315350043/diff/20001/Documentation/snippets/new/moving-the-ends-of-hairpins.ly
File Documentation/snippets/new/moving-the-ends-of-hairpins.ly (right):
On 2017/01/08 11:37:18, thomasmorley651 wrote:
On 2017/01/07 17:12:57, david.nalesnik wrote:
> Please review. Thanks!
Hi David,
very nice.
I can't review C++-code, but applied the patch and tested with:
\layout { \override Hairpin.layer = 200 \override Hairpin.color = #red
}
{
Thomas Morley wrote Friday, January 13, 2017 9:05 PM
> 2017-01-13 17:05 GMT+01:00 Trevor Daniels :
>>
>> Risto Vääräniemi wrote Friday, January 13, 2017 3:15 PM
>>
>>> On 13 January 2017 at 01:20, Trevor Daniels wrote:
>>>
Interesting. Does