changeset 50992:55fac6146d31

8203919: Remove "-add-modules" hint from error message Reviewed-by: mgronlun
author egahlin
date Mon, 04 Jun 2018 22:03:10 +0200
parents 3b52a76c7902
children 7e8c0409a747
files src/hotspot/share/jfr/jni/jfrJavaSupport.cpp test/jdk/jdk/jfr/jvm/TestJfrJavaBase.java
diffstat 2 files changed, 2 insertions(+), 5 deletions(-) [+]
line wrap: on
line diff
--- a/src/hotspot/share/jfr/jni/jfrJavaSupport.cpp	Mon Jun 04 13:53:51 2018 -0500
+++ b/src/hotspot/share/jfr/jni/jfrJavaSupport.cpp	Mon Jun 04 22:03:10 2018 +0200
@@ -584,10 +584,8 @@
 
 static void print_module_resolution_error(outputStream* stream) {
   assert(stream != NULL, "invariant");
-  stream->print_cr("%s not found.", JDK_JFR_MODULE_NAME);
+  stream->print_cr("Module %s not found.", JDK_JFR_MODULE_NAME);
   stream->print_cr("Flight Recorder can not be enabled.");
-  stream->print_cr("To use Flight Recorder, you might need to add" \
-    " \"--add-modules %s\" to the VM command-line options.", JDK_JFR_MODULE_NAME);
 }
 
 bool JfrJavaSupport::is_jdk_jfr_module_available() {
--- a/test/jdk/jdk/jfr/jvm/TestJfrJavaBase.java	Mon Jun 04 13:53:51 2018 -0500
+++ b/test/jdk/jdk/jfr/jvm/TestJfrJavaBase.java	Mon Jun 04 22:03:10 2018 +0200
@@ -43,9 +43,8 @@
 public class TestJfrJavaBase {
 
     private static void checkOutput(OutputAnalyzer output) {
-        output.shouldContain("jdk.jfr not found.");
+        output.shouldContain("Module jdk.jfr not found.");
         output.shouldContain("Flight Recorder can not be enabled.");
-        output.shouldContain("To use Flight Recorder, you might need to add \"--add-modules jdk.jfr\" to the VM command-line options.");
     }
 
     public static void main(String[] args) throws Exception {