----- Original Message -----
From: <lemzw...@googlemail.com>
To: <philehol...@googlemail.com>; <gra...@percival-music.ca>;
<tdanielsmu...@googlemail.com>; <pkx1...@gmail.com>
Cc: <lilypond-devel@gnu.org>; <re...@codereview-hr.appspotmail.com>
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
_______________________________________________
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel
