Re: [Patch] Add support for tempo ranges (issue3248042)

[Valentin Villenave]

On 2010/11/22 17:54:18, Carl wrote:
> Valentin,
>
> When the side-by-side diffs don't work, we can't publish comments on the 
> files.

Indeed.  That's unconvenient.

> There's an indentation error in scm/ly-syntax-constructors.scm, line 90.

Oh, indeed. I thought emacs would fix it, but it didn't.

> 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.

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. I'll publish a new patch set shortly.

Thanks!

Valentin.