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

[Carl . D . Sorensen]

Valentin,

When the side-by-side diffs don't work, we can't publish comments on the
files.

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

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.

Thanks,

Carl


http://codereview.appspot.com/3248042/