Re: Changes from LSR doc run (issue 116040043 address@hidden)

[Phil Holmes]

----- Original Message ----- 
From: <address@hidden>
To: <address@hidden>; <address@hidden>; 
<address@hidden>
Cc: <address@hidden>; <address@hidden>
Sent: Sunday, July 20, 2014 5:37 PM
Subject: Re: Changes from LSR doc run (issue 116040043 
address@hidden)


> https://codereview.appspot.com/116040043/diff/30001/Documentation/snippets/adding-indicators-to-staves-which-get-split-after-a-break.ly
> File
> Documentation/snippets/adding-indicators-to-staves-which-get-split-after-a-break.ly
> (right):
>
> https://codereview.appspot.com/116040043/diff/30001/Documentation/snippets/adding-indicators-to-staves-which-get-split-after-a-break.ly#newcode22
> Documentation/snippets/adding-indicators-to-staves-which-get-split-after-a-break.ly:22:
> %{ Original snippet :
> An outcommented "original snippet"?  If the original snippet is of
> separate value, it should be a separate snippet.
>
> https://codereview.appspot.com/116040043/diff/30001/Documentation/snippets/adding-indicators-to-staves-which-get-split-after-a-break.ly#newcode136
> Documentation/snippets/adding-indicators-to-staves-which-get-split-after-a-break.ly:136:
> %%%%%%%%%%%%%%%%%%%%%%% ATTENTION %%%%%%%%%%%%%%%%%%%%%%%%
> Big block comments in French?
>
> In this form I'd lean towards removing the snippet from the import list
> if that's feasible.
>
> https://codereview.appspot.com/116040043/

It is, but as a singer, I find that this is a valuable snippet.  I'll tidy 
it up in the LSR, so (as with the previous comment) I can't fix this patch, 
but will fix the snippet in a following patch.

--
Phil Holmes