Thanks Martin and Martin; I have corrected this along with some additional documentation updates: http://cr.openjdk.java.net/~mduigou/JDK-8020860/3/webrev/ Mike On Apr 16 2014, at 10:47 , Martin Buchholz <martinrb@google.com> wrote:
Here you access elementCount outside the synchronized block, which is a data race + public boolean addAll(int index, Collection<? extends E> c) { if (index < 0 || index > elementCount) throw new ArrayIndexOutOfBoundsException(index);
On Wed, Apr 16, 2014 at 10:30 AM, Mike Duigou <mike.duigou@oracle.com> wrote: Yes. This has been corrected.
Mike
On Apr 16 2014, at 08:19 , Martin Desruisseaux <martin.desruisseaux@geomatys.fr> wrote:
Hello all
Le 15/04/14 18:14, Mike Duigou a écrit :
I have updated the webrev with what I hope is the final form:
The first changes in the javadoc contains "{@code #keys keys}" and "{@code #elements elements}". I presume that you mean {@link} instead of {@code}?
Martin