<Swing Dev> [8] Review Request: JDK-8025234 [javadoc] fix some errors in javax.swing.**

Alexander Scherbatiy alexandr.scherbatiy at oracle.com
Tue Oct 15 10:24:12 UTC 2013


   The fix looks good for me.

   Thanks,
   Alexandr.

On 10/15/2013 2:04 PM, Dmitry Ginzburg wrote:
> forwarding to swing-dev
>
> 14.10.2013 18:27, Dmitry Ginzburg wrote:
>> See new webrev: 
>> http://cr.openjdk.java.net/~yan/jdk-8025234/webrev.01/ 
>> <http://cr.openjdk.java.net/%7Eyan/jdk-8025234/webrev.01/>
>>
>> Thanks,
>> -Dmitry
>>
>> 04.10.2013 21:22, alexey zhebel wrote:
>>> Yes, change <p/> to <p>. Doc comments are not really written in 
>>> HTML. They get converted to HTML by Javadoc. The <p> here is not a 
>>> paragraph opening tag, it is a marker for Javadoc to start a new 
>>> paragraph.
>>>
>>> Best regards,
>>> Alexey Zhebel
>>>
>>> On 04.10.2013 20:56, Alexander Scherbatiy wrote:
>>>>
>>>>   The mistaken tag <p/> should be corrected to <p>. This avoids 
>>>> fixing the typo next time.
>>>>
>>>>   Thanks,
>>>>   Alexandr.
>>>>
>>>> On 10/4/2013 5:40 PM, Dmitry Ginzburg wrote:
>>>>> Hi guys
>>>>>
>>>>> Have you decided what to do in this situation?
>>>>> Maybe my solution have to be approved?
>>>>>
>>>>> Thanks,
>>>>> -Dmitry
>>>>>
>>>>> 27.09.2013 20:11, alexey zhebel wrote:
>>>>>> Hi Alexander!
>>>>>>
>>>>>> AFAIK, the paragraph separator for Javadoc comments is <p>. So it 
>>>>>> is a typo (the > and / characters are close on the keyboard).
>>>>>>
>>>>>> Here is a good example:
>>>>>> http://www.oracle.com/technetwork/java/javase/documentation/index-137868.html#format 
>>>>>>
>>>>>>
>>>>>> Best regards,
>>>>>> Alexey Zhebel
>>>>>>
>>>>>> On 27.09.2013 16:43, Alexander Scherbatiy wrote:
>>>>>>>
>>>>>>>   Hello Alexey,
>>>>>>>
>>>>>>>   The is the <p/> tag at line: 1384 in the file 
>>>>>>> http://hg.openjdk.java.net/jdk8/awt/jdk/file/ca45169cb4eb/src/share/classes/javax/swing/AbstractButton.java 
>>>>>>>
>>>>>>>      1379     /**
>>>>>>>      1380      * Sets the <code>borderPainted</code> property.
>>>>>>>      1381      * If <code>true</code> and the button has a border,
>>>>>>>      1382      * the border is painted. The default value for the
>>>>>>>      1383      * <code>borderPainted</code> property is 
>>>>>>> <code>true</code>.
>>>>>>>      1384      * <p/>
>>>>>>>      1385      * Some look and feels might not support
>>>>>>>      1386      * the <code>borderPainted</code> property,
>>>>>>>      1387      * in which case they ignore this.
>>>>>>>
>>>>>>>  Could look at this and say is it just a typo and what should be 
>>>>>>> the corrected code?
>>>>>>>
>>>>>>>  Thanks,
>>>>>>>  Alexandr.
>>>>>>>
>>>>>>>
>>>>>>> On 9/27/2013 2:48 PM, Dmitry Ginzburg wrote:
>>>>>>>> According to w3c (http://dev.w3.org/html5/markup/p.html#p) 
>>>>>>>> <http://dev.w3.org/html5/markup/p.html#p> pelement’send tagmay 
>>>>>>>> be omitted if thepelement is immediately followed by an <set of 
>>>>>>>> some tags>, but that's false in our case, it's followed by text.
>>>>>>>>
>>>>>>>> 27.09.2013 14:38, Sergey Bylokhov wrote:
>>>>>>>>> Why not just <p>?
>>>>>>>>>
>>>>>>>>> On 27.09.2013 14:32, Dmitry Ginzburg wrote:
>>>>>>>>>> If it made sense earlier to do the same thing with 
>>>>>>>>>> self-closing tag, it's now the same, but valid for doclint.
>>>>>>>>>>
>>>>>>>>>> 27.09.2013 14:30, Alexander Scherbatiy wrote:
>>>>>>>>>>>
>>>>>>>>>>> --- old/src/share/classes/javax/swing/AbstractButton.java
>>>>>>>>>>> +++ new/src/share/classes/javax/swing/AbstractButton.java
>>>>>>>>>>> @@ -1381,7 +1381,7 @@
>>>>>>>>>>>       * If <code>true</code> and the button has a border,
>>>>>>>>>>>       * the border is painted. The default value for the
>>>>>>>>>>>       * <code>borderPainted</code> property is 
>>>>>>>>>>> <code>true</code>.
>>>>>>>>>>> -     * <p/>
>>>>>>>>>>> +     * <p></p>
>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>> Does it has sense to have open and closed p tags without the 
>>>>>>>>>>> text?
>>>>>>>>>>>
>>>>>>>>>>> Otherwise, the fix looks good for me.
>>>>>>>>>>>
>>>>>>>>>>> Thanks,
>>>>>>>>>>> Alexandr.
>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>> On 9/26/2013 6:20 PM, Dmitry Ginzburg wrote:
>>>>>>>>>>>> Hello, Swing Team.
>>>>>>>>>>>>
>>>>>>>>>>>> Please review the fix for the following issue:
>>>>>>>>>>>> https://bugs.openjdk.java.net/browse/JDK-8025234
>>>>>>>>>>>> The fix is available at:
>>>>>>>>>>>> http://cr.openjdk.java.net/~yan/jdk-8025234/webrev.00/
>>>>>>>>>>>>
>>>>>>>>>>>> This is the fix for javadoc errors, on which doclint was 
>>>>>>>>>>>> showing some issues.
>>>>>>>>>>>>
>>>>>>>>>>>> The patch contains only simple markup fixes; no 
>>>>>>>>>>>> changes/fixes in
>>>>>>>>>>>> documentation text; the specification itself wasn't changed.
>>>>>>>>>>>>
>>>>>>>>>>>> Thanks,
>>>>>>>>>>>> -Dmitry 
>>>>>>>>>
>>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>> -- 
>>>>>>>> Dmitry Ginzburg, FXSQE team member
>>>>>>>
>>>>>>
>>>>>
>>>>>
>>>>
>>>
>>
>>
>
>




More information about the swing-dev mailing list