RFR(M) JDK-8031819: Remove legacy jdk checks and code
Coleen Phillimore
coleen.phillimore at oracle.com
Wed Jun 4 18:10:01 UTC 2014
Harold, I reviewed this and it looks good. One comment.
On 6/4/14, 12:59 PM, Zhengyu Gu wrote:
> Thanks for cleaning up.
>
> javaClasses.cpp:
>
> 1545 // This is unnecessary in 1.7+ but harmless
> 1546 clear_stacktrace(throwable());
>
> I think they can go also.
Can you leave this in because it seems like a good idea and hygenic and
something might break if you take it out?
Coleen
>
> -Zhengyu
>
>
>
> On 6/4/2014 11:34 AM, harold seigel wrote:
>> Hi,
>>
>> Please review this JDK 9 fix for bug JDK-8031819. The fix removes
>> legacy code for old JDK versions. Although the webrev contains lots
>> of files, the changes are not complicated.
>>
>> Open webrev: http://cr.openjdk.java.net/~hseigel/bug_8031819/
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8031819
>>
>> The fix was tested with the Hotspot JTREG tests, JCK Lang, VM, and
>> API/java_lang tests, nsk Quick tests, and JPRT.
>>
>> Thanks! Harold
>
More information about the hotspot-runtime-dev
mailing list