diff --git a/doc/hotspot-unit-tests.html b/doc/hotspot-unit-tests.html
index 556cfecc42d..fcd4a93f8e4 100644
--- a/doc/hotspot-unit-tests.html
+++ b/doc/hotspot-unit-tests.html
@@ -245,7 +245,7 @@ compare C string contents. There are also case-insensitive versions
 <p>All GoogleTest asserts print compared expressions and their values,
 so there is no need to have them in error messages. Asserts print only
 compared values, they do not print any of interim variables, e.g.
-<code>ASSERT_TRUE((val1 == val2 &amp;&amp; isFail(foo(8)) || i == 18)</code>
+<code>ASSERT_TRUE((val1 == val2 &amp;&amp; isFail(foo(8))) || i == 18)</code>
 prints only one value. If you use some complex predicates, please
 consider <code>EXPECT_PRED*</code> or <code>EXPECT_FORMAT_PRED</code>
 assertions family, they check that a predicate returns true/success and
diff --git a/doc/hotspot-unit-tests.md b/doc/hotspot-unit-tests.md
index 62ace4ef6ee..e1222baa2e3 100644
--- a/doc/hotspot-unit-tests.md
+++ b/doc/hotspot-unit-tests.md
@@ -172,7 +172,7 @@ Provide informative, but not too verbose error messages.
 All GoogleTest asserts print compared expressions and their values, so
 there is no need to have them in error messages. Asserts print only
 compared values, they do not print any of interim variables, e.g.
-`ASSERT_TRUE((val1 == val2 && isFail(foo(8)) || i == 18)` prints only
+`ASSERT_TRUE((val1 == val2 && isFail(foo(8))) || i == 18)` prints only
 one value. If you use some complex predicates, please consider
 `EXPECT_PRED*` or `EXPECT_FORMAT_PRED` assertions family, they check that
 a predicate returns true/success and print out all parameters values.
diff --git a/src/hotspot/share/jfr/metadata/metadata.xml b/src/hotspot/share/jfr/metadata/metadata.xml
index 3b5e24ca9a2..1b85cb1350c 100644
--- a/src/hotspot/share/jfr/metadata/metadata.xml
+++ b/src/hotspot/share/jfr/metadata/metadata.xml
@@ -46,10 +46,10 @@
        $ jfr print dump.jfr
 
        Programmatic access:
-       try (var rf = new RecordingFile(Path.of("dump.jfr)) {
+       try (var rf = new RecordingFile(Path.of("dump.jfr"))) {
            while (rf.hasMoreEvents()) {
                RecordedEvent e = rf.readEvent();
-               System.out.println(e.getName() + " " + e.getDuration()));
+               System.out.println(e.getName() + " " + e.getDuration());
            }
        };
   !-->
diff --git a/src/hotspot/share/prims/jvmti.xml b/src/hotspot/share/prims/jvmti.xml
index 0afbf56b3fc..e8e127486ca 100644
--- a/src/hotspot/share/prims/jvmti.xml
+++ b/src/hotspot/share/prims/jvmti.xml
@@ -1218,7 +1218,7 @@ jvmtiEnv *jvmti;
                             <li><i>Timed wait?</i>
                               <ul>
                                 <li>Indefinite (<datalink
-                                    id="JVMTI_THREAD_STATE_WAITING_INDEFINITELY"><code>JVMTI_THREAD_STATE_WAITING_INDEFINITELY</code></datalink></li>
+                                    id="JVMTI_THREAD_STATE_WAITING_INDEFINITELY"><code>JVMTI_THREAD_STATE_WAITING_INDEFINITELY</code></datalink>)</li>
                                 <li>Timed (<datalink
                                     id="JVMTI_THREAD_STATE_WAITING_WITH_TIMEOUT"><code>JVMTI_THREAD_STATE_WAITING_WITH_TIMEOUT</code></datalink>)</li>
                               </ul>
diff --git a/src/java.base/share/man/java.md b/src/java.base/share/man/java.md
index 01f7b3f7e77..a048d2a6dcf 100644
--- a/src/java.base/share/man/java.md
+++ b/src/java.base/share/man/java.md
@@ -1311,7 +1311,7 @@ These `java` options control the runtime behavior of the Java HotSpot VM.
 `-XX:+PreserveFramePointer`
 :   Selects between using the RBP register as a general purpose register
     (`-XX:-PreserveFramePointer`) and using the RBP register to hold the frame
-    pointer of the currently executing method (`-XX:+PreserveFramePointer` . If
+    pointer of the currently executing method (`-XX:+PreserveFramePointer`). If
     the frame pointer is available, then external profiling tools (for example,
     Linux perf) can construct more accurate stack traces.
 
diff --git a/src/java.base/share/man/keytool.md b/src/java.base/share/man/keytool.md
index 22b2877a3e7..8484ec6b5b3 100644
--- a/src/java.base/share/man/keytool.md
+++ b/src/java.base/share/man/keytool.md
@@ -662,7 +662,7 @@ perform.
 
     -   {`-noprompt`}: Do not prompt
 
-    -   {`-addprovider` *name* \[`-providerarg` *arg*\]: Add security provider
+    -   {`-addprovider` *name* \[`-providerarg` *arg*\]}: Add security provider
         by name (such as SunPKCS11) with an optional configure argument.
 
     -   {`-providerclass` *class* \[`-providerarg` *arg*\]}: Add security
@@ -1617,7 +1617,7 @@ name information, the keystore password, and the private key password.
 The rest of the examples assume that you responded to the prompts with values
 equal to those specified in the first `-genkeypair` command. For example, a
 distinguished name of
-`cn=`*myname*`, ou=`*mygroup*`, o=`*mycompany*`, c=`*mycountry*).
+`cn=`*myname*`, ou=`*mygroup*`, o=`*mycompany*`, c=`*mycountry*.
 
 ## Requesting a Signed Certificate from a CA
 
@@ -1914,7 +1914,7 @@ Keystore implementation
     The `keytool` command works on any file-based keystore implementation. It
     treats the keystore location that is passed to it at the command line as a
     file name and converts it to a `FileInputStream`, from which it loads the
-    keystore information.)The `jarsigner` commands can read a keystore from any
+    keystore information. The `jarsigner` commands can read a keystore from any
     location that can be specified with a URL.
 
     For `keytool` and `jarsigner`, you can specify a keystore type at the
diff --git a/src/jdk.compiler/share/man/javac.md b/src/jdk.compiler/share/man/javac.md
index 7761cde2b58..3edbeda6eca 100644
--- a/src/jdk.compiler/share/man/javac.md
+++ b/src/jdk.compiler/share/man/javac.md
@@ -1667,7 +1667,7 @@ internal and subject to change at any time.
     public static int m() {
       try {
          throw new NullPointerException();
-      }  catch (NullPointerException(); {
+      }  catch (NullPointerException e) {
          System.err.println("Caught NullPointerException.");
          return 1;
        } finally {