Re: [PATCH] Optionally add text to metronome marks (i.e. allow \tempo "Allegro" 4=120)

2008-06-24 Thread Reinhold Kainhofer
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Am Dienstag, 24. Juni 2008 schrieb Neil Puttock: > Reinhold, > > I've just realized your convert rule has the wrong version number: > since 2.11.49 has been released, the new \tempo options will only work > from version 2.11.50. Oops, sorry, I develo

Re: [PATCH] Optionally add text to metronome marks (i.e. allow \tempo "Allegro" 4=120)

2008-06-24 Thread Neil Puttock
Reinhold, I've just realized your convert rule has the wrong version number: since 2.11.49 has been released, the new \tempo options will only work from version 2.11.50. Anybody converting to version 2.11.49 will get an invalid warning if their LilyPond file uses metronomeMarkFormatter. You need

Re: [PATCH] Optionally add text to metronome marks (i.e. allow \tempo "Allegro" 4=120)

2008-06-23 Thread Reinhold Kainhofer
Am Sonntag, 22. Juni 2008 schrieb Han-Wen Nienhuys: > On Thu, Jun 19, 2008 at 3:16 PM, Reinhold Kainhofer > > <[EMAIL PROTECTED]> wrote: > >> Oops, you really got me there. If there is a text, I set the tempoText > >> property, but I forgot that if that property is already set, I need to > >> unset

Re: [PATCH] Optionally add text to metronome marks (i.e. allow \tempo "Allegro" 4=120)

2008-06-21 Thread Han-Wen Nienhuys
On Thu, Jun 19, 2008 at 3:16 PM, Reinhold Kainhofer <[EMAIL PROTECTED]> wrote: >> Oops, you really got me there. If there is a text, I set the tempoText >> property, but I forgot that if that property is already set, I need to >> unset it there... Fixed with the attached patch. > > Since that last

Re: [PATCH] Optionally add text to metronome marks (i.e. allow \tempo "Allegro" 4=120)

2008-06-19 Thread Reinhold Kainhofer
Am Freitag, 13. Juni 2008 schrieb Reinhold Kainhofer: > Am Freitag, 13. Juni 2008 schrieb Neil Puttock: > > 2008/6/12 Reinhold Kainhofer <[EMAIL PROTECTED]>: > > > Am Mittwoch, 11. Juni 2008 schrieb Neil Puttock: > > > > > > Here is an updated patch with all the issues resolved. I've also added > >

Re: [PATCH] Optionally add text to metronome marks (i.e. allow \tempo "Allegro" 4=120)

2008-06-13 Thread Reinhold Kainhofer
Am Freitag, 13. Juni 2008 schrieb Neil Puttock: > 2008/6/12 Reinhold Kainhofer <[EMAIL PROTECTED]>: > > Am Mittwoch, 11. Juni 2008 schrieb Neil Puttock: > > > > Here is an updated patch with all the issues resolved. I've also added a > > regression test, the corresponding PDF file is uploaded at: >

Re: [PATCH] Optionally add text to metronome marks (i.e. allow \tempo "Allegro" 4=120)

2008-06-12 Thread Neil Puttock
2008/6/12 Reinhold Kainhofer <[EMAIL PROTECTED]>: > Am Mittwoch, 11. Juni 2008 schrieb Neil Puttock: > Here is an updated patch with all the issues resolved. I've also added a > regression test, the corresponding PDF file is uploaded at: > http://www.fam.tuwien.ac.at/~reinhold/temp/metronome-text.

Re: [PATCH] Optionally add text to metronome marks (i.e. allow \tempo "Allegro" 4=120)

2008-06-12 Thread Reinhold Kainhofer
Am Mittwoch, 11. Juni 2008 schrieb Neil Puttock: > 2008/6/6 Reinhold Kainhofer <[EMAIL PROTECTED]>: > > Attached is a patch (for current master), which is my first attempt to > > implent this. The patch is not perfect yet, but works really well. What > > it does: - Extend the parser to allow the ab

Re: [PATCH] Optionally add text to metronome marks (i.e. allow \tempo "Allegro" 4=120)

2008-06-11 Thread Neil Puttock
2008/6/11 Reinhold Kainhofer <[EMAIL PROTECTED]>: > How about \tempo "new text"? This does not change the tempoUnitDuration and > tempoUnitCount Sorry, I probably didn't make myself clear: I was thinking rather of \tempo "new text" dur = count, but with only the text visible, while at the same t

Re: [PATCH] Optionally add text to metronome marks (i.e. allow \tempo "Allegro" 4=120)

2008-06-10 Thread Reinhold Kainhofer
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Am Mittwoch, 11. Juni 2008 schrieben Sie: > 2008/6/6 Reinhold Kainhofer <[EMAIL PROTECTED]>: > > Attached is a patch (for current master), which is my first attempt to > > implent this. The patch is not perfect yet, but works really well. What > > it d

Re: [PATCH] Optionally add text to metronome marks (i.e. allow \tempo "Allegro" 4=120)

2008-06-10 Thread Neil Puttock
2008/6/6 Reinhold Kainhofer <[EMAIL PROTECTED]>: > Attached is a patch (for current master), which is my first attempt to implent > this. The patch is not perfect yet, but works really well. What it does: > - Extend the parser to allow the above forms for \tempo > - Add a tempoText property, sim

[PATCH] Optionally add text to metronome marks (i.e. allow \tempo "Allegro" 4=120)

2008-06-06 Thread Reinhold Kainhofer
A tempo indication in general is either a text markup, a note=count or both. So far, lilypond only supported the note=count type of tempo indications in its \tempo command. I'm now trying to extend the \tempo function, so that it includes an optional text markup, which is displayed in the tempo