[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Convert stencil to skyline directly (issue 555760043 by address@hidd
From: |
hanwenn |
Subject: |
Re: Convert stencil to skyline directly (issue 555760043 by address@hidden) |
Date: |
Sat, 02 May 2020 14:44:27 -0700 |
https://codereview.appspot.com/555760043/diff/567530050/lily/stencil-integral.cc
File lily/stencil-integral.cc (right):
https://codereview.appspot.com/555760043/diff/567530050/lily/stencil-integral.cc#newcode684
lily/stencil-integral.cc:684: if (false)
On 2020/05/02 14:50:06, hahnjo wrote:
> this looks odd
Done.
https://codereview.appspot.com/555760043/diff/567530050/lily/stencil-integral.cc#newcode692
lily/stencil-integral.cc:692: ;
On 2020/05/02 14:50:06, hahnjo wrote:
> as all of the following conditions also compare head, I think this can
be
> removed
Done.
https://codereview.appspot.com/555760043/diff/567530050/lily/stencil-integral.cc#newcode717
lily/stencil-integral.cc:717: ;
On 2020/05/02 14:50:06, hahnjo wrote:
> same
Done.
https://codereview.appspot.com/555760043/