<Swing Dev> [9] Review request for 7020860: BasicTreeUI contains getters/setters with unclear spec

Sergey Bylokhov Sergey.Bylokhov at oracle.com
Tue Oct 25 18:46:53 UTC 2016


On 25.10.16 21:19, Phil Race wrote:
>
> Hi,
>
> OK with the change below and the rest of the explanation this is
> probably good to go ..

What is the reason to remove(instead of append it) old code which 
describe what the method do, for example:
-     * Sets the row height, this is forwarded to the treeState.
+     * Called when the {@code rowHeight} property is changed in
+     * the drawn tree component.

If we add a notions that some of the methods are called as a result of 
property changes, then probably other methods should be updated as well? 
For example setSelectionModel is called when "selectionModel" is changed.

Small typo after the "{@code editable}":
-     * @param newValue the new value
+     * @param newValue the new value of the {@code editable}> property

> On 07/18/2016 10:37 AM, Semyon Sadetsky wrote:
>>>
>> O.k. I would use "displayed" because it simply proxies
>> tree.isRootVisible() which has "displayed" in its specs.
>>
>>   Returns whether the root node of the drawn tree component *should be
>> displayed*.
>


-- 
Best regards, Sergey.



More information about the swing-dev mailing list