Re: Doc: Added \compoundMeter function to NR (issue 4837050)

2011-09-12 Thread n . puttock
LGTM. http://codereview.appspot.com/4837050/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: Doc: Added \compoundMeter function to NR (issue 4837050)

2011-09-11 Thread pkx166h
New patch uploaded with corrected snippet edit http://codereview.appspot.com/4837050/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: Doc: Added \compoundMeter function to NR (issue 4837050)

2011-09-11 Thread percival . music . ca
LGTM, not tested compile but I trust James. http://codereview.appspot.com/4837050/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: Doc: Added \compoundMeter function to NR (issue 4837050)

2011-09-11 Thread janek . lilypond
LGTM http://codereview.appspot.com/4837050/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: Doc: Added \compoundMeter function to NR (issue 4837050)

2011-09-10 Thread Peekay Ex
: Doc: Added \compoundMeter function to NR (issue 4837050) http://lists.gnu.org/archive/html/lilypond-devel/2011-08/msg01029.html Message headers include [Reply_to:] to 9 addresses, [To:] to 6 addresses, [Cc:] to 2 more addresses and none of them all is my own address.  So, I am very sorry

Re: Doc: Added \compoundMeter function to NR (issue 4837050)

2011-09-09 Thread Francisco Vila
for a review a couple of times and emailed directly but had no response) and then I was away for a couple of weeks and the patched files are now out of sync. Hello. I've found this in my recycle bin: Re: Doc: Added \compoundMeter function to NR (issue 4837050) http://lists.gnu.org/archive/html

Re: Doc: Added \compoundMeter function to NR (issue 4837050)

2011-09-06 Thread Peekay Ex
Janek On Mon, Sep 5, 2011 at 9:54 PM, janek.lilyp...@gmail.com wrote: James, i see this patch is quite orphaned despite being very nice and ready-to-go. Shall i push it for you? cheers, Janek http://codereview.appspot.com/**4837050/http://codereview.appspot.com/4837050/ Thanks for the

Re: Doc: Added \compoundMeter function to NR (issue 4837050)

2011-09-05 Thread janek . lilypond
James, i see this patch is quite orphaned despite being very nice and ready-to-go. Shall i push it for you? cheers, Janek http://codereview.appspot.com/4837050/ ___ lilypond-devel mailing list lilypond-devel@gnu.org

Re: Doc: Added \compoundMeter function to NR (issue 4837050)

2011-08-24 Thread pkx166h
On 2011/08/08 22:09:09, Trevor Daniels wrote: LGTM Although I'm not sure about editing the other language files. Best wait for Francisco to comment. Francisco, can you tell me if this patch is ok to go as I have touched some of the translated docs to remove this snippet. Thanks

Re: Doc: Added \compoundMeter function to NR (issue 4837050)

2011-08-24 Thread pkx166h
On 2011/08/08 22:09:09, Trevor Daniels wrote: LGTM Although I'm not sure about editing the other language files. Best wait for Francisco to comment. Francisco, can you tell me if this patch is ok to go as I have touched some of the translated docs to remove this snippet. Thank

Re: Doc: Added \compoundMeter function to NR (issue 4837050)

2011-08-22 Thread janek . lilypond
Just to make things clear, LGTM (and sorry for my long absence) Thanks for your work and your patience, James! Janek http://codereview.appspot.com/4837050/ ___ lilypond-devel mailing list lilypond-devel@gnu.org

Re: Doc: Added \compoundMeter function to NR (issue 4837050)

2011-08-19 Thread n . puttock
LGTM. http://codereview.appspot.com/4837050/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel