<Swing Dev> [9] Review Request: JDK-8046434: Fix doclint warnings from javax.swing.plaf.metal package

[Alexander Scherbatiy]

   The fix looks good to me.

   Thanks,
   Alexandr.

On 6/30/2014 2:38 PM, andrei.eremeev wrote:
> Hi,
>
> Second iteration: http://cr.openjdk.java.net/~yan/8046434/webrev.01/
>
> Fixes:
>
>  1. the typo
>  2. reverted javadoc about "obsolete classes" and created new issue
>     <https://bugs.openjdk.java.net/browse/JDK-8048702>.
>
> Andrei
>
> On 06/26/2014 05:01 PM, Alexander Scherbatiy wrote:
>>
>>
>> new/src/share/classes/javax/swing/plaf/metal/MetalFileChooserUI.java
>> -    // Obsolete class, not used in this version.
>> +    /**
>> +     * Obsolete class, not used in this version.
>> +     */
>>
>> Adding such javadoc may require a CCC request.
>>
>> Could you use a simple description for obsolete classes and create a 
>> new issue to deprecate them?
>>
>> Thanks,
>> Alexandr.
>>
>> On 6/26/2014 4:41 PM, andrei.eremeev wrote:
>>> Hi Alexander,
>>>
>>> This fix of the type is small. Does it mean the webrev is approved?
>>> Can I push without making new webrev if I fix the type?
>>>
>>> Andrei
>>>
>>> On 06/26/2014 01:05 PM, Alexander Scherbatiy wrote:
>>>>
>>>>  There is a small typo:
>>>>  src/share/classes/javax/swing/plaf/metal/MetalTabbedPaneUI.java
>>>> +    /**
>>>> +     * The color ot tab's background.
>>>> +     */
>>>>      protected Color tabAreaBackground;
>>>>
>>>>
>>>> Thanks,
>>>> Alexandr.
>>>>
>>>> On 6/25/2014 6:13 PM, andrei.eremeev wrote:
>>>>> Hi Swing team,
>>>>>
>>>>> Please, review the fix for the issue:
>>>>> https://bugs.openjdk.java.net/browse/JDK-8046434
>>>>>
>>>>> The fix is available at:
>>>>> http://cr.openjdk.java.net/~yan/8046434/webrev.00/
>>>>>
>>>>> Andrei
>>>>
>>>
>>
>