changeset 53982:de85ab85fbc7

8215165: Improve -Xlog:class+preview message text Summary: Fix up the logging message. Reviewed-by: acorn, lfoltan
author hseigel
date Tue, 11 Dec 2018 10:29:08 -0500
parents d75110673dc9
children 752e57845ad2
files src/hotspot/share/classfile/classFileParser.cpp test/hotspot/jtreg/runtime/ClassFile/PreviewVersion.java
diffstat 2 files changed, 3 insertions(+), 2 deletions(-) [+]
line wrap: on
line diff
--- a/src/hotspot/share/classfile/classFileParser.cpp	Tue Dec 11 15:09:15 2018 +0000
+++ b/src/hotspot/share/classfile/classFileParser.cpp	Tue Dec 11 10:29:08 2018 -0500
@@ -5733,7 +5733,8 @@
         ik->major_version() != JAVA_MIN_SUPPORTED_VERSION &&
         log_is_enabled(Info, class, preview)) {
       ResourceMark rm;
-      log_info(class, preview)("Loading preview feature type %s", ik->external_name());
+      log_info(class, preview)("Loading class %s that depends on preview features (class file version %d.65535)",
+                               ik->external_name(), ik->major_version());
     }
 
     if (log_is_enabled(Debug, class, resolve))  {
--- a/test/hotspot/jtreg/runtime/ClassFile/PreviewVersion.java	Tue Dec 11 15:09:15 2018 +0000
+++ b/test/hotspot/jtreg/runtime/ClassFile/PreviewVersion.java	Tue Dec 11 10:29:08 2018 -0500
@@ -69,7 +69,7 @@
         pb = ProcessTools.createJavaProcessBuilder("--enable-preview", "-Xlog:class+preview",
             "-cp", "." + File.pathSeparator + System.getProperty("test.classes"), "PVTest");
         oa = new OutputAnalyzer(pb.start());
-        oa.shouldContain("[info][class,preview] Loading preview feature type PVTest");
+        oa.shouldContain("[info][class,preview] Loading class PVTest that depends on preview features");
 
         // Subtract 1 from class's major version.  The class should fail to load
         // because its major_version does not match the JVM current version.