Thank you, Lance. I would like second review and a sponsor for this one. (bug: https://bugs.openjdk.java.net/browse/JDK-8043329)
Regards, Sergey. 2016-03-10 14:35 GMT+03:00 Lance @ Oracle <lance.ander...@oracle.com>: > Looks ok > > > <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| > Principal Member of Technical Staff | +1.781.442.2037 > Oracle Java Engineering > 1 Network Drive > Burlington, MA 01803 > lance.ander...@oracle.com > Sent from my iPad > > On Mar 10, 2016, at 5:51 AM, Sergey Ustimenko <merke...@gmail.com> wrote: > > Hi everyone! Could someone plaese review this tiny > javadoc fix and sponsor it? Patch is in attaches. > > > > > diff --git a/src/java.base/share/classes/java/util/Collections.java > b/src/java.base/share/classes/java/util/Collections.java > --- a/src/java.base/share/classes/java/util/Collections.java > +++ b/src/java.base/share/classes/java/util/Collections.java > @@ -2728,7 +2728,7 @@ > * Set s2 = m2.keySet(); // Needn't be in synchronized block > * ... > * synchronized (m) { // Synchronizing on m, not m2 or s2! > - * Iterator i = s.iterator(); // Must be in synchronized block > + * Iterator i = s2.iterator(); // Must be in synchronized block > * while (i.hasNext()) > * foo(i.next()); > * } > > <JDK-8043329.diff> > >
