[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Second set of volta changes (Issue 2673) (issue 6422061)
From: |
Phil Holmes |
Subject: |
Re: Second set of volta changes (Issue 2673) (issue 6422061) |
Date: |
Tue, 24 Jul 2012 20:40:29 +0100 |
----- Original Message -----
From: <address@hidden>
To: <address@hidden>; <address@hidden>;
<address@hidden>
Cc: <address@hidden>; <address@hidden>
Sent: Tuesday, July 24, 2012 8:15 PM
Subject: Re: Second set of volta changes (Issue 2673) (issue 6422061)
http://codereview.appspot.com/6422061/diff/1/ly/engraver-init.ly
File ly/engraver-init.ly (right):
http://codereview.appspot.com/6422061/diff/1/ly/engraver-init.ly#newcode621
ly/engraver-init.ly:621: ":|" "|:" "||:" "|." ":|:" ":|.|:" ":|.:" ".|"
Trailing whitespace.
Git warned me about that, but I wasn't sure how to handle it in this
context. Given that each quoted string has a space between, is it OK to
simply delete the trailing whitespace here?
http://codereview.appspot.com/6422061/
--
Phil Holmes