[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Patch] Add support for tempo ranges (issue3248042)
From: |
Reinhold Kainhofer |
Subject: |
Re: [Patch] Add support for tempo ranges (issue3248042) |
Date: |
Mon, 22 Nov 2010 19:43:26 +0100 |
User-agent: |
KMail/1.13.5 (Linux/2.6.35-22-generic; KDE/4.5.3; i686; ; ) |
Am Montag, 22. November 2010, um 19:05:55 schrieb Valentin Villenave:
> > I'd prefer that we not define a new context property TempoRange.
> > Instead, I'd prefer that we define TempoCount to be either a number or a
> > pair, and modify the code that uses TempoCount so it can handle either
> > one.
I was about to suggest the exact same thing.
> You're absolutely right. The reason why I did that was because I was
> afraid some other code (e.g. performers) may rely on tempoUnitCount,
> but I was wrong.
The Performer uses tempoWholesPerMinute. tempoUnitCount and tempoUnitDuration
are meant for visual display and not be processed any further.
Cheers,
Reinhold
--
------------------------------------------------------------------
Reinhold Kainhofer, address@hidden, http://reinhold.kainhofer.com/
* Financial & Actuarial Math., Vienna Univ. of Technology, Austria
* http://www.fam.tuwien.ac.at/, DVR: 0005886
* LilyPond, Music typesetting, http://www.lilypond.org
- [Patch] Add support for tempo ranges (issue3248042), v . villenave, 2010/11/22
- Re: [Patch] Add support for tempo ranges (issue3248042), Carl . D . Sorensen, 2010/11/22
- Re: [Patch] Add support for tempo ranges (issue3248042), reinhold . kainhofer, 2010/11/22
- Re: [Patch] Add support for tempo ranges (issue3248042), v . villenave, 2010/11/22
- Re: [Patch] Add support for tempo ranges (issue3248042), n . puttock, 2010/11/23
- Re: [Patch] Add support for tempo ranges (issue3248042), Carl . D . Sorensen, 2010/11/23
- Re: [Patch] Add support for tempo ranges (issue3248042), n . puttock, 2010/11/24
- Re: [Patch] Add support for tempo ranges (issue3248042), v . villenave, 2010/11/25
- Re: [Patch] Add support for tempo ranges (issue3248042), Carl . D . Sorensen, 2010/11/25
- Re: [Patch] Add support for tempo ranges (issue3248042), n . puttock, 2010/11/25
- Re: [Patch] Add support for tempo ranges (issue3248042), v . villenave, 2010/11/25
- Re: [Patch] Add support for tempo ranges (issue3248042), n . puttock, 2010/11/25