RFR [9] 8133188: docs: replace <tt> tags (obsolete in html5) for java.util
Alexander Stepanov
alexander.v.stepanov at oracle.com
Tue Aug 11 11:33:57 UTC 2015
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.stepanov at oracle.com
> <mailto:alexander.v.stepanov at oracle.com>> wrote:
>
> Hello,
>
> Could you please review the following fix:
> http://cr.openjdk.java.net/~avstepan/8133188/webrev.01
> <http://cr.openjdk.java.net/%7Eavstepan/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
> <http://cr.openjdk.java.net/%7Eavstepan/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)}"
> (e.g.,
> http://cr.openjdk.java.net/~avstepan/8133188/webrev.01/src/java.base/share/classes/java/util/LinkedList.java.udiff.html
> <http://cr.openjdk.java.net/%7Eavstepan/8133188/webrev.01/src/java.base/share/classes/java/util/LinkedList.java.udiff.html>);
> - please see JDK-8133188 comments.
>
> Thanks,
> Alexander
>
>
More information about the core-libs-dev
mailing list