Re: Make \relative { ... } interpret the first pitch as an absolute one (issue 7546044)

2013-03-06 Thread janek . lilypond
Hi, i like the idea of \relative interpreting the first pitch "in absolute" if there's no explicit reference pitch. However, i don't think we should promote this way of doing things: it's a shorthand, and with LilyPond it's usually better to be explicit. So, +1 for changing the behaviour of \rela

Re: Make \relative { ... } interpret the first pitch as an absolute one (issue 7546044)

2013-03-06 Thread dak
On 2013/03/07 05:58:46, Keith wrote: In order from reasonable to ridiculous, Actually, you'll find me in almost full agreement. Converting \relative{g b d} -> \relative c' {g b d} is reasonable, Yes and no. If the meaning of \relative { ... } remains unchanged, this conversion is gratui

Re: Make \relative { ... } interpret the first pitch as an absolute one (issue 7546044)

2013-03-06 Thread k-ohara5a5a
In order from reasonable to ridiculous, Converting \relative{g b d} -> \relative c' {g b d} is reasonable, \relative{sol ti re} -> \relative $(ly:make-pitch 0 0) {sold ti re} mildly annoying. Changing the default starting pitch of \relative down a fifth will annoy maybe 20 people who use \rela

Re: Change \transpose to \relative in ancient.itely (issue 7538043)

2013-03-06 Thread tdanielsmusic
LGTM, apart from a couple of nitpicks. Thanks! Trevor https://codereview.appspot.com/7538043/diff/1/Documentation/notation/ancient.itely File Documentation/notation/ancient.itely (right): https://codereview.appspot.com/7538043/diff/1/Documentation/notation/ancient.itely#newcode1686 Documenta

Change \transpose to \relative in ancient.itely (issue 7538043)

2013-03-06 Thread aleksandr . andreev
Reviewers: , Message: Please review Description: Change \transpose to \relative in ancient.itely Change \transpose c c' syntax to \relative c' syntax in examples in the ancient notation chapter of notation manual. Please review this at https://codereview.appspot.com/7538043/ Affected files:

Re: Change name of Patch Handler to Patch Meister in CG (issue 7498045)

2013-03-06 Thread Graham Percival
On Wed, Mar 06, 2013 at 09:35:00AM +, d...@gnu.org wrote: > On 2013/03/06 09:22:21, mike7 wrote: > >It is a misleading name that should be changed. I thought I was the > patch > >handler when reading that document because I am the one handling my > patch. > > English is your native language,

Re: Change name of Patch Handler to Patch Meister in CG (issue 7498045)

2013-03-06 Thread dak
On 2013/03/06 09:22:21, mike7 wrote: On 6 mars 2013, at 10:13, mailto:d...@gnu.org wrote: > It's a mystery to me how anybody could interpret this text in any other > manner, but there is no point in being less clear than possible, so > LGTM. > > https://codereview.appspot.com/7498045/ A

Re: Change name of Patch Handler to Patch Meister in CG (issue 7498045)

2013-03-06 Thread m...@mikesolomon.org
On 6 mars 2013, at 10:13, d...@gnu.org wrote: > It's a mystery to me how anybody could interpret this text in any other > manner, but there is no point in being less clear than possible, so > LGTM. > > https://codereview.appspot.com/7498045/ A "handler" handles things, i.e. is the person who ph

Change name of Patch Handler to Patch Meister in CG (issue 7498045)

2013-03-06 Thread dak
It's a mystery to me how anybody could interpret this text in any other manner, but there is no point in being less clear than possible, so LGTM. https://codereview.appspot.com/7498045/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lis