Re: Fixes all black bars in NR (issue 6345088)

[Phil Holmes]

----- Original Message ----- 
From: <address@hidden>
To: <address@hidden>; <address@hidden>; 
<address@hidden>; <address@hidden>
Cc: <address@hidden>; <address@hidden>
Sent: Wednesday, July 11, 2012 4:23 PM
Subject: Re: Fixes all black bars in NR (issue 6345088)


> LGTM.  Thanks a lot!
>
>
> http://codereview.appspot.com/6345088/diff/1/Documentation/snippets/simultaneous-headword.ly
> File Documentation/snippets/simultaneous-headword.ly (right):
>
> http://codereview.appspot.com/6345088/diff/1/Documentation/snippets/simultaneous-headword.ly#newcode16
> Documentation/snippets/simultaneous-headword.ly:16:
> #'((alignment-distances .(12)))
> Why removing the space after the dot?

There were hundreds of instances of notename space slur like c4 (.  Our 
style guide says there should be no space like c4(.  I automatically 
replaced " (" with "(" and the ". 12" was a victim.  I spotted it and 
corrected it in the online snippet and my system, but I forgot to amend my 
commit.  Thanks for checking.

> http://codereview.appspot.com/6345088/

--
Phil Holmes