Tidying up inconsistencies.

Author: david-beaumont

src/hotspot/share/classfile/classLoader.cpp

@@ -253,9 +253,10 @@ Symbol* ClassLoader::package_from_class_name(const Symbol* name, bool* bad_class
 // The following jimage_xxx static functions encapsulate all JImage_file and JImage_mode access.
 // This is done to make it easy to reason about the JImage file state (exists vs initialized etc.).
 
-// Opens the named JImage file.
+// Opens the named JImage file and sets the JImage file reference.
+// Returns true if opening the JImage file was successful (see also jimage_exists()).
 static bool jimage_open(const char* modules_path) {
-  // Current 'error' is not set to anything useful so ignore it here.
+  // Currently 'error' is not set to anything useful, so ignore it here.
   jint error;
   JImage_file = (*JImageOpen)(modules_path, &error);
   return JImage_file != nullptr;

src/hotspot/share/runtime/arguments.cpp

@@ -3556,8 +3556,8 @@ jint Arguments::parse(const JavaVMInitArgs* initial_cmd_args) {
 
   // Parse the options in the /java.base/jdk/internal/vm/options resource, if present.
   // This opens the JImage file for lookup of the options, but a subsequent call to
-  // ClassLoader::enable_preview(bool) MUST be made before other resources can be
-  // read (see finalize_patch_module()).
+  // ClassLoader::init_jimage(bool) MUST be made before other resources can be read
+  // (see finalize_patch_module()).
   char *vmoptions = ClassLoader::lookup_vm_options();
   if (vmoptions != nullptr) {
     code = parse_options_buffer("vm options resource", vmoptions, strlen(vmoptions), &initial_vm_options_args);