8020860: cluster Hashtable/Vector field updates for better transactional memory behaviour

[Mike Duigou]

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 at 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 at oracle.com> wrote:
> Yes. This has been corrected.
> 
> Mike
> 
> On Apr 16 2014, at 08:19 , Martin Desruisseaux <martin.desruisseaux at 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:
> >>
> >> http://cr.openjdk.java.net/~mduigou/JDK-8020860/1/webrev/
> >
> > The first changes in the javadoc contains "{@code #keys keys}" and
> > "{@code #elements elements}". I presume that you mean {@link} instead of
> > {@code}?
> >
> >    Martin
> >
> 
>