RFR 8204620: ModuleEntry::is_non_jdk_module() determination for what is a jdk module is incorrect
Harold David Seigel
harold.seigel at oracle.com
Wed Jun 13 19:25:23 UTC 2018
Hi Lois, Mandy,
Thanks for the reviews.
Please review this updated webrev:
http://cr.openjdk.java.net/~hseigel/bug_8204620.2/webrev/index.html
It has the changes you suggested plus a few other minor ones.
Thanks! Harold
On 6/13/2018 11:09 AM, Lois Foltan wrote:
> On 6/13/2018 8:44 AM, Harold David Seigel wrote:
>
>> Hi,
>>
>> Please review this fix for how the JVM determines when to print a
>> module's version. The fix implements the refinement described in the
>> bug's comment on 12-Jun-2018.
>>
>> This method is currently only called from ClassCastExceptions but
>> will be used for other messages as part of JDK-8166633
>> <https://bugs.openjdk.java.net/browse/JDK-8166633>.
>>
>> Open Webrev:
>> http://javaweb.us.oracle.com/~hseigel/webrev/bug_8204620/webrev/index.html
>>
>> JBS Bug: https://bugs.openjdk.java.net/browse/JDK-8204620
>>
>> This fix was tested with Mach5 tiers 1 and 2 tests and builds on
>> Linux-X64, Windows, Solaris Sparc, and Mac OS X, with tiers 3-5 tests
>> on Linux-x64, and with JCK-11 Lang and VM tests.
>>
>> Thanks, Harold
>>
> Looks good, a couple of comments:
>
> - module.[c/h]pp - maybe consider the method name "show_version"
> instead of "show_module_version". Chances are this will be invoked by
> a ModuleEntry* variable whose name most likely will already have the
> word "module" in it.
> - module.cpp - line #81, since the version field within a ModuleEntry
> is a Symbol*, can you use fast_compare instead of strcmp?
> - test - needs copyright updated.
>
> Thanks for making this change, it will help with the upcoming error
> message work!
> Lois
More information about the hotspot-runtime-dev
mailing list