<Swing Dev> [14] RFR JDK-7020860 - BasicTreeUI contains getters/setters with unclear spec

Tejpal Rebari tejpal.rebari at oracle.com
Wed Dec 4 09:32:56 UTC 2019


Hi Phil,

> On 27-Nov-2019, at 9:59 PM, Philip Race <philip.race at oracle.com> wrote:
> 
> Yes it needs a CSR.
> 
Will raise a CSR once the review is done.
> 
> The updated wording explains when the "set*" methods are used/called but
> does not do the same for the get* methods. Why not for both ?
The setters will be called when the property (like rowHeight, cellRenderer etc.) changes in the drawn tree component.
But the getters won’t be called when the property changes, so they need not to be updated.

Regards
Tejpal

> -phil.
> 
> On 11/24/19, 11:31 PM, Jayathirth Rao wrote:
>> 
>> Its looks okay.
>> Lets wait for Sergey’s inputs on the same.
>> 
>> And I think it needs CSR also.
>> 
>> Thanks,
>> Jay
>> 
>>> On 22-Nov-2019, at 3:04 PM, Tejpal Rebari <tejpal.rebari at oracle.com <mailto:tejpal.rebari at oracle.com>> wrote:
>>> 
>>> Hi Jay,
>>> I have appended the text instead of replacing it, 
>>> and also updated the SelectionModel methods.
>>> 
>>> Please find the updated webrev  : 
>>> http://cr.openjdk.java.net/~trebari/swing/7020860/webrev1/ <http://cr.openjdk.java.net/%7Etrebari/swing/7020860/webrev1/>
>>> 
>>> Thanks
>>> Tejpal
>>> 
>>>> On 18-Nov-2019, at 1:00 PM, Jayathirth D V <jayathirth.d.v at oracle.com <mailto:jayathirth.d.v at oracle.com>> wrote:
>>>> 
>>>> Hi Tejpal,
>>>>  
>>>> I think Sergey has given more inputs on the present change that you have at :
>>>> http://mail.openjdk.java.net/pipermail/swing-dev/2016-October/006815.html <http://mail.openjdk.java.net/pipermail/swing-dev/2016-October/006815.html>
>>>> and http://mail.openjdk.java.net/pipermail/swing-dev/2017-June/007495.html <http://mail.openjdk.java.net/pipermail/swing-dev/2017-June/007495.html>
>>>>  
>>>> Apart from this changes are fine.
>>>>  
>>>> Thanks,
>>>> Jay
>>>>  
>>>> From: Tejpal Rebari 
>>>> Sent: Thursday, October 31, 2019 1:55 PM
>>>> To: swing-dev at openjdk.java.net <mailto:swing-dev at openjdk.java.net>
>>>> Subject: <Swing Dev> [14] RFR JDK-7020860 - BasicTreeUI contains getters/setters with unclear spec
>>>>  
>>>> Hi All,
>>>> Please review the following fix for JDK 14.
>>>>  
>>>> Bug :      https://bugs.openjdk.java.net/browse/JDK-7020860 <https://bugs.openjdk.java.net/browse/JDK-7020860>
>>>> Webrev :  http://cr.openjdk.java.net/~trebari/swing/7020860/webrev0/ <http://cr.openjdk.java.net/%7Etrebari/swing/7020860/webrev0/>
>>>>  
>>>> Issue : Class javax.swing.plaf.basic.BasicTreeUI contains some getters/setters that are not consistent and 
>>>> and have specification that says nothing about how they should be used and how do they behave in BasicTreeUI.
>>>>  
>>>> Fix : This bug was already sent for review by semyon http://cr.openjdk.java.net/~ssadetsky/7020860/webrev.00/ <http://cr.openjdk.java.net/%7Essadetsky/7020860/webrev.00/>.
>>>> I have made the following changes to the above webrev : 
>>>>        * @param newValue the new value of the {@code editable}> property
>>>>        changed to —>>
>>>>                   * @param newValue the new value of the {@code editable} property
>>>>  
>>>> And 
>>>>                              * Returns whether the root node of the drawn tree component is displayable.
>>>>                              
>>>>                              changed to —>>
>>>>                              
>>>>                              * Returns whether the root node of the drawn tree component should be displayed.
>>>>  
>>>>  
>>>> Regards
>>>> Tejpal
>>> 
>> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.java.net/pipermail/swing-dev/attachments/20191204/7c59972a/attachment-0001.html>


More information about the swing-dev mailing list