changeset 10779:95b258c53752

8194871: Fix mistakes in FX API docs Reviewed-by: kcr Contributed-by: nlisker@gmail.com
author kcr
date Wed, 10 Jan 2018 16:59:50 -0800
parents b43e44ecc365
children ef935ffcd223
files modules/javafx.base/src/main/java/javafx/beans/binding/Bindings.java modules/javafx.base/src/main/java/javafx/beans/value/ObservableValue.java modules/javafx.base/src/main/java/javafx/collections/FXCollections.java modules/javafx.base/src/main/java/javafx/collections/ListChangeListener.java modules/javafx.controls/src/main/java/javafx/scene/control/TableView.java modules/javafx.controls/src/main/java/javafx/scene/control/skin/TextAreaSkin.java modules/javafx.graphics/src/main/java/javafx/animation/Animation.java modules/javafx.graphics/src/main/java/javafx/animation/Timeline.java modules/javafx.graphics/src/main/java/javafx/animation/Transition.java modules/javafx.graphics/src/main/java/javafx/scene/Node.java modules/javafx.graphics/src/main/java/javafx/scene/Parent.java modules/javafx.graphics/src/main/java/javafx/scene/image/ImageView.java modules/javafx.graphics/src/main/java/javafx/scene/layout/HBox.java modules/javafx.graphics/src/main/java/javafx/scene/shape/TriangleMesh.java modules/javafx.graphics/src/main/java/javafx/scene/transform/MatrixType.java
diffstat 15 files changed, 109 insertions(+), 90 deletions(-) [+]
line wrap: on
line diff
--- a/modules/javafx.base/src/main/java/javafx/beans/binding/Bindings.java	Mon Jan 08 22:25:50 2018 -0800
+++ b/modules/javafx.base/src/main/java/javafx/beans/binding/Bindings.java	Wed Jan 10 16:59:50 2018 -0800
@@ -5806,12 +5806,12 @@
      * equal to a constant value.
      *
      * @param op1
-     *            the {@code ObservableCharacterValue}
+     *            the {@code ObservableObjectValue}
      * @param op2
      *            the constant value
      * @return the new {@code BooleanBinding}
      * @throws NullPointerException
-     *             if the {@code ObservableCharacterValue} is {@code null}
+     *             if the {@code ObservableObjectValue} is {@code null}
      */
     public static BooleanBinding equal(final ObservableObjectValue<?> op1, Object op2) {
         return equal(op1, ObjectConstant.valueOf(op2), op1);
@@ -5825,10 +5825,10 @@
      * @param op1
      *            the constant value
      * @param op2
-     *            the {@code ObservableCharacterValue}
+     *            the {@code ObservableObjectValue}
      * @return the new {@code BooleanBinding}
      * @throws NullPointerException
-     *             if the {@code ObservableCharacterValue} is {@code null}
+     *             if the {@code ObservableObjectValue} is {@code null}
      */
     public static BooleanBinding equal(Object op1, final ObservableObjectValue<?> op2) {
         return equal(ObjectConstant.valueOf(op1), op2, op2);
@@ -6116,7 +6116,7 @@
     /**
      * Creates a new {@link javafx.beans.binding.ObjectBinding} that contains the element
      * of an {@link javafx.collections.ObservableList} at the specified position. The {@code ObjectBinding}
-     * will contain {@code null}, if the {@code index} points behind the {@code ObservableList}.
+     * will contain {@code null} if the {@code index} points behind the {@code ObservableList}.
      *
      * @param op the {@code ObservableList}
      * @param index the position in the {@code List}
@@ -6164,7 +6164,7 @@
     /**
      * Creates a new {@link javafx.beans.binding.ObjectBinding} that contains the element
      * of an {@link javafx.collections.ObservableList} at the specified position. The {@code ObjectBinding}
-     * will contain {@code null}, if the {@code index} is outside of the {@code ObservableList}.
+     * will contain {@code null} if the {@code index} is outside of the {@code ObservableList}.
      *
      * @param op the {@code ObservableList}
      * @param index the position in the {@code List}
@@ -6180,7 +6180,7 @@
     /**
      * Creates a new {@link javafx.beans.binding.ObjectBinding} that contains the element
      * of an {@link javafx.collections.ObservableList} at the specified position. The {@code ObjectBinding}
-     * will contain {@code null}, if the {@code index} is outside of the {@code ObservableList}.
+     * will contain {@code null} if the {@code index} is outside of the {@code ObservableList}.
      *
      * @param op the {@code ObservableList}
      * @param index the position in the {@code List}, converted to int
@@ -6224,7 +6224,7 @@
     /**
      * Creates a new {@link javafx.beans.binding.BooleanBinding} that contains the element
      * of an {@link javafx.collections.ObservableList} at the specified position. The {@code BooleanBinding}
-     * will hold {@code false}, if the {@code index} points behind the {@code ObservableList}.
+     * will hold {@code false} if the {@code index} points behind the {@code ObservableList}.
      *
      * @param op the {@code ObservableList}
      * @param index the position in the {@code List}
@@ -6276,7 +6276,7 @@
     /**
      * Creates a new {@link javafx.beans.binding.BooleanBinding} that contains the element
      * of an {@link javafx.collections.ObservableList} at the specified position. The {@code BooleanBinding}
-     * will hold {@code false}, if the {@code index} is outside of the {@code ObservableList}.
+     * will hold {@code false} if the {@code index} is outside of the {@code ObservableList}.
      *
      * @param op the {@code ObservableList}
      * @param index the position in the {@code List}
@@ -6291,7 +6291,7 @@
     /**
      * Creates a new {@link javafx.beans.binding.BooleanBinding} that contains the element
      * of an {@link javafx.collections.ObservableList} at the specified position. The {@code BooleanBinding}
-     * will hold {@code false}, if the {@code index} is outside of the {@code ObservableList}.
+     * will hold {@code false} if the {@code index} is outside of the {@code ObservableList}.
      *
      * @param op the {@code ObservableList}
      * @param index the position in the {@code List}, converted to int
@@ -6339,7 +6339,7 @@
     /**
      * Creates a new {@link javafx.beans.binding.DoubleBinding} that contains the element
      * of an {@link javafx.collections.ObservableList} at the specified position. The {@code DoubleBinding}
-     * will hold {@code 0.0}, if the {@code index} points behind the {@code ObservableList}.
+     * will hold {@code 0.0} if the {@code index} points behind the {@code ObservableList}.
      *
      * @param op the {@code ObservableList}
      * @param index the position in the {@code List}
@@ -6391,7 +6391,7 @@
     /**
      * Creates a new {@link javafx.beans.binding.DoubleBinding} that contains the element
      * of an {@link javafx.collections.ObservableList} at the specified position. The {@code DoubleBinding}
-     * will hold {@code 0.0}, if the {@code index} is outside of the {@code ObservableList}.
+     * will hold {@code 0.0} if the {@code index} is outside of the {@code ObservableList}.
      *
      * @param op the {@code ObservableList}
      * @param index the position in the {@code List}
@@ -6406,7 +6406,7 @@
     /**
      * Creates a new {@link javafx.beans.binding.DoubleBinding} that contains the element
      * of an {@link javafx.collections.ObservableList} at the specified position. The {@code DoubleBinding}
-     * will hold {@code 0.0}, if the {@code index} is outside of the {@code ObservableList}.
+     * will hold {@code 0.0} if the {@code index} is outside of the {@code ObservableList}.
      *
      * @param op the {@code ObservableList}
      * @param index the position in the {@code List}, converted to int
@@ -6454,7 +6454,7 @@
     /**
      * Creates a new {@link javafx.beans.binding.FloatBinding} that contains the element
      * of an {@link javafx.collections.ObservableList} at the specified position. The {@code FloatBinding}
-     * will hold {@code 0.0f}, if the {@code index} points behind the {@code ObservableList}.
+     * will hold {@code 0.0f} if the {@code index} points behind the {@code ObservableList}.
      *
      * @param op the {@code ObservableList}
      * @param index the position in the {@code List}
@@ -6506,7 +6506,7 @@
     /**
      * Creates a new {@link javafx.beans.binding.FloatBinding} that contains the element
      * of an {@link javafx.collections.ObservableList} at the specified position. The {@code FloatBinding}
-     * will hold {@code 0.0f}, if the {@code index} is outside of the {@code ObservableList}.
+     * will hold {@code 0.0f} if the {@code index} is outside of the {@code ObservableList}.
      *
      * @param op the {@code ObservableList}
      * @param index the position in the {@code List}
@@ -6521,7 +6521,7 @@
     /**
      * Creates a new {@link javafx.beans.binding.FloatBinding} that contains the element
      * of an {@link javafx.collections.ObservableList} at the specified position. The {@code FloatBinding}
-     * will hold {@code 0.0f}, if the {@code index} is outside of the {@code ObservableList}.
+     * will hold {@code 0.0f} if the {@code index} is outside of the {@code ObservableList}.
      *
      * @param op the {@code ObservableList}
      * @param index the position in the {@code List}, converted to int
@@ -6569,7 +6569,7 @@
     /**
      * Creates a new {@link javafx.beans.binding.IntegerBinding} that contains the element
      * of an {@link javafx.collections.ObservableList} at the specified position. The {@code IntegerBinding}
-     * will hold {@code 0}, if the {@code index} points behind the {@code ObservableList}.
+     * will hold {@code 0} if the {@code index} points behind the {@code ObservableList}.
      *
      * @param op the {@code ObservableList}
      * @param index the position in the {@code List}
@@ -6621,7 +6621,7 @@
     /**
      * Creates a new {@link javafx.beans.binding.IntegerBinding} that contains the element
      * of an {@link javafx.collections.ObservableList} at the specified position. The {@code IntegerBinding}
-     * will hold {@code 0}, if the {@code index} is outside of the {@code ObservableList}.
+     * will hold {@code 0} if the {@code index} is outside of the {@code ObservableList}.
      *
      * @param op the {@code ObservableList}
      * @param index the position in the {@code List}
@@ -6636,7 +6636,7 @@
     /**
      * Creates a new {@link javafx.beans.binding.IntegerBinding} that contains the element
      * of an {@link javafx.collections.ObservableList} at the specified position. The {@code IntegerBinding}
-     * will hold {@code 0}, if the {@code index} is outside of the {@code ObservableList}.
+     * will hold {@code 0} if the {@code index} is outside of the {@code ObservableList}.
      *
      * @param op the {@code ObservableList}
      * @param index the position in the {@code List}, converted to int
@@ -6684,7 +6684,7 @@
     /**
      * Creates a new {@link javafx.beans.binding.LongBinding} that contains the element
      * of an {@link javafx.collections.ObservableList} at the specified position. The {@code LongBinding}
-     * will hold {@code 0L}, if the {@code index} points behind the {@code ObservableList}.
+     * will hold {@code 0L} if the {@code index} points behind the {@code ObservableList}.
      *
      * @param op the {@code ObservableList}
      * @param index the position in the {@code List}
@@ -6736,7 +6736,7 @@
     /**
      * Creates a new {@link javafx.beans.binding.LongBinding} that contains the element
      * of an {@link javafx.collections.ObservableList} at the specified position. The {@code LongBinding}
-     * will hold {@code 0L}, if the {@code index} is outside of the {@code ObservableList}.
+     * will hold {@code 0L} if the {@code index} is outside of the {@code ObservableList}.
      *
      * @param op the {@code ObservableList}
      * @param index the position in the {@code List}
@@ -6751,7 +6751,7 @@
     /**
      * Creates a new {@link javafx.beans.binding.LongBinding} that contains the element
      * of an {@link javafx.collections.ObservableList} at the specified position. The {@code LongBinding}
-     * will hold {@code 0L}, if the {@code index} is outside of the {@code ObservableList}.
+     * will hold {@code 0L} if the {@code index} is outside of the {@code ObservableList}.
      *
      * @param op the {@code ObservableList}
      * @param index the position in the {@code List}, converted to int
@@ -6799,7 +6799,7 @@
     /**
      * Creates a new {@link javafx.beans.binding.StringBinding} that contains the element
      * of an {@link javafx.collections.ObservableList} at the specified position. The {@code StringBinding}
-     * will hold {@code null}, if the {@code index} points behind the {@code ObservableList}.
+     * will hold {@code null} if the {@code index} points behind the {@code ObservableList}.
      *
      * @param op the {@code ObservableList}
      * @param index the position in the {@code List}
@@ -6846,7 +6846,7 @@
     /**
      * Creates a new {@link javafx.beans.binding.StringBinding} that contains the element
      * of an {@link javafx.collections.ObservableList} at the specified position. The {@code StringBinding}
-     * will hold {@code ""}, if the {@code index} is outside of the {@code ObservableList}.
+     * will hold {@code ""} if the {@code index} is outside of the {@code ObservableList}.
      *
      * @param op the {@code ObservableList}
      * @param index the position in the {@code List}
@@ -6861,7 +6861,7 @@
     /**
      * Creates a new {@link javafx.beans.binding.StringBinding} that contains the element
      * of an {@link javafx.collections.ObservableList} at the specified position. The {@code StringBinding}
-     * will hold {@code ""}, if the {@code index} is outside of the {@code ObservableList}.
+     * will hold {@code ""} if the {@code index} is outside of the {@code ObservableList}.
      *
      * @param op the {@code ObservableList}
      * @param index the position in the {@code List}, converted to int
@@ -7064,7 +7064,7 @@
     /**
      * Creates a new {@link javafx.beans.binding.FloatBinding} that contains the element
      * of an {@link javafx.collections.ObservableArray} at the specified position. The {@code FloatBinding}
-     * will hold {@code 0.0f}, if the {@code index} points behind the {@code ObservableArray}.
+     * will hold {@code 0.0f} if the {@code index} points behind the {@code ObservableArray}.
      *
      * @param op the {@code ObservableArray}
      * @param index the position in the {@code ObservableArray}
@@ -7111,7 +7111,7 @@
     /**
      * Creates a new {@link javafx.beans.binding.FloatBinding} that contains the element
      * of an {@link javafx.collections.ObservableArray} at the specified position. The {@code FloatBinding}
-     * will hold {@code 0.0f}, if the {@code index} is outside of the {@code ObservableArray}.
+     * will hold {@code 0.0f} if the {@code index} is outside of the {@code ObservableArray}.
      *
      * @param op the {@code ObservableArray}
      * @param index the position in the {@code ObservableArray}
@@ -7126,7 +7126,7 @@
     /**
      * Creates a new {@link javafx.beans.binding.FloatBinding} that contains the element
      * of an {@link javafx.collections.ObservableArray} at the specified position. The {@code FloatBinding}
-     * will hold {@code 0.0f}, if the {@code index} is outside of the {@code ObservableArray}.
+     * will hold {@code 0.0f} if the {@code index} is outside of the {@code ObservableArray}.
      *
      * @param op the {@code ObservableArray}
      * @param index the position in the {@code ObservableArray}, converted to int
@@ -7169,7 +7169,7 @@
     /**
      * Creates a new {@link javafx.beans.binding.IntegerBinding} that contains the element
      * of an {@link javafx.collections.ObservableArray} at the specified position. The {@code IntegerBinding}
-     * will hold {@code 0}, if the {@code index} points behind the {@code ObservableArray}.
+     * will hold {@code 0} if the {@code index} points behind the {@code ObservableArray}.
      *
      * @param op the {@code ObservableArray}
      * @param index the position in the {@code ObservableArray}
@@ -7216,7 +7216,7 @@
     /**
      * Creates a new {@link javafx.beans.binding.IntegerBinding} that contains the element
      * of an {@link javafx.collections.ObservableArray} at the specified position. The {@code IntegerBinding}
-     * will hold {@code 0}, if the {@code index} is outside of the {@code ObservableArray}.
+     * will hold {@code 0} if the {@code index} is outside of the {@code ObservableArray}.
      *
      * @param op the {@code ObservableArray}
      * @param index the position in the {@code ObservableArray}
@@ -7231,7 +7231,7 @@
     /**
      * Creates a new {@link javafx.beans.binding.IntegerBinding} that contains the element
      * of an {@link javafx.collections.ObservableArray} at the specified position. The {@code IntegerBinding}
-     * will hold {@code 0}, if the {@code index} is outside of the {@code ObservableArray}.
+     * will hold {@code 0} if the {@code index} is outside of the {@code ObservableArray}.
      *
      * @param op the {@code ObservableArray}
      * @param index the position in the {@code ObservableArray}, converted to int
@@ -7494,7 +7494,7 @@
     /**
      * Creates a new {@link javafx.beans.binding.BooleanBinding} that contains the mapping of a specific key
      * in an {@link javafx.collections.ObservableMap}. The {@code BooleanBinding}
-     * will hold {@code false}, if the {@code key} cannot be found in the {@code ObservableMap}.
+     * will hold {@code false} if the {@code key} cannot be found in the {@code ObservableMap}.
      *
      * @param op the {@code ObservableMap}
      * @param key the key in the {@code Map}
@@ -7547,7 +7547,7 @@
     /**
      * Creates a new {@link javafx.beans.binding.BooleanBinding} that contains the mapping of a specific key
      * in an {@link javafx.collections.ObservableMap}. The {@code BooleanBinding}
-     * will hold {@code false}, if the {@code key} cannot be found in the {@code ObservableMap}.
+     * will hold {@code false} if the {@code key} cannot be found in the {@code ObservableMap}.
      *
      * @param op the {@code ObservableMap}
      * @param key the key in the {@code Map}
@@ -7600,7 +7600,7 @@
     /**
      * Creates a new {@link javafx.beans.binding.DoubleBinding} that contains the mapping of a specific key
      * in an {@link javafx.collections.ObservableMap}. The {@code DoubleBinding}
-     * will hold {@code 0.0}, if the {@code key} cannot be found in the {@code ObservableMap}.
+     * will hold {@code 0.0} if the {@code key} cannot be found in the {@code ObservableMap}.
      *
      * @param op the {@code ObservableMap}
      * @param key the key in the {@code Map}
@@ -7653,7 +7653,7 @@
     /**
      * Creates a new {@link javafx.beans.binding.DoubleBinding} that contains the mapping of a specific key
      * in an {@link javafx.collections.ObservableMap}. The {@code DoubleBinding}
-     * will hold {@code 0.0}, if the {@code key} cannot be found in the {@code ObservableMap}.
+     * will hold {@code 0.0} if the {@code key} cannot be found in the {@code ObservableMap}.
      *
      * @param op the {@code ObservableMap}
      * @param key the key in the {@code Map}
@@ -7706,7 +7706,7 @@
     /**
      * Creates a new {@link javafx.beans.binding.FloatBinding} that contains the mapping of a specific key
      * in an {@link javafx.collections.ObservableMap}. The {@code FloatBinding}
-     * will hold {@code 0.0f}, if the {@code key} cannot be found in the {@code ObservableMap}.
+     * will hold {@code 0.0f} if the {@code key} cannot be found in the {@code ObservableMap}.
      *
      * @param op the {@code ObservableMap}
      * @param key the key in the {@code Map}
@@ -7759,7 +7759,7 @@
     /**
      * Creates a new {@link javafx.beans.binding.FloatBinding} that contains the mapping of a specific key
      * in an {@link javafx.collections.ObservableMap}. The {@code FloatBinding}
-     * will hold {@code 0.0f}, if the {@code key} cannot be found in the {@code ObservableMap}.
+     * will hold {@code 0.0f} if the {@code key} cannot be found in the {@code ObservableMap}.
      *
      * @param op the {@code ObservableMap}
      * @param key the key in the {@code Map}
@@ -7812,7 +7812,7 @@
     /**
      * Creates a new {@link javafx.beans.binding.IntegerBinding} that contains the mapping of a specific key
      * in an {@link javafx.collections.ObservableMap}. The {@code IntegerBinding}
-     * will hold {@code 0}, if the {@code key} cannot be found in the {@code ObservableMap}.
+     * will hold {@code 0} if the {@code key} cannot be found in the {@code ObservableMap}.
      *
      * @param op the {@code ObservableMap}
      * @param key the key in the {@code Map}
@@ -7865,7 +7865,7 @@
     /**
      * Creates a new {@link javafx.beans.binding.IntegerBinding} that contains the mapping of a specific key
      * in an {@link javafx.collections.ObservableMap}. The {@code IntegerBinding}
-     * will hold {@code 0}, if the {@code key} cannot be found in the {@code ObservableMap}.
+     * will hold {@code 0} if the {@code key} cannot be found in the {@code ObservableMap}.
      *
      * @param op the {@code ObservableMap}
      * @param key the key in the {@code Map}
@@ -7918,7 +7918,7 @@
     /**
      * Creates a new {@link javafx.beans.binding.LongBinding} that contains the mapping of a specific key
      * in an {@link javafx.collections.ObservableMap}. The {@code LongBinding}
-     * will hold {@code 0L}, if the {@code key} cannot be found in the {@code ObservableMap}.
+     * will hold {@code 0L} if the {@code key} cannot be found in the {@code ObservableMap}.
      *
      * @param op the {@code ObservableMap}
      * @param key the key in the {@code Map}
@@ -7971,7 +7971,7 @@
     /**
      * Creates a new {@link javafx.beans.binding.LongBinding} that contains the mapping of a specific key
      * in an {@link javafx.collections.ObservableMap}. The {@code LongBinding}
-     * will hold {@code 0L}, if the {@code key} cannot be found in the {@code ObservableMap}.
+     * will hold {@code 0L} if the {@code key} cannot be found in the {@code ObservableMap}.
      *
      * @param op the {@code ObservableMap}
      * @param key the key in the {@code Map}
@@ -8024,7 +8024,7 @@
     /**
      * Creates a new {@link javafx.beans.binding.StringBinding} that contains the mapping of a specific key
      * in an {@link javafx.collections.ObservableMap}. The {@code StringBinding}
-     * will hold {@code null}, if the {@code key} cannot be found in the {@code ObservableMap}.
+     * will hold {@code null} if the {@code key} cannot be found in the {@code ObservableMap}.
      *
      * @param op the {@code ObservableMap}
      * @param key the key in the {@code Map}
@@ -8072,7 +8072,7 @@
     /**
      * Creates a new {@link javafx.beans.binding.StringBinding} that contains the mapping of a specific key
      * in an {@link javafx.collections.ObservableMap}. The {@code StringBinding}
-     * will hold {@code ""}, if the {@code key} cannot be found in the {@code ObservableMap}.
+     * will hold {@code ""} if the {@code key} cannot be found in the {@code ObservableMap}.
      *
      * @param op the {@code ObservableMap}
      * @param key the key in the {@code Map}
--- a/modules/javafx.base/src/main/java/javafx/beans/value/ObservableValue.java	Mon Jan 08 22:25:50 2018 -0800
+++ b/modules/javafx.base/src/main/java/javafx/beans/value/ObservableValue.java	Wed Jan 10 16:59:50 2018 -0800
@@ -31,8 +31,8 @@
 /**
  * An {@code ObservableValue} is an entity that wraps a value and allows to
  * observe the value for changes. In general this interface should not be
- * implemented directly but one of its sub-interfaces (
- * {@code ObservableBooleanValue} etc.).
+ * implemented directly but one of its sub-interfaces
+ * ({@code ObservableBooleanValue} etc.).
  * <p>
  * The value of the {@code ObservableValue} can be requested with
  * {@link #getValue()}.
--- a/modules/javafx.base/src/main/java/javafx/collections/FXCollections.java	Mon Jan 08 22:25:50 2018 -0800
+++ b/modules/javafx.base/src/main/java/javafx/collections/FXCollections.java	Wed Jan 10 16:59:50 2018 -0800
@@ -234,7 +234,7 @@
     private static ObservableMap EMPTY_OBSERVABLE_MAP = new EmptyObservableMap();
 
     /**
-     * Creates and empty unmodifiable observable map.
+     * Creates an empty unmodifiable observable map.
      * @param <K> the type of the wrapped key
      * @param <V> the type of the wrapped value
      * @return An empty unmodifiable observable map
@@ -436,7 +436,7 @@
 
 
     /**
-     * Creates and empty unmodifiable observable list.
+     * Creates an empty unmodifiable observable list.
      * @param <E> The type of List to be wrapped
      * @return An empty unmodifiable observable list
      * @see Collections#emptyList()
@@ -506,7 +506,7 @@
     private static ObservableSet EMPTY_OBSERVABLE_SET = new EmptyObservableSet();
 
     /**
-     * Creates and empty unmodifiable observable set.
+     * Creates an empty unmodifiable observable set.
      * @param <E> The type of List to be wrapped
      * @return An empty unmodifiable observable set
      * @see Collections#emptySet()
--- a/modules/javafx.base/src/main/java/javafx/collections/ListChangeListener.java	Mon Jan 08 22:25:50 2018 -0800
+++ b/modules/javafx.base/src/main/java/javafx/collections/ListChangeListener.java	Wed Jan 10 16:59:50 2018 -0800
@@ -45,20 +45,20 @@
      * Each change must be one of the following:
      * <ul>
      * <li><b>Permutation change</b> : {@link #wasPermutated()} returns true in this case.
-     * The permutation happened at range between {@link #getFrom() from}(inclusive) and {@link #getTo() to}(exclusive) and
+     * The permutation happened at range between {@link #getFrom() from} (inclusive) and {@link #getTo() to} (exclusive) and
      * can be queried by calling {@link #getPermutation(int)} method.
      * <li><b>Add or remove change</b> : In this case, at least one of the {@link #wasAdded()}, {@link #wasRemoved()} returns true.
      * If both methods return true, {@link #wasReplaced()} will also return true.
      * <p>The {@link #getRemoved()} method returns a list of elements that have been
      * replaced or removed from the list.
-     * <p> The range between {@link #getFrom() from}(inclusive) and {@link #getTo() to}(exclusive)
+     * <p> The range between {@link #getFrom() from} (inclusive) and {@link #getTo() to} (exclusive)
      * denotes the sublist of the list that contain new elements. Note that this is a half-open
      * interval, so if no elements were added, {@code getFrom()} is equal to {@code getTo()}.
      * <p>It is possible to get a list of added elements by calling getAddedSubList().
      * <p>Note that in order to maintain correct indexes of the separate add/remove changes, these changes
      * <b>must</b> be sorted by their {@code from} index.
      * <li><b>Update change</b> : {@link #wasUpdated()} return true on an update change.
-     * All elements between {@link #getFrom() from}(inclusive) and {@link #getTo() to}(exclusive) were updated.
+     * All elements between {@link #getFrom() from} (inclusive) and {@link #getTo() to} (exclusive) were updated.
      * </ul>
      *
      * <b>Important:</b> It's necessary to call {@link #next()} method before calling
--- a/modules/javafx.controls/src/main/java/javafx/scene/control/TableView.java	Mon Jan 08 22:25:50 2018 -0800
+++ b/modules/javafx.controls/src/main/java/javafx/scene/control/TableView.java	Wed Jan 10 16:59:50 2018 -0800
@@ -146,10 +146,10 @@
  *
  * <pre>
  * {@code
- * TableView<Person> table = new TableView<Person>();}</pre>
+ * TableView<Person> table = new TableView<>();}</pre>
  *
  * <p>With the basic table defined, we next focus on the data model. As mentioned,
- * for this example, we'll be using a {@literal ObservableList<Person>}. We can immediately
+ * for this example, we'll be using an {@literal ObservableList<Person>}. We can immediately
  * set such a list directly in to the TableView, as such:
  *
  * <pre>
@@ -174,10 +174,10 @@
  * ObservableList<Person> teamMembers = ...;
  * table.setItems(teamMembers);
  *
- * TableColumn<Person,String> firstNameCol = new TableColumn<Person,String>("First Name");
- * firstNameCol.setCellValueFactory(new PropertyValueFactory("firstName"));
- * TableColumn<Person,String> lastNameCol = new TableColumn<Person,String>("Last Name");
- * lastNameCol.setCellValueFactory(new PropertyValueFactory("lastName"));
+ * TableColumn<Person,String> firstNameCol = new TableColumn<>("First Name");
+ * firstNameCol.setCellValueFactory(new PropertyValueFactory<>("firstName"));
+ * TableColumn<Person,String> lastNameCol = new TableColumn<>("Last Name");
+ * lastNameCol.setCellValueFactory(new PropertyValueFactory<>("lastName"));
  *
  * table.getColumns().setAll(firstNameCol, lastNameCol);}</pre>
  *
@@ -205,7 +205,10 @@
  *         // p.getValue() returns the Person instance for a particular TableView row
  *         return p.getValue().firstNameProperty();
  *     }
- *  });
+ * });
+ *
+ * // or with a lambda expression:
+ * firstNameCol.setCellValueFactory(p -> p.getValue().firstNameProperty());
  * }</pre>
  *
  * <h3>TableView Selection / Focus APIs</h3>
--- a/modules/javafx.controls/src/main/java/javafx/scene/control/skin/TextAreaSkin.java	Mon Jan 08 22:25:50 2018 -0800
+++ b/modules/javafx.controls/src/main/java/javafx/scene/control/skin/TextAreaSkin.java	Wed Jan 10 16:59:50 2018 -0800
@@ -809,6 +809,7 @@
 
     // Public for FXVKSkin
     /**
+     * Gets the {@code Bounds} of the caret of the skinned {@code TextArea}.
      * @return the {@code Bounds} of the caret shape, relative to the {@code TextArea}.
      */
     public Bounds getCaretBounds() {
--- a/modules/javafx.graphics/src/main/java/javafx/animation/Animation.java	Mon Jan 08 22:25:50 2018 -0800
+++ b/modules/javafx.graphics/src/main/java/javafx/animation/Animation.java	Wed Jan 10 16:59:50 2018 -0800
@@ -263,14 +263,14 @@
      * Defines the direction/speed at which the {@code Animation} is expected to
      * be played.
      * <p>
-     * The absolute value of {@code rate} indicates the speed which the
+     * The absolute value of {@code rate} indicates the speed at which the
      * {@code Animation} is to be played, while the sign of {@code rate}
      * indicates the direction. A positive value of {@code rate} indicates
      * forward play, a negative value indicates backward play and {@code 0.0} to
      * stop a running {@code Animation}.
      * <p>
      * Rate {@code 1.0} is normal play, {@code 2.0} is 2 time normal,
-     * {@code -1.0} is backwards, etc...
+     * {@code -1.0} is backwards, etc.
      *
      * <p>
      * Inverting the rate of a running {@code Animation} will cause the
--- a/modules/javafx.graphics/src/main/java/javafx/animation/Timeline.java	Mon Jan 08 22:25:50 2018 -0800
+++ b/modules/javafx.graphics/src/main/java/javafx/animation/Timeline.java	Wed Jan 10 16:59:50 2018 -0800
@@ -73,7 +73,7 @@
  * <p>
  * This Timeline will run for 10s, animating the node by x axis to value 25 and then back to 0 on the second cycle.
  * <p>
- * <b>Warning :</b> A running Timeline is being referenced from the FX runtime. Infinite Timeline
+ * <b>Warning:</b> A running Timeline is being referenced from the FX runtime. Infinite Timeline
  * might result in a memory leak if not stopped properly. All the objects with animated properties would not be garbage collected.
  *
  * @see Animation
--- a/modules/javafx.graphics/src/main/java/javafx/animation/Transition.java	Mon Jan 08 22:25:50 2018 -0800
+++ b/modules/javafx.graphics/src/main/java/javafx/animation/Transition.java	Wed Jan 10 16:59:50 2018 -0800
@@ -157,9 +157,16 @@
     }
 
     /**
-     * Returns the target {@link Node} for animation of this {@code Transition}.
-     * This method returns {@code node} if it is set, else returns its
-     * {@code parent.getTargetNode()} otherwise null.
+     * Returns the first non-{@code null} target {@code Node} in the parent hierarchy of
+     * this {@code Transition}, or {@code null} if such a node is not found.
+     * <p>
+     * A parent animation is one that can have child animations. Examples are
+     * {@link javafx.animation.SequentialTransition SequentialTransition} and
+     * {@link javafx.animation.ParallelTransition ParallelTransition}. A parent animation can
+     * also be a child of another parent animation.
+     * <p>
+     * Note that if this {@code Transition} has a target node set and is not a parent animation,
+     * it will be ignored during the call as this method only queries parent animations.
      * @return the target {@code Node}
      */
     protected Node getParentTargetNode() {
--- a/modules/javafx.graphics/src/main/java/javafx/scene/Node.java	Mon Jan 08 22:25:50 2018 -0800
+++ b/modules/javafx.graphics/src/main/java/javafx/scene/Node.java	Wed Jan 10 16:59:50 2018 -0800
@@ -234,7 +234,7 @@
  * scene graph. The id can also be used identify nodes for applying styles; see
  * the CSS section below.
  *
- * <h4>Coordinate System</h4>
+ * <h3>Coordinate System</h3>
  * <p>
  * The {@code Node} class defines a traditional computer graphics "local"
  * coordinate system in which the {@code x} axis increases to the right and the
@@ -267,7 +267,7 @@
  * important context-specific information about coordinate mapping and how
  * it can affect rendering.
  *
- * <h4>Transformations</h4>
+ * <h3>Transformations</h3>
  * <p>
  * Any {@code Node} can have transformations applied to it. These include
  * translation, rotation, scaling, or shearing.
@@ -2691,7 +2691,7 @@
 
     /**
      * Defines the x coordinate of the translation that is added to this {@code Node}'s
-     * transform for the purpose of layout.  The value should be computed as the
+     * transform for the purpose of layout. The value should be computed as the
      * offset required to adjust the position of the node from its current
      * {@link #layoutBoundsProperty() layoutBounds minX} position (which might not be 0) to the desired location.
      *
@@ -2766,7 +2766,7 @@
 
     /**
      * Defines the y coordinate of the translation that is added to this {@code Node}'s
-     * transform for the purpose of layout.  The value should be computed as the
+     * transform for the purpose of layout. The value should be computed as the
      * offset required to adjust the position of the node from its current
      * {@link #layoutBoundsProperty() layoutBounds minY} position (which might not be 0) to the desired location.
      *
@@ -3383,10 +3383,11 @@
      * the transform created by setting the following additional variables
      * <ol>
      * <li>{@link #getTransforms transforms} ObservableList</li>
-     * <li>{@link #scaleXProperty scaleX}, {@link #scaleYProperty scaleY}</li>
+     * <li>{@link #scaleXProperty scaleX}, {@link #scaleYProperty scaleY}, {@link #scaleZProperty scaleZ}</li>
      * <li>{@link #rotateProperty rotate}</li>
      * <li>{@link #layoutXProperty layoutX}, {@link #layoutYProperty layoutY}</li>
-     * <li>{@link #translateXProperty translateX}, {@link #translateYProperty translateY}</li>
+     * <li>{@link #translateXProperty translateX}, {@link #translateYProperty translateY},
+     * {@link #translateZProperty translateZ}</li>
      * </ol>
      * <p>
      * The resulting bounds will be conceptually in the coordinate space of the
@@ -3398,14 +3399,14 @@
      * <p>
      * This property will always have a non-null value.
      * <p>
-     * Note that boundsInParent is automatically recomputed whenever the
+     * Note that {@code boundsInParent} is automatically recomputed whenever the
      * geometry of a node changes, or when any of the following the change:
-     * transforms ObservableList, translateX, translateY, layoutX, layoutY,
-     * scaleX, scaleY, or the rotate variable. For this reason, it is an error
+     * transforms {@code ObservableList}, any of the translate, layout or scale
+     * variables, or the rotate variable. For this reason, it is an error
      * to bind any of these values in a node to an expression that depends upon
      * this variable. For example, the x or y variables of a shape, or
-     * translateX, translateY should never be bound to boundsInParent
-     * for the purpose of positioning the node.
+     * {@code translateX}, {@code translateY} should never be bound to
+     * {@code boundsInParent} for the purpose of positioning the node.
      * @return the boundsInParent for this {@code Node}
      */
     public final ReadOnlyObjectProperty<Bounds> boundsInParentProperty() {
@@ -3463,25 +3464,29 @@
      * {@link javafx.scene.control.Control Control}, or {@link javafx.scene.web.WebView WebView})
      * then the layoutBounds will always be {@code 0,0 width x height}.
      * If the node type is not resizable ({@link javafx.scene.shape.Shape Shape},
-     * {@link javafx.scene.text.Text Text}, or {@link Group}), then the layoutBounds
+     * {@link javafx.scene.text.Text Text}, or {@link Group}), then the {@code layoutBounds}
      * are computed based on the node's geometric properties and does not include the
      * node's clip, effect, or transforms.  See individual class documentation
      * for details.
      * <p>
-     * Note that the {@link #layoutXProperty layoutX}, {@link #layoutYProperty layoutY}, {@link #translateXProperty translateX}, and
-     * {@link #translateYProperty translateY} variables are not included in the layoutBounds.
+     * Note that the {@link #layoutXProperty layoutX}, {@link #layoutYProperty layoutY},
+     * {@link #translateXProperty translateX}, and {@link #translateYProperty translateY}
+     * variables are not included in the layoutBounds.
      * This is important because layout code must first determine the current
-     * size and location of the node (using layoutBounds) and then set
+     * size and location of the node (using {@code layoutBounds}) and then set
      * {@code layoutX} and {@code layoutY} to adjust the translation of the
      * node so that it will have the desired layout position.
      * <p>
      * Because the computation of layoutBounds is often tied to a node's
      * geometric variables, it is an error to bind any such variables to an
      * expression that depends upon {@code layoutBounds}. For example, the
-     * x or y variables of a shape should never be bound to layoutBounds
+     * x or y variables of a shape should never be bound to {@code layoutBounds}
      * for the purpose of positioning the node.
      * <p>
-     * The layoutBounds will never be null.
+     * Note that for 3D shapes, the layout bounds is actually a rectangular box
+     * with X, Y, and Z values, although only X and Y are used in layout calculations.
+     * <p>
+     * The {@code layoutBounds} will never be null.
      *
      */
     private LazyBoundsProperty layoutBounds = new LazyBoundsProperty() {
@@ -5608,7 +5613,7 @@
     /**
      * Defines the factor by which coordinates are scaled about the center of the
      * object along the X axis of this {@code Node}. This is used to stretch or
-     * animate the node either manually or by using an animation.
+     * shrink the node either manually or by using an animation.
      * <p>
      * This scale factor is not included in {@link #layoutBoundsProperty layoutBounds} by
      * default, which makes it ideal for scaling the entire node after
@@ -5636,7 +5641,7 @@
     /**
      * Defines the factor by which coordinates are scaled about the center of the
      * object along the Y axis of this {@code Node}. This is used to stretch or
-     * animate the node either manually or by using an animation.
+     * shrink the node either manually or by using an animation.
      * <p>
      * This scale factor is not included in {@link #layoutBoundsProperty layoutBounds} by
      * default, which makes it ideal for scaling the entire node after
@@ -5664,7 +5669,7 @@
     /**
      * Defines the factor by which coordinates are scaled about the center of the
      * object along the Z axis of this {@code Node}. This is used to stretch or
-     * animate the node either manually or by using an animation.
+     * shrink the node either manually or by using an animation.
      * <p>
      * This scale factor is not included in {@link #layoutBoundsProperty layoutBounds} by
      * default, which makes it ideal for scaling the entire node after
--- a/modules/javafx.graphics/src/main/java/javafx/scene/Parent.java	Mon Jan 08 22:25:50 2018 -0800
+++ b/modules/javafx.graphics/src/main/java/javafx/scene/Parent.java	Wed Jan 10 16:59:50 2018 -0800
@@ -1829,6 +1829,9 @@
                                                         maxX, maxY, maxZ);
     }
 
+    /**
+     * Updates the bounds of this {@code Parent} and its children.
+     */
     @Override protected void updateBounds() {
         for (int i=0, max=children.size(); i<max; i++) {
             children.get(i).updateBounds();
--- a/modules/javafx.graphics/src/main/java/javafx/scene/image/ImageView.java	Mon Jan 08 22:25:50 2018 -0800
+++ b/modules/javafx.graphics/src/main/java/javafx/scene/image/ImageView.java	Wed Jan 10 16:59:50 2018 -0800
@@ -504,7 +504,7 @@
      * scaling to fit the image within the fitting bounding box.
      * <p>
      * If set to {@code true}, it affects the dimensions of this
-     * {@code ImageView} in the following way *
+     * {@code ImageView} in the following way
      * <ul>
      * <li>If only {@code fitWidth} is set, height is scaled to preserve ratio
      * <li>If only {@code fitHeight} is set, width is scaled to preserve ratio
@@ -513,7 +513,7 @@
      * </ul>
      *
      * If unset or set to {@code false}, it affects the dimensions of this
-     * {@code ImageView} in the following way *
+     * {@code ImageView} in the following way
      * <ul>
      * <li>If only {@code fitWidth} is set, image's view width is scaled to
      * match and height is unchanged;
--- a/modules/javafx.graphics/src/main/java/javafx/scene/layout/HBox.java	Mon Jan 08 22:25:50 2018 -0800
+++ b/modules/javafx.graphics/src/main/java/javafx/scene/layout/HBox.java	Wed Jan 10 16:59:50 2018 -0800
@@ -171,7 +171,7 @@
      * If set, the hbox will use the priority to allocate additional space if the
      * hbox is resized larger than it's preferred width.
      * If multiple hbox children have the same horizontal grow priority, then the
-     * extra space will be split evening between them.
+     * extra space will be split evenly between them.
      * If no horizontal grow priority is set on a child, the hbox will never
      * allocate it additional horizontal space if available.
      * Setting the value to null will remove the constraint.
--- a/modules/javafx.graphics/src/main/java/javafx/scene/shape/TriangleMesh.java	Mon Jan 08 22:25:50 2018 -0800
+++ b/modules/javafx.graphics/src/main/java/javafx/scene/shape/TriangleMesh.java	Wed Jan 10 16:59:50 2018 -0800
@@ -67,7 +67,7 @@
  * The term texCoord is used to indicate a single pair of 2D texture
  * coordinates (u, v) for a single vertex, while the term texCoords (plural) is used
  * to indicate sets of texture coordinates for multiple vertices.
- * Lastly, the term face is used to indicate 3 set of interleaving points,
+ * Lastly, the term face is used to indicate 3 sets of interleaving points,
  * normals (optional, depending on the associated VertexFormat)
  * and texture coordinates that together represent the geometric topology of a
  * single triangle, while the term faces (plural) is used to indicate sets of
--- a/modules/javafx.graphics/src/main/java/javafx/scene/transform/MatrixType.java	Mon Jan 08 22:25:50 2018 -0800
+++ b/modules/javafx.graphics/src/main/java/javafx/scene/transform/MatrixType.java	Wed Jan 10 16:59:50 2018 -0800
@@ -31,7 +31,7 @@
  */
 public enum MatrixType {
     /**
-     * A 2D affine transformation matrix of 2 rows and 3 columns containing
+     * A 2D affine transformation matrix of 2 rows and 3 columns. Contains
      * the following values:
      * <pre>
      * mxx, mxy, tx,
@@ -52,7 +52,7 @@
     MT_2D_3x3(3, 3),
 
     /**
-     * A 3D affine transformation matrix of 3 rows and 4 columns containing
+     * A 3D affine transformation matrix of 3 rows and 4 columns. Contains
      * the following values:
      * <pre>
      * mxx, mxy, mxz, tx,