The fix looks good to me.
Thanks,
Alexandr.
On 9/1/2016 11:46 PM, Iris Clark wrote:
Hi.
Let's try this again.
Please review the following one character change to remove an unnecessary ')'
in the first sentence of Toolkit.isDynamicLayoutActive(). The matching '(' was
removed during resolution of JDK-8027234 (build 120) which clarified the
specification of this method.
Bug:
8165269: (doc) Toolkit.isDynamicLayoutActive(): orphan ')' in first
sentence
https://bugs.openjdk.java.net/browse/JDK-8165269
patch (from webrev):
http://cr.openjdk.java.net/~iris/8165269/webrev/jdk.patch
Thanks,
iris
diff -r 14918637b76e src/java.desktop/share/classes/java/awt/Toolkit.java
--- a/src/java.desktop/share/classes/java/awt/Toolkit.java Thu Sep 01
16:18:14 2016 +0530
+++ b/src/java.desktop/share/classes/java/awt/Toolkit.java Thu Sep 01
13:10:03 2016 -0700
@@ -208,7 +208,7 @@
/**
* Returns whether dynamic layout of Containers on resize is currently
- * enabled on the underlying operating system and/or window manager). If
the
+ * enabled on the underlying operating system and/or window manager. If the
* platform supports it, {@code setDynamicLayout(boolean)} may be used to
* programmatically enable or disable platform dynamic layout. Regardless
of
* whether that toggling is supported, or whether {@code true} or {@code
