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

[Carl . D . Sorensen]

On 2010/11/25 11:42:40, Valentin Villenave wrote:
> On 2010/11/25 00:10:01, Neil Puttock wrote:
> > That sounds OK to me.


In scm/translation-functions.scm, I'd prefer to see the argument called
"count", not "count-or-range".  The property is timeUnitCount, IIUC.
The count can either be a number or a range, but it's still a count.

But I won't hold up approval of the patch for this; it's just an
opinion.

Thanks,

Carl


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