<Swing Dev> RFR: JDK-8235818: Inline information from broken external links in java.desktop

Phil Race philip.race at oracle.com
Fri Dec 13 19:07:16 UTC 2019



On 12/12/19 2:40 AM, Prasanta Sadhukhan wrote:
>
> It seems jdk/jdk has introduced stricter check for img tag to have 
> "alt" subtag.
>

I don't think a new check has been introduced.
The old patch fails for me on a client repo that is a whole 7 days out 
of date.

doclint was enabled in the build in January 2013 and we have a bug fix 
in swing on the alt image tag from the same period :
https://bugs.openjdk.java.net/browse/JDK-8025234
>
> Modified webrev to add alt subtag to img tag like this
>
> 201  * <p style="text-align:center"><img src="doc-files/plain1.gif"
>  202  * alt="The preceding text describes this graphic.">
>
> No visible change to the spec for this modification. mach5 is green
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8235818
>
> webrev: http://cr.openjdk.java.net/~psadhukhan/8235818/webrev.0/
>


Ok. Approved for push to jdk/client (not jdk/jdk)

-phil.

> Regards
>
> Prasanta
>
> On 12-Dec-19 12:06 AM, Phil Race wrote:
>> +1
>>
>> -phil.
>>
>> On 12/10/19 2:12 AM, Prasanta Sadhukhan wrote:
>>>
>>> Modified webrev: 
>>> http://cr.openjdk.java.net/~psadhukhan/8227607/webrev.3/
>>>
>>> with updated spec 
>>> http://cr.openjdk.java.net/~psadhukhan/8227607/docs1/api/java.desktop/javax/swing/text/Document.html
>>>
>>> Regards
>>>
>>> Prasanta
>>>
>>> On 09-Dec-19 11:00 PM, Philip Race wrote:
>>>> >
>>>>
>>>>   * ||
>>>>     <http://cr.openjdk.java.net/%7Epsadhukhan/8227607/docs/api/java.desktop/javax/swing/text/Document.html#putProperty%28java.lang.Object,java.lang.Object%29>
>>>>
>>>>
>>>> For more information on the |Document| class, see The Swing 
>>>> Connection 
>>>> <http://www.oracle.com/technetwork/java/javase/tech/articles-jsp-139072.html> 
>>>>
>>>>
>>>>
>>>> This link above  seems to be obsolete as well. I think it should 
>>>> just be removed
>>>>
>>>> > If an element is a a leaf,isLeaf() returns true.
>>>>
>>>> add a space after the comma.
>>>>
>>>> Also can we wrap all  - or at least the ones that make sense -  API 
>>>> references in {@code ... }  ?
>>>>
>>>> Method calls for sure.
>>>>
>>>> interrface -> interface.
>>>>
>>>> -phil.
>>>>
>>>> On 12/9/19, 3:16 AM, Prasanta Sadhukhan wrote:
>>>>>
>>>>> Modified version with new subtitle ""Overview and Programming 
>>>>> Tips" and graphical_schema
>>>>>
>>>>> http://cr.openjdk.java.net/~psadhukhan/8227607/webrev.2/
>>>>>
>>>>> spec change will look like 
>>>>> http://cr.openjdk.java.net/~psadhukhan/8227607/docs/api/java.desktop/javax/swing/text/Document.html
>>>>>
>>>>> Regards
>>>>> Prasanta
>>>>> On 08-Dec-19 1:47 AM, Philip Race wrote:
>>>>>> I got very confused as to whether there was even an updated 
>>>>>> version here.
>>>>>> This conversation became too complex for the inline diff.
>>>>>> I think that it would be best to first generate a webrev put it 
>>>>>> on cr.
>>>>>>
>>>>>>
>>>>>> I do think we need to have a subtitle that prefaces this new 
>>>>>> content rather
>>>>>> than just appending it to "Properties" ..
>>>>>>
>>>>>> I suggest "Overview and Programming Tips" which makes it clearer 
>>>>>> it is not (so much) specification as developer guide.
>>>>>>
>>>>>> > Swing has a interesting interface, called the _|Element| 
>>>>>> <http://cr.openjdk.java.net/%7Epsadhukhan/docs/api/java.desktop/javax/swing/text/Element.html>_,
>>>>>>
>>>>>> I do think this needs rephrasing.
>>>>>>
>>>>>> "Element is an important interface used in constructing a Document.
>>>>>> It has the power ...."
>>>>>>
>>>>>> And each time it says "In Swing" I think, why does it bother 
>>>>>> saying that ...
>>>>>> confusion with an SGML Element ?
>>>>>> Maybe it would read better written as "In the Swing text API's 
>>>>>> document model"
>>>>>>
>>>>>> -phil.
>>>>>>
>>>>>> On 12/6/19, 7:22 PM, Prasanta Sadhukhan wrote:
>>>>>>> Anything more needs to be added?!!! Can it be committed?
>>>>>>>
>>>>>>> On 06-Dec-19 9:49 AM, Prasanta Sadhukhan wrote:
>>>>>>>> I guess I already shared... 
>>>>>>>> http://cr.openjdk.java.net/~psadhukhan/docs/api/java.desktop/javax/swing/text/Document.html
>>>>>>>>
>>>>>>>> Regards
>>>>>>>>
>>>>>>>> Prasanta
>>>>>>>>
>>>>>>>> On 06-Dec-19 4:44 AM, Sergey Bylokhov wrote:
>>>>>>>>> Hi, Prasanta.
>>>>>>>>>
>>>>>>>>> Can you please share the final generated JavaDoc, it is hard 
>>>>>>>>> to read the diff below.
>>>>>>>>>
>>>>>>>>> On 12/4/19 10:09 am, Prasanta Sadhukhan wrote:
>>>>>>>>>> Inlined information from element_interface page into Document 
>>>>>>>>>> class <spec will look like 
>>>>>>>>>> http://cr.openjdk.java.net/~psadhukhan/docs/api/java.desktop/javax/swing/text/Document.html]
>>>>>>>>>>
>>>>>>>>>> /diff -r 53eff6c5da11 
>>>>>>>>>> src/java.desktop/share/classes/javax/swing/Timer.java/
>>>>>>>>>>
>>>>>>>>>> --- a/src/java.desktop/share/classes/javax/swing/Timer.java 
>>>>>>>>>> Sun Oct 06 21:42:50 2019 -0700
>>>>>>>>>> +++ b/src/java.desktop/share/classes/javax/swing/Timer.java 
>>>>>>>>>> Wed Dec 04 22:07:48 2019 +0530
>>>>>>>>>> @@ -124,13 +124,6 @@
>>>>>>>>>>    * <a 
>>>>>>>>>> href="https://docs.oracle.com/javase/tutorial/uiswing/misc/timer.html" 
>>>>>>>>>>
>>>>>>>>>>    * target = "_top">How to Use Timers</a>,
>>>>>>>>>>    * a section in <em>The Java Tutorial.</em>
>>>>>>>>>> *- * For more examples and help in choosing between**
>>>>>>>>>> **- * this <code>Timer</code> class and**
>>>>>>>>>> **- * <code>java.util.Timer</code>,**
>>>>>>>>>> **- * see**
>>>>>>>>>> **- * <a 
>>>>>>>>>> href="http://java.sun.com/products/jfc/tsc/articles/timer/"**
>>>>>>>>>> **- * target="_top">Using Timers in Swing Applications</a>,**
>>>>>>>>>> **- * an article in <em>The Swing Connection.</em>**
>>>>>>>>>> *  * <p>
>>>>>>>>>>    * <strong>Warning:</strong>
>>>>>>>>>>    * Serialized objects of this class will not be compatible 
>>>>>>>>>> with
>>>>>>>>>>
>>>>>>>>>> /diff -r 53eff6c5da11 
>>>>>>>>>> src/java.desktop/share/classes/javax/swing/text/Document.java/
>>>>>>>>>>
>>>>>>>>>> --- 
>>>>>>>>>> a/src/java.desktop/share/classes/javax/swing/text/Document.java 
>>>>>>>>>> Sun Oct 06 21:42:50 2019 -0700
>>>>>>>>>> +++ 
>>>>>>>>>> b/src/java.desktop/share/classes/javax/swing/text/Document.java 
>>>>>>>>>> Wed Dec 04 22:07:48 2019 +0530
>>>>>>>>>> @@ -167,9 +167,38 @@
>>>>>>>>>>    *
>>>>>>>>>>    * <p>For more information on the <code>Document</code> 
>>>>>>>>>> class, see
>>>>>>>>>>    * <a 
>>>>>>>>>> href="http://www.oracle.com/technetwork/java/javase/tech/articles-jsp-139072.html">The 
>>>>>>>>>> Swing Connection</a>
>>>>>>>>>> *- * and most particularly the article,**
>>>>>>>>>> **- * <a 
>>>>>>>>>> href="http://java.sun.com/products/jfc/tsc/articles/text/element_interface">**
>>>>>>>>>> **- * The Element Interface</a>.**
>>>>>>>>>> **+ * <p>Swing has a interesting interface, called the 
>>>>>>>>>> <u>{@link javax.swing.text.Element}</u>, **
>>>>>>>>>> **+ * which has the power to describe various structural 
>>>>>>>>>> parts of a document, **
>>>>>>>>>> **+ * such as paragraphs, lines of text, or even (in HTML 
>>>>>>>>>> documents) items in lists. **
>>>>>>>>>> **+ * Conceptually, the Element interface captures some of 
>>>>>>>>>> the spirit of an SGML document. **
>>>>>>>>>> **+ * So if you know SGML, you may already have some 
>>>>>>>>>> understanding of Swing's Element interface.**
>>>>>>>>>> **+ * <p>In Swing, the interface Element defines a structural 
>>>>>>>>>> piece of a Document, **
>>>>>>>>>> **+ * like a paragraph, a line of text, or a list item in an 
>>>>>>>>>> HTML document.**
>>>>>>>>>> **+ * <p>Every Element is either a branch or a leaf. If an 
>>>>>>>>>> element is a branch, **
>>>>>>>>>> **+ * the isLeaf() method returns false. If an element is a a 
>>>>>>>>>> leaf,isLeaf() returns true.**
>>>>>>>>>> **+ * <p>Branches can have any number of children. Leaves do 
>>>>>>>>>> not have children. **
>>>>>>>>>> **+ * To determine how many children a branch has, you can 
>>>>>>>>>> call getElementCount(). **
>>>>>>>>>> **+ * To determine the parent of an Element, you can call 
>>>>>>>>>> getParentElement(). **
>>>>>>>>>> **+ * Root elements don't have parents, so calling 
>>>>>>>>>> getParentElement() on a root returns null.**
>>>>>>>>>> **+ * <p>An Element represents a specific region in a 
>>>>>>>>>> Document that begins with startOffset **
>>>>>>>>>> **+ * and ends just before endOffset. **
>>>>>>>>>> **+ * The start offset of a branch Element is usually the 
>>>>>>>>>> start offset of its first child. **
>>>>>>>>>> **+ * Similarly, the end offset of a branch Element is 
>>>>>>>>>> usually the end offset of its last child.**
>>>>>>>>>> **+ * <p>Every Element is associated with an AttributeSet 
>>>>>>>>>> that you can access by calling getAttributes(). **
>>>>>>>>>> **+ * In an Element, and AttributeSet is essentially a set of 
>>>>>>>>>> key/value pairs. **
>>>>>>>>>> **+ * These pairs are generally used for markup -- such as 
>>>>>>>>>> determining the Element's **
>>>>>>>>>> *
>>>>>>>>>>
>>>>>>>>>> *+ * foreground color, font size, and so on. But it is up to 
>>>>>>>>>> the model, and the developer, **
>>>>>>>>>> **+ * to determine what is stored in the AttributeSet.**
>>>>>>>>>> **+ * <p>You can obtain the root Element (or Elements) of a 
>>>>>>>>>> Document by calling the **
>>>>>>>>>> **+ * methods getDefaultRootElement() and getRootElements(), 
>>>>>>>>>> which are defined in the Document interface.**
>>>>>>>>>> **+ * <p>The Document interrface is responsible for 
>>>>>>>>>> translating a linear view of the **
>>>>>>>>>> **+ * characters into Element operations. It is up to each 
>>>>>>>>>> Document implementation **
>>>>>>>>>> **+ * to define what the Element structure is.**
>>>>>>>>>> **+ * <p>It is recommended to look into {@link 
>>>>>>>>>> javax.swing.text.EditorKit} and **
>>>>>>>>>> **+ * {@link javax.swing.text.View}.**
>>>>>>>>>> **+ * View is responsible for rendering a particular Element, 
>>>>>>>>>> and **
>>>>>>>>>> **+ * EditorKit is responsible for a ViewFactory that is able 
>>>>>>>>>> to decide what **
>>>>>>>>>> **+ * View should be created based on an Element.*
>>>>>>>>>>
>>>>>>>>>> Regards
>>>>>>>>>>
>>>>>>>>>> Prasanta
>>>>>>>>>>
>>>>>>>>>> On 04-Dec-19 10:45 AM, Sergey Bylokhov wrote:
>>>>>>>>>>> On 12/3/19 9:08 pm, Prasanta Sadhukhan wrote:
>>>>>>>>>>>> Hi Sergey,
>>>>>>>>>>>>
>>>>>>>>>>>> I could not find the page for element_interface itself so 
>>>>>>>>>>>> how to get the information from there?
>>>>>>>>>>>
>>>>>>>>>>> I guess it is the same as you posted before:
>>>>>>>>>>> https://web.archive.org/web/20120418045506/http://java.sun.com/products/jfc/tsc/articles/text/element_interface/ 
>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>>>
>>>>>>>>>>>> Regards
>>>>>>>>>>>>
>>>>>>>>>>>> Prasanta
>>>>>>>>>>>>
>>>>>>>>>>>> On 04-Dec-19 7:31 AM, Sergey Bylokhov wrote:
>>>>>>>>>>>>> Hi, Prasanta.
>>>>>>>>>>>>> Probably we can inline some information from the 
>>>>>>>>>>>>> element_interface.html to the JavaDoc itself? The JavaDoc 
>>>>>>>>>>>>> already has some similar schemes.
>>>>>>>>>>>>>
>>>>>>>>>>>>> On 12/2/19 2:37 am, Prasanta Sadhukhan wrote:
>>>>>>>>>>>>>> Hi All,
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> Please review a doc-fix for removing some dead links from 
>>>>>>>>>>>>>> the specifications. These links were part of java.sun.com 
>>>>>>>>>>>>>> but has not been migrated to docs.oracle.com so it's 
>>>>>>>>>>>>>> better to remove them
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8227607
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> diff -r 7637e77c4c8a 
>>>>>>>>>>>>>> src/java.desktop/share/classes/javax/swing/Timer.java
>>>>>>>>>>>>>> --- 
>>>>>>>>>>>>>> a/src/java.desktop/share/classes/javax/swing/Timer.java 
>>>>>>>>>>>>>> Mon Dec 02 11:58:14 2019 +0530
>>>>>>>>>>>>>> +++ 
>>>>>>>>>>>>>> b/src/java.desktop/share/classes/javax/swing/Timer.java 
>>>>>>>>>>>>>> Mon Dec 02 16:02:08 2019 +0530
>>>>>>>>>>>>>> @@ -124,13 +124,6 @@
>>>>>>>>>>>>>>    * <a 
>>>>>>>>>>>>>> href="https://docs.oracle.com/javase/tutorial/uiswing/misc/timer.html" 
>>>>>>>>>>>>>>
>>>>>>>>>>>>>>    * target = "_top">How to Use Timers</a>,
>>>>>>>>>>>>>>    * a section in <em>The Java Tutorial.</em>
>>>>>>>>>>>>>> *- * For more examples and help in choosing between**
>>>>>>>>>>>>>> **- * this <code>Timer</code> class and**
>>>>>>>>>>>>>> **- * <code>java.util.Timer</code>,**
>>>>>>>>>>>>>> **- * see**
>>>>>>>>>>>>>> **- * <a 
>>>>>>>>>>>>>> href="http://java.sun.com/products/jfc/tsc/articles/timer/"** 
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> **- * target="_top">Using Timers in Swing 
>>>>>>>>>>>>>> Applications</a>,**
>>>>>>>>>>>>>> **- * an article in <em>The Swing Connection.</em>*
>>>>>>>>>>>>>>    * <p>
>>>>>>>>>>>>>>    * <strong>Warning:</strong>
>>>>>>>>>>>>>>    * Serialized objects of this class will not be 
>>>>>>>>>>>>>> compatible with
>>>>>>>>>>>>>> diff -r 7637e77c4c8a 
>>>>>>>>>>>>>> src/java.desktop/share/classes/javax/swing/text/Document.java 
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> --- 
>>>>>>>>>>>>>> a/src/java.desktop/share/classes/javax/swing/text/Document.java 
>>>>>>>>>>>>>> Mon Dec 02 11:58:14 2019 +0530
>>>>>>>>>>>>>> +++ 
>>>>>>>>>>>>>> b/src/java.desktop/share/classes/javax/swing/text/Document.java 
>>>>>>>>>>>>>> Mon Dec 02 16:02:08 2019 +0530
>>>>>>>>>>>>>> @@ -167,9 +167,6 @@
>>>>>>>>>>>>>>    *
>>>>>>>>>>>>>>    * <p>For more information on the <code>Document</code> 
>>>>>>>>>>>>>> class, see
>>>>>>>>>>>>>>    * <a 
>>>>>>>>>>>>>> href="http://www.oracle.com/technetwork/java/javase/tech/articles-jsp-139072.html">The 
>>>>>>>>>>>>>> Swing Connection</a>
>>>>>>>>>>>>>> *- * and most particularly the article,**
>>>>>>>>>>>>>> **- * <a 
>>>>>>>>>>>>>> href="http://java.sun.com/products/jfc/tsc/articles/text/element_interface">**
>>>>>>>>>>>>>> **- * The Element Interface</a>.*
>>>>>>>>>>>>>>    *
>>>>>>>>>>>>>>    * @author  Timothy Prinzing
>>>>>>>>>>>>>>
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> Regards
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> Prasanta
>>>>>>>>>>>>>>
>>>>>>>>>>>>>
>>>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.java.net/pipermail/swing-dev/attachments/20191213/483b63b5/attachment-0001.htm>


More information about the swing-dev mailing list