[prev in list] [next in list] [prev in thread] [next in thread]
List: openjdk-2d-dev
Subject: [OpenJDK 2D-Dev] <2D Dev> [7u6] Request for review: 7027300 Unsynchronized HashMap access causes end
From: philip.race () oracle ! com (Phil Race)
Date: 2012-06-19 16:24:13
Message-ID: 4FE0A7AD.40903 () oracle ! com
[Download RAW message or body]
Approved.
-phil.
On 6/19/2012 2:47 AM, Andrew Brygin wrote:
> Hi Vladislav,
>
> the fix looks fine to me.
> Please make sure you get a second reviewer.
>
> Thanks,
> Andrew
>
> On 19.06.2012 13:41, Vladislav Karnaukhov wrote:
>> Hi Andrew and all,
>>
>> please see updated webrev here:
>> http://cr.openjdk.java.net/~vkarnauk/7027300/webrev.01/
>>
>> Regards,
>> - Vlad
>>
>> On 19.06.12 11:35, Andrew Brygin wrote:
>>> Hello Vladislav,
>>>
>>> the fix itself seems to be pretty close to the fix for jdk8,
>>> but introduces different import order, and an additional comment.
>>>
>>> From my point of view, we should keep jdk7 and jdk8 sources as close
>>> as it is possible just to reduce the backporting overhead.
>>> In this particular case, I do not see a strong need in mentioned
>>> modifications
>>> so could you please consider to do a direct backport of the jdk8
>>> change?
>>>
>>> Thanks,
>>> Andrew
>>>
>>> On 18.06.2012 17:31, Vladislav Karnaukhov wrote:
>>>> Hello,
>>>>
>>>> please review a fix for CR 7027300: Unsynchronized HashMap access
>>>> causes endless loop.
>>>>
>>>> The issue is that SunLayoutEngine.getEngine uses non-synchronized
>>>> HashMap, which may cause an endless loop.
>>>> The fix is to use ConcurrentHashMap instead of HashMap.
>>>>
>>>> Bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7027300
>>>> Webrev: http://cr.openjdk.java.net/~vkarnauk/7027300/webrev.00/
>>>>
>>>> Regards,
>>>> - Vlad
>>>
>>
>
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic