[Bug][JDK10] Fix duplicated "the" occurrences in docs and inline comments

Roger Riggs roger.riggs at oracle.com
Sun Oct 29 22:25:09 UTC 2017


Hi Christoph,

I'll sponsor it.  [1]
BTW, email wrapped some of the lines in the patch; I'll fix it up.

Thanks, Roger

https://bugs.openjdk.java.net/browse/JDK-8190323


On 10/29/17 11:24 AM, Christoph Dreis wrote:
> Hey,
>
> Just noticed an occurrence of a duplicated "the" in a Locale doc and looked
> for other occurrences in java.base while being on it.
>
> I don't know if documentation changes follow a special workflow because of
> possible translations. Please let me know!
>
> But I'd be happy if this is sponsored or moved to the appropriate mailing
> list.
>
> Cheers,
> Christoph
>
>
> ===== PATCH ======
> diff -r 359c604930af src/java.base/share/classes/java/lang/WeakPairMap.java
> --- a/src/java.base/share/classes/java/lang/WeakPairMap.java	Fri Oct 27
> 09:51:48 2017 -0700
> +++ b/src/java.base/share/classes/java/lang/WeakPairMap.java	Sun Oct 29
> 16:13:11 2017 +0100
> @@ -88,7 +88,7 @@
>        * Maps the specified key pair to the specified value in this
> WeakPairMap.
>        * Neither the keys nor the value can be null.
>        * <p>The value can be retrieved by calling the {@link #get} method
> -     * with the the same keys (compared by identity).
> +     * with the same keys (compared by identity).
>        *
>        * @param k1 the 1st of the pair of keys with which the specified value
> is to
>        *           be associated
>
> diff -r 359c604930af
> src/java.base/share/classes/java/lang/invoke/MethodHandleImpl.java
> --- a/src/java.base/share/classes/java/lang/invoke/MethodHandleImpl.java
> Fri Oct 27 09:51:48 2017 -0700
> +++ b/src/java.base/share/classes/java/lang/invoke/MethodHandleImpl.java
> Sun Oct 29 16:13:11 2017 +0100
> @@ -1196,7 +1196,7 @@
>   
>           static
>           MethodHandle bindCaller(MethodHandle mh, Class<?> hostClass) {
> -            // Code in the the boot layer should now be careful while
> creating method handles or
> +            // Code in the boot layer should now be careful while creating
> method handles or
>               // functional interface instances created from method
> references to @CallerSensitive  methods,
>               // it needs to be ensured the handles or interface instances
> are kept safe and are not passed
>               // from the boot layer to untrusted code.
>
> diff -r 359c604930af
> src/java.base/share/classes/java/lang/invoke/StringConcatFactory.java
> --- a/src/java.base/share/classes/java/lang/invoke/StringConcatFactory.java
> Fri Oct 27 09:51:48 2017 -0700
> +++ b/src/java.base/share/classes/java/lang/invoke/StringConcatFactory.java
> Sun Oct 29 16:13:11 2017 +0100
> @@ -195,7 +195,7 @@
>           // In case we need to double-back onto the StringConcatFactory
> during this
>           // static initialization, make sure we have the reasonable defaults
> to complete
>           // the static initialization properly. After that, actual users
> would use the
> -        // the proper values we have read from the the properties.
> +        // the proper values we have read from the properties.
>           STRATEGY = DEFAULT_STRATEGY;
>           // CACHE_ENABLE = false; // implied
>           // CACHE = null;         // implied
>
> diff -r 359c604930af src/java.base/share/classes/java/util/Locale.java
> --- a/src/java.base/share/classes/java/util/Locale.java	Fri Oct 27 09:51:48
> 2017 -0700
> +++ b/src/java.base/share/classes/java/util/Locale.java	Sun Oct 29 16:13:11
> 2017 +0100
> @@ -1909,7 +1909,7 @@
>        * Returns a name for the locale that is appropriate for display to the
>        * user. This will be the values returned by getDisplayLanguage(),
>        * getDisplayScript(), getDisplayCountry(), and getDisplayVariant()
> assembled
> -     * into a single string. The the non-empty values are used in order,
> +     * into a single string. The non-empty values are used in order,
>        * with the second and subsequent names in parentheses.  For example:
>        * <blockquote>
>        * language (script, country, variant)<br>
>
> diff -r 359c604930af
> src/java.base/share/classes/java/util/ServiceLoader.java
> --- a/src/java.base/share/classes/java/util/ServiceLoader.java	Fri Oct 27
> 09:51:48 2017 -0700
> +++ b/src/java.base/share/classes/java/util/ServiceLoader.java	Sun Oct 29
> 16:13:11 2017 +0100
> @@ -1409,7 +1409,7 @@
>        *
>        * <p> To achieve laziness the actual work of locating providers is
> done
>        * when processing the stream. If a service provider cannot be loaded
> for any
> -     * of the the reasons specified in the <a href="#errors">Errors</a>
> section
> +     * of the reasons specified in the <a href="#errors">Errors</a> section
>        * above then {@link ServiceConfigurationError} is thrown by whatever
> method
>        * caused the service provider to be loaded. </p>
>        *
>
> diff -r 359c604930af
> src/java.base/share/classes/jdk/internal/module/ModuleBootstrap.java
> --- a/src/java.base/share/classes/jdk/internal/module/ModuleBootstrap.java
> Fri Oct 27 09:51:48 2017 -0700
> +++ b/src/java.base/share/classes/jdk/internal/module/ModuleBootstrap.java
> Sun Oct 29 16:13:11 2017 +0100
> @@ -66,7 +66,7 @@
>    * -m and --add-modules options. The modules are located on a module path
> that
>    * is constructed from the upgrade module path, system modules, and
> application
>    * module path. The Configuration is instantiated as the boot layer with
> each
> - * module in the the configuration defined to a class loader.
> + * module in the configuration defined to a class loader.
>    */
>   
>   public final class ModuleBootstrap {
>
> diff -r 359c604930af
> src/java.base/share/classes/jdk/internal/module/ModuleHashesBuilder.java
> ---
> a/src/java.base/share/classes/jdk/internal/module/ModuleHashesBuilder.java
> Fri Oct 27 09:51:48 2017 -0700
> +++
> b/src/java.base/share/classes/jdk/internal/module/ModuleHashesBuilder.java
> Sun Oct 29 16:13:11 2017 +0100
> @@ -74,7 +74,7 @@
>        * the outgoing edges from M to non-candidate modules.
>        */
>       public Map<String, ModuleHashes> computeHashes(Set<String> roots) {
> -        // build a graph containing the the packaged modules and
> +        // build a graph containing the packaged modules and
>           // its transitive dependences matching --hash-modules
>           Graph.Builder<String> builder = new Graph.Builder<>();
>           Deque<ResolvedModule> deque = new
> ArrayDeque<>(configuration.modules());
>
> diff -r 359c604930af
> src/java.base/share/classes/jdk/internal/module/Resources.java
> --- a/src/java.base/share/classes/jdk/internal/module/Resources.java	Fri
> Oct 27 09:51:48 2017 -0700
> +++ b/src/java.base/share/classes/jdk/internal/module/Resources.java	Sun
> Oct 29 16:13:11 2017 +0100
> @@ -56,7 +56,7 @@
>       /**
>        * Derive a <em>package name</em> for a resource. The package name
>        * returned by this method may not be a legal package name. This method
> -     * returns null if the the resource name ends with a "/" (a directory)
> +     * returns null if the resource name ends with a "/" (a directory)
>        * or the resource name does not contain a "/".
>        */
>       public static String toPackageName(String name) {
>
> diff -r 359c604930af
> src/java.base/share/classes/jdk/internal/org/objectweb/asm/Frame.java
> --- a/src/java.base/share/classes/jdk/internal/org/objectweb/asm/Frame.java
> Fri Oct 27 09:51:48 2017 -0700
> +++ b/src/java.base/share/classes/jdk/internal/org/objectweb/asm/Frame.java
> Sun Oct 29 16:13:11 2017 +0100
> @@ -192,7 +192,7 @@
>       private static final int LOCAL = 0x2000000;
>   
>       /**
> -     * Kind of the the types that are relative to the stack of an input
> stack
> +     * Kind of the types that are relative to the stack of an input stack
>        * map frame. The value of such types is a position relatively to the
> top of
>        * this stack.
>        */
>
> diff -r 359c604930af
> src/java.base/share/classes/sun/security/provider/certpath/ResponderId.java
> ---
> a/src/java.base/share/classes/sun/security/provider/certpath/ResponderId.jav
> a	Fri Oct 27 09:51:48 2017 -0700
> +++
> b/src/java.base/share/classes/sun/security/provider/certpath/ResponderId.jav
> a	Sun Oct 29 16:13:11 2017 +0100
> @@ -119,7 +119,7 @@
>        * When encoded in DER this object will use the byKey option, a
>        * SHA-1 hash of the responder's public key.
>        *
> -     * @param pubKey the the OCSP responder's public key
> +     * @param pubKey the OCSP responder's public key
>        *
>        * @throws IOException if the internal DER-encoding of the
>        *      {@code KeyIdentifier} fails.
>
> diff -r 359c604930af
> src/java.base/share/classes/sun/security/provider/certpath/SunCertPathBuilde
> rException.java
> ---
> a/src/java.base/share/classes/sun/security/provider/certpath/SunCertPathBuil
> derException.java	Fri Oct 27 09:51:48 2017 -0700
> +++
> b/src/java.base/share/classes/sun/security/provider/certpath/SunCertPathBuil
> derException.java	Sun Oct 29 16:13:11 2017 +0100
> @@ -94,7 +94,7 @@
>       }
>   
>       /**
> -     * Creates a <code>SunCertPathBuilderException</code> withe the
> specified
> +     * Creates a <code>SunCertPathBuilderException</code> with the
> specified
>        * detail message and adjacency list.
>        *
>        * @param msg the detail message
>
> diff -r 359c604930af
> src/java.base/share/classes/sun/security/ssl/CertStatusReqItemV2.java
> --- a/src/java.base/share/classes/sun/security/ssl/CertStatusReqItemV2.java
> Fri Oct 27 09:51:48 2017 -0700
> +++ b/src/java.base/share/classes/sun/security/ssl/CertStatusReqItemV2.java
> Sun Oct 29 16:13:11 2017 +0100
> @@ -145,7 +145,7 @@
>        * @return the encoded length of this {@code CertStatusReqItemV2}
>        */
>       int length() {
> -        // The length is the the status type (1 byte) + the request length
> +        // The length is the status type (1 byte) + the request length
>           // field (2 bytes) + the StatusRequest data length.
>           return request.length() + 3;
>       }
>
> diff -r 359c604930af src/java.base/share/classes/sun/security/ssl/MAC.java
> --- a/src/java.base/share/classes/sun/security/ssl/MAC.java	Fri Oct 27
> 09:51:48 2017 -0700
> +++ b/src/java.base/share/classes/sun/security/ssl/MAC.java	Sun Oct 29
> 16:13:11 2017 +0100
> @@ -188,7 +188,7 @@
>        * @param type record type
>        * @param bb a ByteBuffer in which the position and limit
>        *        demarcate the data to be MAC'd.
> -     * @param isSimulated if true, simulate the the MAC computation
> +     * @param isSimulated if true, simulate the MAC computation
>        *
>        * @return the MAC result
>        */
>
> diff -r 359c604930af
> src/java.base/share/classes/sun/security/ssl/TrustStoreManager.java
> --- a/src/java.base/share/classes/sun/security/ssl/TrustStoreManager.java
> Fri Oct 27 09:51:48 2017 -0700
> +++ b/src/java.base/share/classes/sun/security/ssl/TrustStoreManager.java
> Sun Oct 29 16:13:11 2017 +0100
> @@ -333,7 +333,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 359c604930af
> src/java.base/share/classes/sun/text/normalizer/Trie2.java
> --- a/src/java.base/share/classes/sun/text/normalizer/Trie2.java	Fri
> Oct 27 09:51:48 2017 -0700
> +++ b/src/java.base/share/classes/sun/text/normalizer/Trie2.java	Sun
> Oct 29 16:13:11 2017 +0100
> @@ -592,7 +592,7 @@
>           //   may be lower when iterating over the code points for a single
> lead surrogate.
>           private int            limitCP;
>   
> -        // True while iterating over the the Trie2 values for code points.
> +        // True while iterating over the Trie2 values for code points.
>           // False while iterating over the alternate values for lead
> surrogates.
>           private boolean        doingCodePoints = true;
>
>



More information about the core-libs-dev mailing list