re: [PATCH 1/2] Fix spelling definiton - definition

2012-01-20 Thread Lilyfan
Message du 20/01/12 03:42 De : Stefan Weil A : lilypond-devel@gnu.org Copie à : Stefan Weil Objet : [PATCH 1/2] Fix spelling definiton - definition Signed-off-by: Stefan Weil --- po/cs.po po/de.po po/el.po po/es.po po/fr.po po/it.po po/ja.po po/lilypond.pot po/nl.po po/vi.po

Re: [PATCH 1/2] Fix spelling definiton - definition

2012-01-20 Thread James
Jean-Charles, On 20 January 2012 08:02, Lilyfan lily...@orange.fr wrote: Message du 20/01/12 03:42 De : Stefan Weil A : lilypond-devel@gnu.org Copie à : Stefan Weil Objet : [PATCH 1/2] Fix spelling definiton - definition Signed-off-by: Stefan Weil --- po/cs.po ... Please *never* patch

Re: [PATCH 1/2] Fix spelling definiton - definition

2012-01-20 Thread Lilyfan
Message du 20/01/12 09:15 De : James A : Lilyfan Copie à : StefanWeil , lilypond-devel@gnu.org Objet : Re: [PATCH 1/2] Fix spelling definiton - definition Jean-Charles, On 20 January 2012 08:02, Lilyfan wrote: Message du 20/01/12 03:42 De : Stefan Weil A : lilypond-devel@gnu.org

Re: Don't wrap EventChord around rhythmic events by default. (issue 5440084)

2012-01-20 Thread dak
Ok, this latest iteration is quite close to what I want to end up committing. All the work of picking apart articulations and events have gone into the rhythmic event iterator. What it does is to broadcast all articulations that have a listener, and keep the rest as articulations. One side

Re: Issue 2100: Explanation of branches for CG (issue 5539062)

2012-01-20 Thread janek . lilypond
LGTM. Good job, Carl! http://codereview.appspot.com/5539062/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: lilypond-book: Fix links in texinfo output (issue 2224). (issue 5557056)

2012-01-20 Thread graham
LGTM http://codereview.appspot.com/5557056/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: lilypond-book: Group line-width settings together (issue 2222). (issue 5553056)

2012-01-20 Thread graham
LGTM http://codereview.appspot.com/5553056/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: Update lilygit.tcl (Issue 2092) (issue 5504092)

2012-01-20 Thread graham
LGTM http://codereview.appspot.com/5504092/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: Don't wrap EventChord around rhythmic events by default. (issue 5440084)

2012-01-20 Thread md5i . mail
http://codereview.appspot.com/5440084/diff/7001/lily/parser.yy File lily/parser.yy (right): http://codereview.appspot.com/5440084/diff/7001/lily/parser.yy#newcode432 lily/parser.yy:432: %type scm list_music I must *strongly* recommend that the name of either music_list or list_music be changed.

Re: Don't wrap EventChord around rhythmic events by default. (issue 5440084)

2012-01-20 Thread mtsolo
http://codereview.appspot.com/5440084/diff/11001/lily/rhythmic-music-iterator.cc File lily/rhythmic-music-iterator.cc (right): http://codereview.appspot.com/5440084/diff/11001/lily/rhythmic-music-iterator.cc#newcode62 lily/rhythmic-music-iterator.cc:62: if (scm_is_true ly_lily_module_constant

Re: Don't wrap EventChord around rhythmic events by default. (issue 5440084)

2012-01-20 Thread dak
http://codereview.appspot.com/5440084/diff/7001/lily/parser.yy File lily/parser.yy (right): http://codereview.appspot.com/5440084/diff/7001/lily/parser.yy#newcode432 lily/parser.yy:432: %type scm list_music On 2012/01/20 13:55:56, md5i wrote: I must *strongly* recommend that the name of either

Re: Don't wrap EventChord around rhythmic events by default. (issue 5440084)

2012-01-20 Thread dak
On 2012/01/20 14:08:18, MikeSol wrote: http://codereview.appspot.com/5440084/diff/11001/lily/rhythmic-music-iterator.cc File lily/rhythmic-music-iterator.cc (right): http://codereview.appspot.com/5440084/diff/11001/lily/rhythmic-music-iterator.cc#newcode62

Re: Don't wrap EventChord around rhythmic events by default. (issue 5440084)

2012-01-20 Thread dak
On 2012/01/20 14:55:38, dak wrote: On 2012/01/20 14:08:18, MikeSol wrote: Otherwise, it's better to use the C++ function. In this case: ly_is_listened_event_class (in translator.cc) Will do. Very funny. After putting the (required) prototype into translator.hh, it takes hours to

Re: Don't wrap EventChord around rhythmic events by default. (issue 5440084)

2012-01-20 Thread mtsolo
On 2012/01/20 15:33:07, dak wrote: Should I be forking the rhythmic-music-iterator stuff into the separate review I'd say leave it with this - it's easy to forget that two patches need to be tested in concert. http://codereview.appspot.com/5440084/

Re: [PATCH 1/2] Fix spelling definiton - definition

2012-01-20 Thread Stefan Weil
Am 20.01.2012 09:02, schrieb Lilyfan: Message du 20/01/12 03:42 De : Stefan Weil A : lilypond-devel@gnu.org Copie à : Stefan Weil Objet : [PATCH 1/2] Fix spelling definiton - definition Signed-off-by: Stefan Weil --- po/cs.po po/de.po po/el.po po/es.po po/fr.po po/it.po po/ja.po po/lilypond.pot

Re: [PATCH 1/2] Fix spelling definiton - definition

2012-01-20 Thread Francisco Vila
2012/1/20 Stefan Weil s...@weilnetz.de: Salut, my patch also fixes the English text in python/convertrules.py. As soon as this file is changed, all po files no longer have a translation for this text because they are still referring to the old wrong text. That's a scenario which is not

[PATCH] Fix spelling definiton - definition

2012-01-20 Thread Stefan Weil
Signed-off-by: Stefan Weil s...@weilnetz.de --- python/convertrules.py |2 +- 1 file changed, 1 insertions(+), 1 deletions(-) diff --git a/python/convertrules.py b/python/convertrules.py index 32cb5a5..b7d845f 100644 --- a/python/convertrules.py +++ b/python/convertrules.py @@ -392,7 +392,7

Re: Don't wrap EventChord around rhythmic events by default. (issue 5440084)

2012-01-20 Thread n . puttock
Hi David, Should I wait for a new patch or can I test using the latest one here? I've tried it out briefly on a real music example, and have a problem with identifiers: foo = \mark \default \relative c' { \foo c1 } - error: syntax error, unexpected EVENT_IDENTIFIER \foo If I add a note

Re: [PATCH] Fix spelling definiton - definition

2012-01-20 Thread Francisco Vila
2012/1/20 Stefan Weil s...@weilnetz.de: Signed-off-by: Stefan Weil s...@weilnetz.de 1.7.2.5 Applied, thanks. It would be better if you send patches as an attachment. Also, to keep your name in the author field you should make the patch with the git format-patch command. -- Francisco Vila.

[PATCH] Spelling fixes

2012-01-20 Thread Stefan Weil
Hi, my last two patches were sent inline (with git send-email), but I just learned that they should have been appended instead. So here are both patches again. Both fix some spelling issues. Kind regards, Stefan Weil From 808f92bccd5efff1348d614209d933da98f37ac2 Mon Sep 17 00:00:00 2001

Re: Don't wrap EventChord around rhythmic events by default. (issue 5440084)

2012-01-20 Thread David Kastrup
n.putt...@gmail.com writes: Hi David, Should I wait for a new patch or can I test using the latest one here? I've tried it out briefly on a real music example, and have a problem with identifiers: foo = \mark \default \relative c' { \foo c1 } You have the newest. The problem is

Re: Don't wrap EventChord around rhythmic events by default. (issue 5440084)

2012-01-20 Thread dak
On 2012/01/20 17:33:44, dak wrote: mailto:n.putt...@gmail.com writes: Hi David, Should I wait for a new patch or can I test using the latest one here? I've tried it out briefly on a real music example, and have a problem with identifiers: foo = \mark \default \relative c' {

Re: Don't wrap EventChord around rhythmic events by default. (issue 5440084)

2012-01-20 Thread Neil Puttock
On 20 January 2012 17:32, David Kastrup d...@gnu.org wrote: n.putt...@gmail.com writes: Hi David, Should I wait for a new patch or can I test using the latest one here? I've tried it out briefly on a real music example, and have a problem with identifiers: foo = \mark \default

The LilyPond Report #23 has been released

2012-01-20 Thread David Kastrup
Greetings! After a long hiatus, the LilyPond Report #23 URL:http://news.lilynet.net/?The-LilyPond-Report-23 has been released on 2012-01-20. The report contains current news about the development of LilyPond, the GNU Music Typesetter URL:http://www.lilypond.org/ and features

Re: Don't wrap EventChord around rhythmic events by default. (issue 5440084)

2012-01-20 Thread dak
On 2012/01/20 17:59:40, Neil Puttock wrote: I can't think of anything better than adding another type such as `command-event' to things like TempoChangeEvent and MarkEvent and filtering that out too. Emacs stands for Editor Macros. ;; Keyboard Macro Editor. Press C-c C-c to finish; press

Re: Don't wrap EventChord around rhythmic events by default. (issue 5440084)

2012-01-20 Thread Carl . D . Sorensen
Wow -- what a lot of work. And it really seems to be cleaning things up! Thanks! http://codereview.appspot.com/5440084/diff/11001/scm/modal-transforms.scm File scm/modal-transforms.scm (right): http://codereview.appspot.com/5440084/diff/11001/scm/modal-transforms.scm#newcode129

Re: [PATCH] Spelling fixes

2012-01-20 Thread Pavel Roskin
On Fri, 20 Jan 2012 17:58:47 +0100 Stefan Weil s...@weilnetz.de wrote: Hi, my last two patches were sent inline (with git send-email), but I just learned that they should have been appended instead. So here are both patches again. Both fix some spelling issues. By the way, somebody

Re: [PATCH] Spelling fixes

2012-01-20 Thread James
Hello, On 20 January 2012 16:58, Stefan Weil s...@weilnetz.de wrote: Hi, my last two patches were sent inline (with git send-email), but I just learned that they should have been appended instead. So here are both patches again. Both fix some spelling issues. Kind regards, Stefan Weil

Re: [PATCH] Spelling fixes

2012-01-20 Thread Francisco Vila
2012/1/21 James pkx1...@gmail.com: Hello, On 20 January 2012 16:58, Stefan Weil s...@weilnetz.de wrote: Hi, my last two patches were sent inline (with git send-email), but I just learned that they should have been appended instead. So here are both patches again. Both fix some spelling

Re: [PATCH] Spelling fixes

2012-01-20 Thread Stefan Weil
Am 21.01.2012 00:10, schrieb Pavel Roskin: On Fri, 20 Jan 2012 17:58:47 +0100 Stefan Weils...@weilnetz.de wrote: Hi, tre my last two patches were sent inline (with git send-email), but I just learned that they should have been appended instead. So here are both patches again. Both fix some

Re: Don't wrap EventChord around rhythmic events by default. (issue 5440084)

2012-01-20 Thread dak
http://codereview.appspot.com/5440084/diff/11001/scm/modal-transforms.scm File scm/modal-transforms.scm (right): http://codereview.appspot.com/5440084/diff/11001/scm/modal-transforms.scm#newcode129 scm/modal-transforms.scm:129: (define (make-scale music) On 2012/01/20 23:02:21, Carl wrote: I'm