Duplicate words typos in comments/javadoc/strings

Andrey Turbanov turbanoff at gmail.com
Mon Jan 28 22:23:38 UTC 2019


> On 28/01/2019 17:04, Lance Andersen wrote:
> > The updates look fine
> >> -     *   This includes the case where a module reads another another with the
> >> +     *   This includes the case where a module reads another with the
> >>
> This one is typo, it should say "another module".
>
> -Alan

Thanks to review. Patch updated. I'm not really sure what to do next.
I should find someone who can create issue and commit my patch to
repository, right?

diff -r fcfeed9fef45 src/java.base/share/classes/java/lang/Object.java
--- a/src/java.base/share/classes/java/lang/Object.java    Mon Jan 28
21:44:57 2019 +0100
+++ b/src/java.base/share/classes/java/lang/Object.java    Tue Jan 29
01:21:13 2019 +0300
@@ -430,7 +430,7 @@
      * }</pre>
      *
      * @param  timeoutMillis the maximum time to wait, in milliseconds
-     * @param  nanos   additional time, in nanoseconds, in the range
range 0-999999 inclusive
+     * @param  nanos   additional time, in nanoseconds, in the range
0-999999 inclusive
      * @throws IllegalArgumentException if {@code timeoutMillis} is negative,
      *         or if the value of {@code nanos} is out of range
      * @throws IllegalMonitorStateException if the current thread is not
diff -r fcfeed9fef45 src/java.base/share/classes/java/lang/String.java
--- a/src/java.base/share/classes/java/lang/String.java    Mon Jan 28
21:44:57 2019 +0100
+++ b/src/java.base/share/classes/java/lang/String.java    Tue Jan 29
01:21:13 2019 +0300
@@ -2698,7 +2698,7 @@
      * <p>
      * Otherwise, returns a substring of this string beginning with the first
      * code point that is not a {@link Character#isWhitespace(int) white space}
-     * up to to and including the last code point of this string.
+     * up to and including the last code point of this string.
      * <p>
      * This method may be used to trim
      * {@link Character#isWhitespace(int) white space} from
diff -r fcfeed9fef45
src/java.base/share/classes/java/lang/invoke/MethodHandle.java
--- a/src/java.base/share/classes/java/lang/invoke/MethodHandle.java
 Mon Jan 28 21:44:57 2019 +0100
+++ b/src/java.base/share/classes/java/lang/invoke/MethodHandle.java
 Tue Jan 29 01:21:13 2019 +0300
@@ -1083,7 +1083,7 @@
       * else if (makeVarargs)
       *   return asVarargsCollector(type().lastParameterType());
       * else
-      *   return return asFixedArity();
+      *   return asFixedArity();
       * }</pre></blockquote>
       * @param makeVarargs true if the return method handle should
have variable arity behavior
       * @return a method handle of the same type, with possibly
adjusted variable arity behavior
diff -r fcfeed9fef45 src/java.base/share/classes/java/lang/invoke/VarHandle.java
--- a/src/java.base/share/classes/java/lang/invoke/VarHandle.java
Mon Jan 28 21:44:57 2019 +0100
+++ b/src/java.base/share/classes/java/lang/invoke/VarHandle.java
Tue Jan 29 01:21:13 2019 +0300
@@ -2160,7 +2160,7 @@
          * Construct a {@linkplain VarHandleDesc} given a kind, name,
and declaring
          * class.
          *
-         * @param kind the kind of of the var handle
+         * @param kind the kind of the var handle
          * @param name the unqualified name of the field, for field
var handles; otherwise ignored
          * @param declaringClass a {@link ClassDesc} describing the
declaring class,
          *                       for field var handles
@@ -2217,7 +2217,7 @@

         /**
          * Returns a {@linkplain VarHandleDesc} corresponding to a
{@link VarHandle}
-         * for for an array type.
+         * for an array type.
          *
          * @param arrayClass a {@link ClassDesc} describing the type
of the array
          * @return the {@linkplain VarHandleDesc}
diff -r fcfeed9fef45 src/java.base/share/classes/java/lang/module/Resolver.java
--- a/src/java.base/share/classes/java/lang/module/Resolver.java
Mon Jan 28 21:44:57 2019 +0100
+++ b/src/java.base/share/classes/java/lang/module/Resolver.java
Tue Jan 29 01:21:13 2019 +0300
@@ -659,7 +659,7 @@
      * Checks the readability graph to ensure that
      * <ol>
      *   <li><p> A module does not read two or more modules with the same name.
-     *   This includes the case where a module reads another another with the
+     *   This includes the case where a module reads another module with the
      *   same name as itself. </p></li>
      *   <li><p> Two or more modules in the configuration don't export the same
      *   package to a module that reads both. This includes the case where a
diff -r fcfeed9fef45 src/java.base/share/classes/java/util/SimpleTimeZone.java
--- a/src/java.base/share/classes/java/util/SimpleTimeZone.java    Mon
Jan 28 21:44:57 2019 +0100
+++ b/src/java.base/share/classes/java/util/SimpleTimeZone.java    Tue
Jan 29 01:21:13 2019 +0300
@@ -310,7 +310,7 @@
      *                        See the class description for the
special cases of this parameter.
      * @param endDayOfWeek    The daylight saving time ending day-of-week.
      *                        See the class description for the
special cases of this parameter.
-     * @param endTime         The daylight saving ending time in time time mode
+     * @param endTime         The daylight saving ending time in time mode
      *                        specified by <code>endTimeMode</code>.
      * @param endTimeMode     The mode of the end time specified by endTime
      * @param dstSavings      The amount of time in milliseconds saved during
diff -r fcfeed9fef45 src/java.base/share/classes/java/util/stream/WhileOps.java
--- a/src/java.base/share/classes/java/util/stream/WhileOps.java
Mon Jan 28 21:44:57 2019 +0100
+++ b/src/java.base/share/classes/java/util/stream/WhileOps.java
Tue Jan 29 01:21:13 2019 +0300
@@ -658,7 +658,7 @@
         // True if no splitting should be performed, if true then
         // this spliterator may be used for an underlying spliterator whose
         // covered elements have an encounter order
-        // See use in stream take/dropWhile default default methods
+        // See use in stream take/dropWhile default methods
         final boolean noSplitting;
         // True when operations are cancelled for all related spliterators
         // For taking, spliterators cannot split or traversed
diff -r fcfeed9fef45
src/java.base/share/classes/javax/security/auth/callback/ConfirmationCallback.java
--- a/src/java.base/share/classes/javax/security/auth/callback/ConfirmationCallback.java
   Mon Jan 28 21:44:57 2019 +0100
+++ b/src/java.base/share/classes/javax/security/auth/callback/ConfirmationCallback.java
   Tue Jan 29 01:21:13 2019 +0300
@@ -58,7 +58,7 @@
     public static final int YES_NO_OPTION               = 0;

     /**
-     * YES/NO/CANCEL confirmation confirmation option.
+     * YES/NO/CANCEL confirmation option.
      *
      * <p> An underlying security service specifies this as the
      * {@code optionType} to a {@code ConfirmationCallback}
@@ -68,7 +68,7 @@
     public static final int YES_NO_CANCEL_OPTION        = 1;

     /**
-     * OK/CANCEL confirmation confirmation option.
+     * OK/CANCEL confirmation option.
      *
      * <p> An underlying security service specifies this as the
      * {@code optionType} to a {@code ConfirmationCallback}
diff -r fcfeed9fef45
src/java.base/share/classes/jdk/internal/access/SharedSecrets.java
--- a/src/java.base/share/classes/jdk/internal/access/SharedSecrets.java
   Mon Jan 28 21:44:57 2019 +0100
+++ b/src/java.base/share/classes/jdk/internal/access/SharedSecrets.java
   Tue Jan 29 01:21:13 2019 +0300
@@ -77,7 +77,7 @@

     public static JavaUtilJarAccess javaUtilJarAccess() {
         if (javaUtilJarAccess == null) {
-            // Ensure JarFile is initialized; we know that that class
+            // Ensure JarFile is initialized; we know that this class
             // provides the shared secret
             unsafe.ensureClassInitialized(JarFile.class);
         }
diff -r fcfeed9fef45
src/java.base/share/classes/jdk/internal/platform/Metrics.java
--- a/src/java.base/share/classes/jdk/internal/platform/Metrics.java
 Mon Jan 28 21:44:57 2019 +0100
+++ b/src/java.base/share/classes/jdk/internal/platform/Metrics.java
 Tue Jan 29 01:21:13 2019 +0300
@@ -322,7 +322,7 @@
      * Returns the largest amount of physical memory, in bytes, that
      * have been allocated in the Isolation Group.
      *
-     * @return The largest amount of memory in bytes or or 0 if this
+     * @return The largest amount of memory in bytes or 0 if this
      *         metric is not available.
      *
      */
@@ -362,7 +362,7 @@
      * Returns the largest amount of kernel physical memory, in bytes, that
      * have been allocated in the Isolation Group.
      *
-     * @return The largest amount of memory in bytes or or 0 if this
+     * @return The largest amount of memory in bytes or 0 if this
      *         metric is not available.
      *
      */
@@ -402,7 +402,7 @@
      * Returns the largest amount of networking physical memory, in bytes,
      * that have been allocated in the Isolation Group.
      *
-     * @return The largest amount of memory in bytes or or 0 if this
+     * @return The largest amount of memory in bytes or 0 if this
      *         metric is not available.
      *
      */
@@ -442,7 +442,7 @@
      * Returns the largest amount of physical memory and swap space,
      * in bytes, that have been allocated in the Isolation Group.
      *
-     * @return The largest amount of memory in bytes or or 0 if this
+     * @return The largest amount of memory in bytes or 0 if this
      *         metric is not available.
      *
      */
diff -r fcfeed9fef45
src/java.base/share/classes/sun/security/ssl/HandshakeHash.java
--- a/src/java.base/share/classes/sun/security/ssl/HandshakeHash.java
  Mon Jan 28 21:44:57 2019 +0100
+++ b/src/java.base/share/classes/sun/security/ssl/HandshakeHash.java
  Tue Jan 29 01:21:13 2019 +0300
@@ -246,7 +246,7 @@
             try {
                 baos.writeTo(result.baos);
             } catch (IOException ex) {
-                throw new RuntimeException("unable to to clone hash state");
+                throw new RuntimeException("unable to clone hash state");
             }
             return result;
         }
diff -r fcfeed9fef45
src/java.base/share/classes/sun/security/ssl/RenegoInfoExtension.java
--- a/src/java.base/share/classes/sun/security/ssl/RenegoInfoExtension.java
   Mon Jan 28 21:44:57 2019 +0100
+++ b/src/java.base/share/classes/sun/security/ssl/RenegoInfoExtension.java
   Tue Jan 29 01:21:13 2019 +0300
@@ -145,7 +145,7 @@
             if (!chc.conContext.isNegotiated) {
                 if (chc.activeCipherSuites.contains(
                         CipherSuite.TLS_EMPTY_RENEGOTIATION_INFO_SCSV)) {
-                    // Using the the TLS_EMPTY_RENEGOTIATION_INFO_SCSV instead.
+                    // Using the TLS_EMPTY_RENEGOTIATION_INFO_SCSV instead.
                     return null;
                 }

diff -r fcfeed9fef45
src/java.base/share/classes/sun/security/ssl/SSLTransport.java
--- a/src/java.base/share/classes/sun/security/ssl/SSLTransport.java
 Mon Jan 28 21:44:57 2019 +0100
+++ b/src/java.base/share/classes/sun/security/ssl/SSLTransport.java
 Tue Jan 29 01:21:13 2019 +0300
@@ -71,7 +71,7 @@
      * destination application data buffers.
      *
      * For SSL/TLS connections, if no source data, the network data may be
-     * received from the underlying underlying SSL/TLS input stream.
+     * received from the underlying SSL/TLS input stream.
      *
      * @param context      the transportation context
      * @param srcs         an array of {@code ByteBuffers} containing the
diff -r fcfeed9fef45
src/java.base/share/classes/sun/security/ssl/TrustStoreManager.java
--- a/src/java.base/share/classes/sun/security/ssl/TrustStoreManager.java
   Mon Jan 28 21:44:57 2019 +0100
+++ b/src/java.base/share/classes/sun/security/ssl/TrustStoreManager.java
   Tue Jan 29 01:21:13 2019 +0300
@@ -331,7 +331,7 @@
         }

         /**
-         * Load the the KeyStore as described in the specified descriptor.
+         * Load the KeyStore as described in the specified descriptor.
          */
         private static KeyStore loadKeyStore(
                 TrustStoreDescriptor descriptor) throws Exception {
diff -r fcfeed9fef45
src/java.base/share/classes/sun/security/util/ConstraintsParameters.java
--- a/src/java.base/share/classes/sun/security/util/ConstraintsParameters.java
   Mon Jan 28 21:44:57 2019 +0100
+++ b/src/java.base/share/classes/sun/security/util/ConstraintsParameters.java
   Tue Jan 29 01:21:13 2019 +0300
@@ -37,7 +37,7 @@
  * This class contains parameters for checking against constraints that extend
  * past the publicly available parameters in
java.security.AlgorithmConstraints.

- * This is currently on passed between  between PKIX, AlgorithmChecker,
+ * This is currently on passed between PKIX, AlgorithmChecker,
  * and DisabledAlgorithmConstraints.
  */
 public class ConstraintsParameters {
diff -r fcfeed9fef45
src/java.base/share/classes/sun/security/util/math/SmallValue.java
--- a/src/java.base/share/classes/sun/security/util/math/SmallValue.java
   Mon Jan 28 21:44:57 2019 +0100
+++ b/src/java.base/share/classes/sun/security/util/math/SmallValue.java
   Tue Jan 29 01:21:13 2019 +0300
@@ -28,7 +28,7 @@
 /**
  * A "small" value that can be used with the field arithmetic library. This
  * interface enables optimizations based on the fact that certain values are
- * known to be small, where the definition of small is specific to the the
+ * known to be small, where the definition of small is specific to the
  * arithmetic implementation.
  */


More information about the core-libs-dev mailing list