Re: Issue 3917: Add \alternatingTimeSignatures (issue 97110045)

2014-05-10 Thread k-ohara5a5a
On 2014/05/10 09:52:41, Trevor Daniels wrote: (We try to avoid referring to the reader with 'you', Instruction manuals, especially those for a machine, usually make clear what actions the user takes versus what the machine does. This manual doesn't even use third person (as in "The user indica

Re: Issue 3917: Add \alternatingTimeSignatures (issue 97110045)

2014-05-10 Thread k-ohara5a5a
The score is harder to understand with this function, than it was after I expanded the function's contents in-line. The boundary of the action of this function (part timing, mostly printed representation) is hard to understand and remember. This version of the function does partial changes to th

Re: Issue 3917: Add \alternatingTimeSignatures (issue 97110045)

2014-05-10 Thread lilyliska
https://codereview.appspot.com/97110045/diff/80001/ly/music-functions-init.ly File ly/music-functions-init.ly (right): https://codereview.appspot.com/97110045/diff/80001/ly/music-functions-init.ly#newcode144 ly/music-functions-init.ly:144: #{ \once\override Staff.TimeSignature.stencil = On 2014/

Re: Issue 3917: Add \alternatingTimeSignatures (issue 97110045)

2014-05-10 Thread janek . lilypond
Come comments. https://codereview.appspot.com/97110045/diff/80001/ly/music-functions-init.ly File ly/music-functions-init.ly (right): https://codereview.appspot.com/97110045/diff/80001/ly/music-functions-init.ly#newcode144 ly/music-functions-init.ly:144: #{ \once\override Staff.TimeSignature.st

Re: Issue 3917: Add \alternatingTimeSignatures (issue 97110045)

2014-05-10 Thread Phil Holmes
- Original Message - From: "Urs Liska" To: Sent: Saturday, May 10, 2014 12:23 PM Subject: Re: Issue 3917: Add \alternatingTimeSignatures (issue 97110045) Thanks, I'll update accordingly. Generally speaking, should a modification of this kind trigger a new review cycle or can I update

Re: Issue 3917: Add \alternatingTimeSignatures (issue 97110045)

2014-05-10 Thread lilyliska
https://codereview.appspot.com/97110045/diff/60001/Documentation/notation/rhythms.itely File Documentation/notation/rhythms.itely (right): https://codereview.appspot.com/97110045/diff/60001/Documentation/notation/rhythms.itely#newcode1754 Documentation/notation/rhythms.itely:1754: aren't printed

Re: Issue 3917: Add \alternatingTimeSignatures (issue 97110045)

2014-05-10 Thread Urs Liska
Am 10.05.2014 11:52, schrieb tdanielsmu...@googlemail.com: I've suggested some style changes to conform to the rest of the NR. (We try to avoid referring to the reader with 'you', and entering LilyPond code within text, instead leaving the example to speak for itself.) Otherwise, LGTM. Trevor

Re: Issue 3917: Add \alternatingTimeSignatures (issue 97110045)

2014-05-10 Thread tdanielsmusic
I've suggested some style changes to conform to the rest of the NR. (We try to avoid referring to the reader with 'you', and entering LilyPond code within text, instead leaving the example to speak for itself.) Otherwise, LGTM. Trevor https://codereview.appspot.com/97110045/diff/60001/Docume

Strange four-a-ed name in bibliography

2014-05-10 Thread Francisco Vila
Hello. Back in 1998, Han-Wen wrote "Aln Smaill" in Documentation/tex/computer-notation.bib and there it stays for 17 years now (currently under the name Documentation/essay/computer-notation.bib) It could be done on purpose, just to see who plagiarized our bibliography. I find it funny and the

Re: Issue 3917: Add \alternatingTimeSignatures (issue 97110045)

2014-05-10 Thread lilyliska
This is the right one, hopefully better understandable https://codereview.appspot.com/97110045/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: Issue 3917: Add \alternatingTimeSignatures (issue 97110045)

2014-05-10 Thread lilyliska
On 2014/05/10 08:33:44, uliska wrote: rewording doc section Oops? This Patch set is invalid ... https://codereview.appspot.com/97110045/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel