[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Changes spanner-placement to a ly:dir. (issue4286042)
From: |
Han-Wen Nienhuys |
Subject: |
Re: Changes spanner-placement to a ly:dir. (issue4286042) |
Date: |
Sat, 12 Mar 2011 11:20:17 -0300 |
[+lilydevel]
On Sat, Mar 12, 2011 at 11:17 AM, <address@hidden> wrote:
>
> http://codereview.appspot.com/4286042/diff/2001/lily/balloon.cc
> File lily/balloon.cc (right):
>
> http://codereview.appspot.com/4286042/diff/2001/lily/balloon.cc#newcode77
> lily/balloon.cc:77: return SCM_EOL;
> not sure I follow. Why not
>
> if (!me->spanned_rank().contains(pos)) return
>
> ?
>
> also, you couldl just look at me->break_index_ to see if you're on the
> right piece?
>
> wanted = (d ==LEFT) ? parent->broken_.begin() : parent->broken_.end()-1;
> if (me!=wanted) return
>
> http://codereview.appspot.com/4286042/diff/2001/lily/system.cc
> File lily/system.cc (right):
>
> http://codereview.appspot.com/4286042/diff/2001/lily/system.cc#newcode272
> lily/system.cc:272: if (pos < (int)start)
> new style casts.
>
> http://codereview.appspot.com/4286042/
>
--
Han-Wen Nienhuys - address@hidden - http://www.xs4all.nl/~hanwen
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: Changes spanner-placement to a ly:dir. (issue4286042),
Han-Wen Nienhuys <=