Remove if/else for shift/raise (issue 573770043 by hanw...@gmail.com)

2020-05-01 Thread jonas . hahnfeld
https://codereview.appspot.com/573770043/diff/553990043/lily/stencil-integral.cc File lily/stencil-integral.cc (right): https://codereview.appspot.com/573770043/diff/553990043/lily/stencil-integral.cc#newcode966 lily/stencil-integral.cc:966: copy.raise (y_pos[i] - my_y); hm, the old code passed

Re: Remove if/else for shift/raise (issue 573770043 by hanw...@gmail.com)

2020-05-01 Thread hanwenn
Reviewers: hahnjo, https://codereview.appspot.com/573770043/diff/553990043/lily/stencil-integral.cc File lily/stencil-integral.cc (right): https://codereview.appspot.com/573770043/diff/553990043/lily/stencil-integral.cc#newcode966 lily/stencil-integral.cc:966: copy.raise (y_pos[i] - my_y); On 20

Re: Remove if/else for shift/raise (issue 573770043 by hanw...@gmail.com)

2020-05-02 Thread jonas . hahnfeld
LGTM (provided there are no regtest differences) https://codereview.appspot.com/573770043/