Re: Second set of volta changes (Issue 2673) (issue 6422061)

[dak]

On 2012/07/24 19:40:35, email_philholmes.net wrote:
> ----- Original Message -----
> From: <mailto:address@hidden>
> To: <address@hidden>; <address@hidden>;
> <mailto:address@hidden>
> Cc: <address@hidden>;
<mailto: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?

Sure, just the trailing one.  It's probably just a single space or tab.

http://codereview.appspot.com/6422061/