Re: Modifies Spanner::spanner_length to better handle line-spanners. (issue4621044)

[Colin Campbell]

On 11-06-16 02:36 AM, address@hidden wrote:
> On Jun 15, 2011, at 10:39 PM, address@hidden wrote:
>
> > LGTM.
> >
> >
> > http://codereview.appspot.com/4621044/diff/1/lily/spanner.cc
> > File lily/spanner.cc (right):
> >
> > http://codereview.appspot.com/4621044/diff/1/lily/spanner.cc#newcode243
> > lily/spanner.cc:243:
> > whitespace
> >
> > http://codereview.appspot.com/4621044/diff/1/lily/spanner.cc#newcode249
> > lily/spanner.cc:249: lr[d] = robust_scm2double (ly_assoc_get
> > (ly_symbol2scm ("X"),
> > indent:
> >
> > do
> >   lr[d] = robust_scm2double (ly_assoc_get (ly_symbol2scm ("X"),
> >
> > http://codereview.appspot.com/4621044/diff/1/lily/spanner.cc#newcode252
> > lily/spanner.cc:252:
> > whitespace
> >
> > http://codereview.appspot.com/4621044/
> Clean regtests, so changes made and pushed.
> d02c9ba13cc3a7c96cc6374e232e94501a833ed1
>
> Cheers,
> MS


Would you update the status of your reitveld issue, please, which is 
still showing as "patch - new"?  Thanks, Mike.

Colin Campbell
Bug Squad

--
A chief event of life is the day in which we have encountered a mind
that startled us.
 -Ralph Waldo Emerson, writer and philosopher (1803-1882)