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