[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Minor cleanups in stencil-integral.cc (issue 579630043 by address@hi
From: |
hanwenn |
Subject: |
Re: Minor cleanups in stencil-integral.cc (issue 579630043 by address@hidden) |
Date: |
Sat, 02 May 2020 15:26:12 -0700 |
https://codereview.appspot.com/579630043/diff/555740043/lily/stencil-integral.cc
File lily/stencil-integral.cc (right):
https://codereview.appspot.com/579630043/diff/555740043/lily/stencil-integral.cc#newcode465
lily/stencil-integral.cc:465: // more convoluted, but it's fairly hot
path.
On 2020/04/24 21:18:12, dak wrote:
> Sorry for not being clear: the question was not why this change was
effective in
> saving time, but why it was valid. When thickness is zero, you only
update the
> upper skyline. Why would the lower skyline no longer need updating?
Acknowledged.
https://codereview.appspot.com/579630043/
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: Minor cleanups in stencil-integral.cc (issue 579630043 by address@hidden),
hanwenn <=