[OpenJDK 2D-Dev] RFR. Repeated words typos in java.desktop

[Alexey Ivanov]

Hi Andrey,

src/java.desktop/share/classes/com/sun/java/swing/plaf/gtk/GTKLookAndFeel.java
      /*
       * Used to override if system (desktop) text anti-aliasing 
settings should
-     * be used. The reasons for this are are is that currently its "off"
+     * be used. The reasons for this are is that currently its "off"
       * for CJK locales which is not likely to be a good universal 
answer, and
       * also its off for remote display. So this provides an unsupported
       * way to explicitly request that it be "on".

I guess it makes sense to correct the grammar as well:
-     * be used. The reasons for this are are is that currently its "off"
+     * be used. The reasons for this are that currently it's "off"
       * for CJK locales which is not likely to be a good universal 
answer, and
-      * also its off for remote display. So this provides an unsupported
+      * also it's off for remote display. So this provides an unsupported
       * way to explicitly request that it be "on".


src/java.desktop/share/classes/javax/imageio/metadata/IIOInvalidTreeException.java
- * error may occur at a different point that that where it is
+ * error may occur at a different point than where it is


src/java.desktop/share/classes/javax/imageio/metadata/doc-files/tiff_metadata.html
  <li><code>PlanarConfiguration</code> if this field is present with value
-<code>Planar</code> is is reset to <code>Chunky</code>.</li>
+<code>Planar</code>, it is reset to <code>Chunky</code>.</li>

I believe this is the correct way.


src/java.desktop/share/classes/javax/swing/table/DefaultTableCellRenderer.java
-     * to prune code-paths are often called in the case of renders
+     * prune code-paths that are often called in the case of renders


src/java.desktop/share/classes/javax/swing/text/FlowView.java
-     * Create a View that should be used to hold a
+     * Create a View that should be used to hold
       * a rows worth of children in a flow.  This is

This does not sound right. Should it rather be “…to hold rows worth of 
children in a flow.” or “…to hold a row worth…”?


src/java.desktop/share/classes/javax/swing/text/ParagraphView.java
The same comment as for FlowView.java above.


src/java.desktop/share/classes/sun/awt/util/PerformanceLogger.java
       * Sets the value of a given time and returns the index of the
-     * slot that that time was stored in.
+     * slot where the time was stored in.

Does it sound better?


src/java.desktop/windows/classes/sun/awt/windows/WPathGraphics.java
-                /* let's see if we can use a a default pen
+                /* let's see if we can use the default pen

Looks this requires the definite article as the default pen is somewhat 
defined.


Regards,
Alexey

On 31/01/2019 22:28, Andrey Turbanov wrote:
> Hello.
> I would like to contribute small patch to fix repeated words typos.
>
> Andrey Turbanov
>
>
> <SNIP>