RFR [9] 8133188: docs: replace <tt> tags (obsolete in html5) for java.util

Alexander Stepanov alexander.v.stepanov at oracle.com
Tue Aug 11 16:10:50 UTC 2015


Thanks!

Regards,
Alexander

On 8/11/2015 7:10 PM, Martin Buchholz wrote:
> Thanks!   Looks good to me.
>
> On Tue, Aug 11, 2015 at 4:33 AM, Alexander Stepanov 
> <alexander.v.stepanov at oracle.com 
> <mailto:alexander.v.stepanov at 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
>     <http://cr.openjdk.java.net/%7Eavstepan/8133188/webrev.02>
>
>     specdiff report (please update the web page):
>     http://cr.openjdk.java.net/~avstepan/8133188/specdiff.util/overview-summary.html
>     <http://cr.openjdk.java.net/%7Eavstepan/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)}"
>>         <mailto:%7B at codeObjects.equals%28o,e%29%7D>
>>         (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