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

[v . villenave]

On 2010/11/22 18:51:55, Reinhold wrote:
> Why do you define two different variables count-markup and
range-markup here,
> one of which is always #f?
> I would simply create one count-markup, defined as (if (pair?
count-or-range)
> (create markup for range) (create markup for count)).
> Then you don't need the (if count-markup...) here, either.

Sorry Reinhold, I misunderstood your count-markup comment (I thought you
were referring to metronome-engraver.cc). You're right, it makes up for
a much nicer code.

New patch set, that addresses Carl's and Neil's comments as well. Thanks
guys, you've been really helpful!

Cheers,
Valentin.

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