Thanks! Looks good to me. On Tue, Aug 11, 2015 at 4:33 AM, Alexander Stepanov < alexander.v.stepa...@oracle.com> wrote:
> Hello Martin, > > the changes were reverted for the following files: > > java.base/share/classes/java/util/concurrent/CopyOnWriteArrayList.java > java.base/share/classes/java/util/concurrent/CopyOnWriteArraySet.java > java.base/share/classes/java/util/AbstractQueue.java > java.base/share/classes/java/util/Deque.java > java.base/share/classes/java/util/NavigableMap.java > java.base/share/classes/java/util/NavigableSet.java > > webrev updated: > http://cr.openjdk.java.net/~avstepan/8133188/webrev.02 > > specdiff report (please update the web page): > > http://cr.openjdk.java.net/~avstepan/8133188/specdiff.util/overview-summary.html > (the changes in concurrent are seemingly due to inherited documentation). > > Regards, > Alexander > > > On 8/10/2015 5:28 PM, Martin Buchholz wrote: > > Hi Alexander, > > I've been hoping that someone does cleanups of the jdk as you are doing > now. > > Some of these source files are part of jsr166 and are maintained > separately, in jsr166 CVS. > http://g.oswego.edu/dl/concurrency-interest/ > And that has already been <tt>/<code>-cleaned (although it has been done > slightly differently) and will be integrated into jdk9 in an upcoming > merge. So please leave out: > concurrent/ > *Queue.java > *Deque.java > Navigable*.java > > On Mon, Aug 10, 2015 at 6:39 AM, Alexander Stepanov < > alexander.v.stepa...@oracle.com> wrote: > >> Hello, >> >> Could you please review the following fix: >> http://cr.openjdk.java.net/~avstepan/8133188/webrev.01 >> for >> https://bugs.openjdk.java.net/browse/JDK-8133188 >> >> Just a next portion of "<tt>" tags removed. >> >> specdiff report: >> >> http://cr.openjdk.java.net/~avstepan/8133188/specdiff.util/overview-summary.html >> - some constructs like >> "<tt>(o==null ? e==null : o.equals(e))</tt>" >> were replaced with "{@code Objects.equals(o, e)}" >> <%7B@codeObjects.equals(o,e)%7D> >> (e.g., >> http://cr.openjdk.java.net/~avstepan/8133188/webrev.01/src/java.base/share/classes/java/util/LinkedList.java.udiff.html); >> - please see JDK-8133188 comments. >> >> Thanks, >> Alexander >> > > >