On 2011/03/07 01:38:36, Colin Campbell wrote:
Ah, I was thinking that \partcombineautomatic once would go automatic
for one
note, then return to whatever was in force before it was called. If
you would
confirm that, I'll reword the explanation to suit.
Yes, that's exactly how it works:
-)
On 2011/03/07 19:48:23, J_lowe wrote:
Layout wise - Looks fine.
Colin,
From: Colin Campbell [mailto:c...@shaw.ca]
Sent: 10 March 2011 13:57
To: James Lowe
Subject: Fwd: part combine doc patch
Good morning, James
Attached is a patch which needs pushing, if you would oblige
---
Pu
Layout wise - Looks fine.
http://codereview.appspot.com/4188056/
___
lilypond-devel mailing list
lilypond-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/lilypond-devel
On 2011/03/06 12:13:15, Reinhold wrote:
http://codereview.appspot.com/4188056/diff/22001/Documentation/notation/simultaneous.itely
File Documentation/notation/simultaneous.itely (right):
http://codereview.appspot.com/4188056/diff/22001/Documentation/notation/simultaneous.itely#newcode871
Doc
http://codereview.appspot.com/4188056/diff/22001/Documentation/notation/simultaneous.itely
File Documentation/notation/simultaneous.itely (right):
http://codereview.appspot.com/4188056/diff/22001/Documentation/notation/simultaneous.itely#newcode871
Documentation/notation/simultaneous.itely:871:
LGTM, apart from a minor nitpick in original text
http://codereview.appspot.com/4188056/diff/22001/Documentation/notation/simultaneous.itely
File Documentation/notation/simultaneous.itely (left):
http://codereview.appspot.com/4188056/diff/22001/Documentation/notation/simultaneous.itely#oldcode8
Patch revised to remove the doc-section.sh bits which were pushed
separately. The remainder is just the partcombine explanation.
Ordinarily, I suppose this needn't go on reitveld, but wotthehell
archie, it started here so I'm putting the last bit up to close out the
process.
http://codereview.app
http://codereview.appspot.com/4188056/diff/20001/scripts/auxiliar/doc-section.sh
File scripts/auxiliar/doc-section.sh (right):
http://codereview.appspot.com/4188056/diff/20001/scripts/auxiliar/doc-section.sh#newcode33
scripts/auxiliar/doc-section.sh:33: FROMDIR="$HOME/lilypond-git"
I totally agr
Am Samstag, 26. Februar 2011, um 21:01:40 schrieben Sie:
> A question for Reinhold, though:
> do I gather correctly that \partcombine is applied to a Staff, and turns
> the combining mechanism on, while \partcombineAutomatic is applied to a
> single Voice?
Not really. When part-combining does its
On 2011/02/26 20:01:39, Colin Campbell wrote:
I like that very much, James, thanks! A question for Reinhold,
though: do I
gather correctly that \partcombine is applied to a Staff, and turns
the
combining mechanism on, while \partcombineAutomatic is applied to a
single
Voice? That being so
Actually thinking about it more would it make more sense to have the
auto
function listed first then add the words "... May need some manual
adjustment.'
and then list the rest?
James
I like that very much, James, thanks! A question for Reinhold, though:
do I gather correctly that \par
...@gmail.com
Cc: re...@codereview.appspotmail.com; lilypond-devel@gnu.org
Subject: Re: DOC: NR 1.5.2 Multiple voices - part combining (issue4188056)
On 2011/02/24 04:29:24, Colin Campbell
> Perhaps I should add something like: decide, but the results may need
adjustment
> in some cases."
On 2011/02/24 04:29:24, Colin Campbell
Perhaps I should add something like: decide, but the results may need
adjustment
in some cases."?
Yes that sounds like a better way of putting it.
http://codereview.appspot.com/4188056/
___
lilypond-devel m
On 2011/02/22 12:15:31, Reinhold wrote:
http://codereview.appspot.com/4188056/diff/1003/Documentation/notation/simultaneous.itely
File Documentation/notation/simultaneous.itely (right):
http://codereview.appspot.com/4188056/diff/1003/Documentation/notation/simultaneous.itely#newcode852
Docum
http://codereview.appspot.com/4188056/diff/1003/Documentation/notation/simultaneous.itely
File Documentation/notation/simultaneous.itely (right):
http://codereview.appspot.com/4188056/diff/1003/Documentation/notation/simultaneous.itely#newcode852
Documentation/notation/simultaneous.itely:852: ch
http://codereview.appspot.com/4188056/diff/2004/Documentation/notation/simultaneous.itely
File Documentation/notation/simultaneous.itely (right):
http://codereview.appspot.com/4188056/diff/2004/Documentation/notation/simultaneous.itely#newcode820
Documentation/notation/simultaneous.itely:820: un
one correction.
http://codereview.appspot.com/4188056/diff/2004/Documentation/notation/simultaneous.itely
File Documentation/notation/simultaneous.itely (right):
http://codereview.appspot.com/4188056/diff/2004/Documentation/notation/simultaneous.itely#newcode820
Documentation/notation/simultane
revised patch uploaded.
http://codereview.appspot.com/4188056/diff/1003/Documentation/notation/simultaneous.itely
File Documentation/notation/simultaneous.itely (right):
http://codereview.appspot.com/4188056/diff/1003/Documentation/notation/simultaneous.itely#newcode846
Documentation/notation/s
http://codereview.appspot.com/4188056/diff/1003/Documentation/notation/simultaneous.itely
File Documentation/notation/simultaneous.itely (right):
http://codereview.appspot.com/4188056/diff/1003/Documentation/notation/simultaneous.itely#newcode846
Documentation/notation/simultaneous.itely:846: ch
On 2011/02/16 06:08:03, Keith wrote:
Looks good as it is,
better if you can add one markup that Reinhold missed in the example.
http://codereview.appspot.com/4188056/diff/1/Documentation/notation/simultaneous.itely
File Documentation/notation/simultaneous.itely (right):
http://codereview.a
http://codereview.appspot.com/4188056/diff/1/Documentation/notation/simultaneous.itely
File Documentation/notation/simultaneous.itely (right):
http://codereview.appspot.com/4188056/diff/1/Documentation/notation/simultaneous.itely#newcode846
Documentation/notation/simultaneous.itely:846: change t
Looks good as it is,
better if you can add one markup that Reinhold missed in the example.
http://codereview.appspot.com/4188056/diff/1/Documentation/notation/simultaneous.itely
File Documentation/notation/simultaneous.itely (right):
http://codereview.appspot.com/4188056/diff/1/Documentation/no
LGTM.
http://codereview.appspot.com/4188056/diff/1/Documentation/notation/simultaneous.itely
File Documentation/notation/simultaneous.itely (right):
http://codereview.appspot.com/4188056/diff/1/Documentation/notation/simultaneous.itely#newcode846
Documentation/notation/simultaneous.itely:846: c
23 matches
Mail list logo
