[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Regtest changes phase 1 (issue 6454121)
From: |
PhilEHolmes |
Subject: |
Re: Regtest changes phase 1 (issue 6454121) |
Date: |
Wed, 08 Aug 2012 16:32:38 +0000 |
http://codereview.appspot.com/6454121/diff/1/input/regression/context-mod-with.ly
File input/regression/context-mod-with.ly (right):
http://codereview.appspot.com/6454121/diff/1/input/regression/context-mod-with.ly#newcode4
input/regression/context-mod-with.ly:4: texidoc = "Context modifications
can be stored into a variable as a
The context mods have not been touched. All I've done is documented
what they look like in the regtest, using /mark to do the documentation.
If you're happy with that, this should be good to go.
http://codereview.appspot.com/6454121/diff/1/input/regression/relative-repeat.ly
File input/regression/relative-repeat.ly (right):
http://codereview.appspot.com/6454121/diff/1/input/regression/relative-repeat.ly#newcode10
input/regression/relative-repeat.ly:10: \alternative { a1_"Alt1"
e_"Alt2" b_"Alt3" }
I used the bass clef because without \relative c' it looked daft. I
took the \relative out because it was originally documented as "Relative
mode for repeats" so I thought the \relative was confusing things.
Happy to restore it and get rid of the bass clef, though. Also happy to
add the {} syntax on the alternatives and change the text.
http://codereview.appspot.com/6454121/
- Regtest changes phase 1 (issue 6454121), PhilEHolmes, 2012/08/08
- Re: Regtest changes phase 1 (issue 6454121), tdanielsmusic, 2012/08/08
- Re: Regtest changes phase 1 (issue 6454121), tdanielsmusic, 2012/08/08
- Re: Regtest changes phase 1 (issue 6454121), tdanielsmusic, 2012/08/08
- Re: Regtest changes phase 1 (issue 6454121), tdanielsmusic, 2012/08/08