Here is a somewhat less inane version: http://cr.openjdk.java.net/~bpb/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> 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> wrote: > >> 168 int gain = step * srcWidth; >> 169 int bound = srcWidth * step; >> >> This looks odd. Is it what you meant ?