Re: Sketch for fix of issue 307 (issue4813048)

[address@hidden]

On Jul 26, 2011, at 5:01 AM, address@hidden wrote:

> regtest missing.
> 
> 
> http://codereview.appspot.com/4813048/diff/3001/lily/slur-scoring.cc
> File lily/slur-scoring.cc (right):
> 
> http://codereview.appspot.com/4813048/diff/3001/lily/slur-scoring.cc#newcode286
> lily/slur-scoring.cc:286: end_ys[d] += additional_ys + 0.5; // 0.5 =
> m4g1c padding over extra encompass. make tweakable?
> "magic"
> 
> Do you need the 0.5 at all? The quanting and the shape of the slur
> already offer quite some flexibility.
> 

The test case from the tracker doesn't work without the 0.5, which is truly a 
magic number that represents "that which was necessary to make the test case 
work and not look ugly."
I'll add a regtest later today.

Cheers,
MS