<Swing Dev> [9] Review request for 8169719 WrappedPlainView.modelToView() should return Rectangle2D

[Sergey Bylokhov]

Looks fine.

On 15.11.16 23:51, Phil Race wrote:
> +1
>
> -phil
>
> On 11/15/2016 12:32 PM, Alexandr Scherbatiy wrote:
>>
>> Could you review the updated fix:
>>   http://cr.openjdk.java.net/~alexsch/8169719/webrev.02
>>
>>   - The test is removed.
>>
>>   Floating point values mean that the test passes but only integral
>> values do not mean that the test fail.
>>
>>   Thanks,
>>   Alexandr.
>>
>> On 11/15/2016 9:52 PM, Phil Race wrote:
>>> The assumption that just because values *can* be non-integral that
>>> they *will* be non-integral is not valid. Even if you know that the
>>> probability is small, it is not zero.
>>>
>>> This seems like it could lead to spurious failures of this test.
>>>
>>> -phil.
>>>
>>>
>>> On 11/15/2016 09:18 AM, Alexandr Scherbatiy wrote:
>>>> On 11/15/2016 5:08 PM, Sergey Bylokhov wrote:
>>>>> Should "@SuppressWarnings("deprecation")" be removed from this method?
>>>>> Why the test is linux and windows specific?
>>>> Could you review the updated fix:
>>>>   http://cr.openjdk.java.net/~alexsch/8169719/webrev.01/
>>>>   - "@SuppressWarnings("deprecation")"is removed
>>>>
>>>>   The test is not suitable for Mac OS X because chars advances on
>>>> Mac OS X have integer values.
>>>>
>>>>   Thanks,
>>>>   Alexandr.
>>>>
>>>>>
>>>>> On 15.11.16 16:49, Alexandr Scherbatiy wrote:
>>>>>>
>>>>>> Hello,
>>>>>>
>>>>>> Could you review the fix:
>>>>>>   bug: https://bugs.openjdk.java.net/browse/JDK-8169719
>>>>>>   webrev: http://cr.openjdk.java.net/~alexsch/8169719/webrev.00
>>>>>>
>>>>>>   WrappedPlainView.modelToView() method is updated to use
>>>>>> Utilities.getTabbedTextWidth() which returns floating point text
>>>>>> width.
>>>>>>
>>>>>>  Thanks,
>>>>>>  Alexandr.
>>>>>>
>>>>>
>>>>>
>>>>
>>>
>>
>


-- 
Best regards, Sergey.