[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
RE: Rewrite regtest mozart-hrn-3.ly (issue4811066)
From: |
James Lowe |
Subject: |
RE: Rewrite regtest mozart-hrn-3.ly (issue4811066) |
Date: |
Mon, 8 Aug 2011 15:40:31 +0000 |
commit 403e66a45e752231fb49deea2e8ccb1976505c7c
Thanks Mr Holmes!
James
)-----Original Message-----
)From: address@hidden
)[mailto:address@hidden On
)Behalf Of James Lowe
)Sent: 08 August 2011 16:13
)To: Phil Holmes; address@hidden;
)address@hidden; address@hidden;
)address@hidden; address@hidden;
)address@hidden
)Subject: RE: Rewrite regtest mozart-hrn-3.ly (issue4811066)
)Importance: Low
)
)I'll push it.
)
)james
)
))-----Original Message-----
))From: address@hidden
))[mailto:address@hidden
)On )Behalf Of Phil Holmes
))Sent: 08 August 2011 15:54
))To: address@hidden; address@hidden;
))address@hidden; address@hidden; address@hidden;
))address@hidden; address@hidden
))Subject: Re: Rewrite regtest mozart-hrn-3.ly (issue4811066)
))
))----- Original Message -----
))From: <address@hidden>
))To: <address@hidden>; <address@hidden>;
))<address@hidden>; <address@hidden>
))Cc: <address@hidden>; <address@hidden>
))Sent: Sunday, August 07, 2011 11:42 PM
))Subject: Re: Rewrite regtest mozart-hrn-3.ly (issue4811066)
))
))
))> LGTM. My comment is a tiny nitpick; I don't think it needs to hold up )>
)pushing this.
))>
))>
))>
))http://codereview.appspot.com/4811066/diff/8003/input/regression/m
)o
))zar
))> t-hrn-3.ly File input/regression/mozart-hrn-3.ly (right):
))>
))>
))http://codereview.appspot.com/4811066/diff/8003/input/regression/m
)o
))zar
))> t-hrn-3.ly#newcode27
))> input/regression/mozart-hrn-3.ly:27: \typewriter { )>
)"http://mutopiaproject.org/" } this would be slightly nicer if you )> used
)\url instead of \typewriter.
))> That would use that typeface, but would also produce a clickable link.
))>
))> http://codereview.appspot.com/4811066/
))
))Nitpick changed - I've kept typewriter for the look, but added url. Patch
))attached. Please push.
))
))--
))Phil Holmes
)
)_______________________________________________
)lilypond-devel mailing list
)address@hidden
)https://lists.gnu.org/mailman/listinfo/lilypond-devel
- Rewrite regtest mozart-hrn-3.ly (issue4811066), PhilEHolmes, 2011/08/02
- Re: Rewrite regtest mozart-hrn-3.ly (issue4811066), percival . music . ca, 2011/08/02
- Re: Rewrite regtest mozart-hrn-3.ly (issue4811066), n . puttock, 2011/08/02
- Re: Rewrite regtest mozart-hrn-3.ly (issue4811066), PhilEHolmes, 2011/08/06
- Re: Rewrite regtest mozart-hrn-3.ly (issue4811066), n . puttock, 2011/08/06
- Re: Rewrite regtest mozart-hrn-3.ly (issue4811066), percival . music . ca, 2011/08/07
- Re: Rewrite regtest mozart-hrn-3.ly (issue4811066), Phil Holmes, 2011/08/08
- RE: Rewrite regtest mozart-hrn-3.ly (issue4811066), James Lowe, 2011/08/08
- RE: Rewrite regtest mozart-hrn-3.ly (issue4811066),
James Lowe <=
- Re: Rewrite regtest mozart-hrn-3.ly (issue4811066), Phil Holmes, 2011/08/08
- RE: Rewrite regtest mozart-hrn-3.ly (issue4811066), James Lowe, 2011/08/08
- RE: Rewrite regtest mozart-hrn-3.ly (issue4811066), James Lowe, 2011/08/08
- Re: Rewrite regtest mozart-hrn-3.ly (issue4811066), Phil Holmes, 2011/08/08
- RE: Rewrite regtest mozart-hrn-3.ly (issue4811066), James Lowe, 2011/08/08
- Re: Rewrite regtest mozart-hrn-3.ly (issue4811066), Graham Percival, 2011/08/08
- Re: Rewrite regtest mozart-hrn-3.ly (issue4811066), Phil Holmes, 2011/08/09
- Re: Rewrite regtest mozart-hrn-3.ly (issue4811066), Neil Puttock, 2011/08/09
- Re: Rewrite regtest mozart-hrn-3.ly (issue4811066), Graham Percival, 2011/08/08