Serguei,
Thanks!
-Dmitry
On 2015-04-04 11:58, serguei.spit...@oracle.com wrote:
> Dmitry,
>
> I think, no re-review is necessary after fixing this typo.
> You had addressed the review comments.
>
> Thanks,
> Serguei
>
> On 4/4/15 12:06 AM, Dmitry Samersoff wrote:
>> Serguei,
>>
>> Yes, it's a typeo. Time to buy glasses.
>>
>> Thank you for catching it.
>>
>> Webrev updated in-place (press shift-reload).
>>
>> -Dmitry
>>
>>
>> On 2015-04-04 01:52, serguei.spit...@oracle.com wrote:
>>> Dmitry,
>>>
>>> I'm puzzled a little bit with the webrev.02:
>>>
>>> agent/src/share/classes/com/sun/java/swing/ui/CommonToolBar.java
>>>
>>> - buttonSize = new Dimension(CommonUI.buttconPrefSize);
>>> + buttonSize = new Dimension(CommonUI.getButtonPrefSize());
>>>
>>>
>>> Why do you replace the CommonUI.buttconPrefSize with the
>>> CommonUI.getButtonPrefSize() ?
>>> It looks like a typo, did you really want to do one of the following:
>>>
>>> - buttonSize = new Dimension(CommonUI.buttconPrefSize);
>>> + buttonSize = new Dimension(CommonUI.getButtconPrefSize());
>>>
>>>
>>> In such a case, the fix must be updated in the
>>> agent/src/share/classes/com/sun/java/swing/ui/CommonUI.java
>>>
>>> + public static Dimension getButtonPrefSize()
>>> + {
>>> + return buttonPrefSize;
>>> + }
>>>
>>> =>
>>>
>>> + public static Dimension getButtconPrefSize()
>>> + {
>>> + return buttconPrefSize;
>>> + }
>>>
>>>
>>>
>>> Thanks,
>>> Serguei
>>>
>>> On 4/3/15 2:27 PM, Dmitry Samersoff wrote:
>>>> Everybody,
>>>>
>>>> Please, review updated webrev:
>>>>
>>>> http://cr.openjdk.java.net/~dsamersoff/JDK-8067991/webrev.02/
>>>>
>>>> Fixed a compilation error that wasn't caught during incremental build.
>>>>
>>>> -Dmitry
>>>>
>>>>
>>>>
>>>> On 2015-04-02 15:54, Dmitry Samersoff wrote:
>>>>> Everybody,
>>>>>
>>>>> Please review a small fix:
>>>>>
>>>>> The fix goes to 9 and 8u at the same time.
>>>>>
>>>>> http://cr.openjdk.java.net/~dsamersoff/JDK-8067991/webrev.01/
>>>>> -Dmitry
>>>>>
>>
>
--
Dmitry Samersoff
Oracle Java development team, Saint Petersburg, Russia
* I would love to change the world, but they won't give me the sources.
