On 05/11/2015 05:25 PM, Andrew Haley wrote: > On 05/11/2015 04:39 PM, Alan Bateman wrote: >> On 11/05/2015 15:14, Andrew Haley wrote: >>> I mixed up my nextPutIndex and nextGetIndex. Sorry. >>> >>> http://cr.openjdk.java.net/~aph/8079459/ >>> >>> This is also the cause of Bug >>> https://bugs.openjdk.java.net/browse/JDK-8079860 >>> >> This looks good, easy to miss when looking at a big patch. >> >> It would be good to extend the test coverage. In this case it was caught >> by JCK so not an OpenJDK test.
Test coverage is here extended to include boundary conditions for all primitive types: Fix: http://cr.openjdk.java.net/~aph/8079459-2-jdk/ Testcase: http://cr.openjdk.java.net/~aph/8079459-2-hs/ Andrew.