<Swing Dev> <AWT Dev> JDK 9 RFR of JDK-8031082 Fix non-missing doclint problems in client libraries

[Joe Darcy]

Hello Sergey,

On 1/8/2014 11:53 PM, Sergey Bylokhov wrote:
> Hello, Joe.
> Why @see tag was removed in the ComponentPeer.java?

Because they were resulting in "error: reference not found" messages. 
The syntax appears correct, but the error is generated anyway. Since the 
peer package doesn't seem to be included in the main doc bundle, I 
didn't think removing the @see tags would be that much of a loss of 
functionality.

> In the JLabel.java @exception was removed is this change of 
> specifications?

The @exception tag was for an unchecked exception and was blank; 
removing it is not a specification change since it doesn't specify 
anything ;-)

> It would be good to split the changed line in the MouseInfoPeer.java.

Reformatted. Updated webrev:

      http://cr.openjdk.java.net/~darcy/8031082.1/

Thanks,

-Joe

>
> On 09.01.2014 5:08, Joseph Darcy wrote:
>> Happy new year!
> Happy new year!
>>
>> In case this was overlooked during the new year's holiday, please 
>> look over the changes in
>>
>> http://cr.openjdk.java.net/~darcy/8031082.0/
>>
>> which resolve many of the outstanding doclint issues in the client 
>> libraries.
>>
>> Thanks,
>>
>> -Joe
>>
>> On 12/31/2013 12:51 AM, Joe Darcy wrote:
>>> Hello,
>>>
>>> Please review my fix for
>>>
>>>     JDK-8031082 Fix non-missing doclint problems in client libraries
>>>
>>> The fix resolves nearly all of the outstanding doclint issues in the 
>>> client libraries, others than the missing javadoc tags. The affected 
>>> files are
>>>
>>> src/share/classes/java/awt/Graphics2D.java
>>> src/share/classes/java/awt/MediaTracker.java
>>> src/share/classes/java/awt/font/TextAttribute.java
>>> src/share/classes/java/awt/peer/ComponentPeer.java
>>> src/share/classes/java/awt/peer/DialogPeer.java
>>> src/share/classes/java/awt/peer/KeyboardFocusManagerPeer.java
>>> src/share/classes/java/awt/peer/MouseInfoPeer.java
>>> src/share/classes/java/awt/peer/PanelPeer.java
>>> src/share/classes/java/awt/peer/TextAreaPeer.java
>>> src/share/classes/java/awt/peer/WindowPeer.java
>>> src/share/classes/java/awt/print/Paper.java
>>> src/share/classes/java/awt/print/Printable.java
>>> src/share/classes/java/beans/XMLEncoder.java
>>> src/share/classes/javax/accessibility/AccessibleContext.java
>>> src/share/classes/javax/imageio/ImageWriter.java
>>> src/share/classes/javax/imageio/plugins/bmp/BMPImageWriteParam.java
>>> src/share/classes/javax/imageio/stream/ImageInputStream.java
>>> src/share/classes/javax/imageio/stream/ImageOutputStream.java
>>> src/share/classes/javax/print/Doc.java
>>> src/share/classes/javax/print/DocFlavor.java
>>> src/share/classes/javax/print/MultiDoc.java
>>> src/share/classes/javax/print/MultiDocPrintJob.java
>>> src/share/classes/javax/print/ServiceUI.java
>>> src/share/classes/javax/print/StreamPrintServiceFactory.java
>>> src/share/classes/javax/print/attribute/AttributeSet.java
>>> src/share/classes/javax/print/attribute/standard/Chromaticity.java
>>> src/share/classes/javax/print/attribute/standard/Copies.java
>>> src/share/classes/javax/print/attribute/standard/Fidelity.java
>>> src/share/classes/javax/print/attribute/standard/Finishings.java
>>> src/share/classes/javax/print/attribute/standard/JobKOctets.java
>>> src/share/classes/javax/print/attribute/standard/JobState.java
>>> src/share/classes/javax/print/attribute/standard/MediaName.java
>>> src/share/classes/javax/print/attribute/standard/MediaSize.java
>>> src/share/classes/javax/print/attribute/standard/MediaSizeName.java
>>> src/share/classes/javax/print/attribute/standard/MediaTray.java
>>> src/share/classes/javax/print/attribute/standard/MultipleDocumentHandling.java 
>>>
>>> src/share/classes/javax/print/attribute/standard/NumberUp.java
>>> src/share/classes/javax/print/attribute/standard/PageRanges.java
>>> src/share/classes/javax/print/attribute/standard/PrinterMoreInfoManufacturer.java 
>>>
>>> src/share/classes/javax/print/attribute/standard/PrinterResolution.java
>>> src/share/classes/javax/print/attribute/standard/SheetCollate.java
>>> src/share/classes/javax/print/attribute/standard/Sides.java
>>> src/share/classes/javax/sound/sampled/AudioInputStream.java
>>> src/share/classes/javax/sound/sampled/AudioPermission.java
>>> src/share/classes/javax/sound/sampled/ReverbType.java
>>> src/share/classes/javax/swing/DefaultComboBoxModel.java
>>> src/share/classes/javax/swing/JComboBox.java
>>> src/share/classes/javax/swing/JEditorPane.java
>>> src/share/classes/javax/swing/JLabel.java
>>> src/share/classes/javax/swing/JLayeredPane.java
>>> src/share/classes/javax/swing/JOptionPane.java
>>> src/share/classes/javax/swing/JTextArea.java
>>> src/share/classes/javax/swing/JTextPane.java
>>> src/share/classes/javax/swing/plaf/TextUI.java
>>> src/share/classes/javax/swing/plaf/basic/BasicTextUI.java
>>> src/share/classes/javax/swing/plaf/nimbus/AbstractRegionPainter.java
>>> src/share/classes/javax/swing/plaf/nimbus/NimbusStyle.java
>>> src/share/classes/javax/swing/text/CompositeView.java
>>> src/share/classes/javax/swing/text/DefaultEditorKit.java
>>> src/share/classes/javax/swing/text/Document.java
>>> src/share/classes/javax/swing/text/GlyphView.java
>>> src/share/classes/javax/swing/text/JTextComponent.java
>>> src/share/classes/javax/swing/text/NavigationFilter.java
>>> src/share/classes/javax/swing/text/html/HTMLDocument.java
>>> src/share/classes/javax/swing/text/html/StyleSheet.java
>>>
>>> and the full webrev is hosted at
>>>
>>> http://cr.openjdk.java.net/~darcy/8031082.0/
>>>
>>> Thanks,
>>>
>>> -Joe
>>
>
>