<Swing Dev> [9] Review request for 8158734 JEditorPane.createEditorKitForContentType throws NPE after 6882559
Sergey Bylokhov
Sergey.Bylokhov at oracle.com
Wed Jun 8 14:43:53 UTC 2016
On 08.06.16 17:31, mikhail cherkasov wrote:
> And one more change: cr.openjdk.java.net/~mcherkas/8158734/9/webrev.02
> I added a removing of old class loader if a new is null.
Why it should be removed? I am not sure but is the null classloader is a
bootclassloader?
>
> On 6/8/2016 3:16 PM, Alexander Potochkin wrote:
>> Hello Mikhail
>>
>> This version looks good to me
>>
>> Thanks
>> alexp
>>
>>> Hi Sergey,
>>>
>>> you are right, it's easier to check classloader for null before to
>>> add it to hashtable:
>>> http://cr.openjdk.java.net/~mcherkas/8158734/9/webrev.01/src/java.desktop/share/classes/javax/swing/JEditorPane.java.udiff.html
>>>
>>>
>>> On 6/7/2016 10:51 PM, Sergey Bylokhov wrote:
>>>> HI, Mikhail.
>>>> Can you please clarify the reason of usage Optional here if the
>>>> optional itself can be null?
>>>>
>>>> On 07.06.16 22:05, mikhail cherkasov wrote:
>>>>> Hello,
>>>>>
>>>>> Could you please review the fix for:
>>>>> https://bugs.openjdk.java.net/browse/JDK-8158734
>>>>> webrev: http://cr.openjdk.java.net/~mcherkas/8158734/9/webrev.00/
>>>>>
>>>>> A simple null check was added.
>>>>>
>>>>> Thanks,
>>>>> Mikhail.
>>>>>
>>>>>
>>>>
>>>>
>>>
>>
>
--
Best regards, Sergey.
More information about the swing-dev
mailing list