RFR: 8194143: remove unneeded casts in LocationImpl and MirrorImpl classes
David Holmes
david.holmes at oracle.com
Thu Jan 18 09:08:11 UTC 2018
On 18/01/2018 6:56 PM, Egor Ushakov wrote:
> Hi Serguei,
>
> I do not see jdk11 repo on http://hg.openjdk.java.net,
> do I need to use http://hg.openjdk.java.net/jdk/hs?
Yes. jdk/hs always contains the latest version under development (as
does jdk/jdk).
Cheers,
David
> Thanks,
> Egor
>
> On 17-Jan-18 20:37, serguei.spitsyn at oracle.com wrote:
>> On 1/17/18 09:32, Egor Ushakov wrote:
>>> Is the link http://cr.openjdk.java.net/~avu/egor.ushakov/cast_fix
>>> broken? for me it works well.
>>> attaching the patch just in case.
>>
>> It was my mistake this time, sorry. :(
>> Please, see my previous message.
>>
>>
>>> It was created on jdk10, do I need to recreate it based on jdk11 with
>>> the 2018 copyrights?
>>
>> Yes, and it needs to be committed with the reviewers listed.
>>
>>
>> Thanks,
>> Serguei
>>
>>>
>>> On 17-Jan-18 20:26, serguei.spitsyn at oracle.com wrote:
>>>> Hi Egor and David,
>>>>
>>>> Both webrevs below are from Daniil Titov on the JVMTI bug: JDK-8153629.
>>>> Also, I expect new webrev is based on the JDK 11 jdk/hs repository
>>>> and with 2018 copyright comments updated.
>>>>
>>>> Thanks,
>>>> Serguei
>>>>
>>>>
>>>> On 1/17/18 02:24, David Holmes wrote:
>>>>> Hi Egor,
>>>>>
>>>>> On 17/01/2018 7:15 PM, Egor Ushakov wrote:
>>>>>> Hello,
>>>>>>
>>>>>> Please review and push the fix.
>>>>>> This is a formal letter after the discussion on the alias.
>>>>>>
>>>>>> Bug:https://bugs.openjdk.java.net/browse/JDK-8194143
>>>>>> Webrev:http://cr.openjdk.java.net/~avu/egor.ushakov/cast_fix/
>>>>>
>>>>> The underlying link actually pointed to:
>>>>>
>>>>> http://cr.openjdk.java.net/%7Edtitov/8153629/webrev.00
>>>>>
>>>>> :)
>>>>>
>>>>> Real webrev looks fine.
>>>>>
>>>>> Thanks,
>>>>> David
>>>>>
>>>>>> --
>>>>>> Egor Ushakov
>>>>>> Software Developer
>>>>>> JetBrains
>>>>>> http://www.jetbrains.com
>>>>>> The Drive to Develop
>>>>>>
>>>>
>>>
>>
>
More information about the serviceability-dev
mailing list