+1
-phil.
On 09/02/2016 07:59 AM, Brian Burkhalter wrote:
Here is a somewhat less inane version:
http://cr.openjdk.java.net/~bpb/8154075/webrev.01/
<http://cr.openjdk.java.net/%7Ebpb/8154075/webrev.01/>
All the same tests still pass.
Thanks,
Brian
On Sep 1, 2016, at 1:37 PM, Brian Burkhalter
<brian.burkhal...@oracle.com <mailto:brian.burkhal...@oracle.com>> wrote:
I did not notice that. It is indeed bizarre. I was trying to take
some calculations outside the loops to obtain a small performance
improvement and did not notice the name transposition. I’ll update it.
Thanks,
Brian
On Sep 1, 2016, at 1:27 PM, Phil Race <philip.r...@oracle.com
<mailto:philip.r...@oracle.com>> wrote:
168 int gain = step * srcWidth;
169 int bound = srcWidth * step;
This looks odd. Is it what you meant ?
