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

[n . puttock]

Hi Valentin,

This generally looks fine (even if it runs a cart and horses through my
patch for issue 1205. ;)

The only thing I'm concerned about is the hybrid type; it's poorly
named: `number-or-pair?' doesn't suggest a number-pair, but
`number-or-number-pair?' is even worse.

I think using a list might a price worth paying to avoid creating a new
predicate.

Cheers,
Neil



http://codereview.appspot.com/3248042/diff/6001/input/regression/metronome-range.ly
File input/regression/metronome-range.ly (right):

http://codereview.appspot.com/3248042/diff/6001/input/regression/metronome-range.ly#newcode11
input/regression/metronome-range.ly:11: \tempo 4 = 66~72
66 ~ 72

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