changeset 11016:5c87692d0056

8204653: Fix mistakes in FX API docs Reviewed-by: kcr
author nlisker
date Fri, 03 Aug 2018 06:38:48 -0700
parents 8451c9f32969
children 328f6b32bf0c
files modules/javafx.base/src/main/java/javafx/beans/InvalidationListener.java modules/javafx.base/src/main/java/javafx/beans/WeakInvalidationListener.java modules/javafx.base/src/main/java/javafx/beans/value/ObservableValue.java modules/javafx.graphics/src/main/java/javafx/scene/Node.java modules/javafx.graphics/src/main/java/javafx/scene/canvas/GraphicsContext.java modules/javafx.graphics/src/main/java/javafx/scene/image/PixelFormat.java modules/javafx.graphics/src/main/java/javafx/scene/layout/BorderStroke.java modules/javafx.graphics/src/main/java/javafx/scene/layout/Region.java modules/javafx.graphics/src/main/java/javafx/scene/paint/Color.java modules/javafx.graphics/src/main/java/javafx/scene/shape/Mesh.java modules/javafx.graphics/src/main/java/javafx/scene/transform/Rotate.java modules/javafx.graphics/src/main/java/javafx/scene/transform/Scale.java modules/javafx.graphics/src/main/java/javafx/scene/transform/Shear.java modules/javafx.graphics/src/main/java/javafx/scene/transform/Translate.java
diffstat 14 files changed, 113 insertions(+), 106 deletions(-) [+]
line wrap: on
line diff
--- a/modules/javafx.base/src/main/java/javafx/beans/InvalidationListener.java	Fri Aug 03 17:01:47 2018 +0530
+++ b/modules/javafx.base/src/main/java/javafx/beans/InvalidationListener.java	Fri Aug 03 06:38:48 2018 -0700
@@ -31,7 +31,7 @@
  * An {@code InvalidationListener} is notified whenever an
  * {@link Observable} becomes invalid. It can be registered and
  * unregistered with {@link Observable#addListener(InvalidationListener)}
- * respectively {@link Observable#removeListener(InvalidationListener)}
+ * respectively {@link Observable#removeListener(InvalidationListener)}.
  * <p>
  * For an in-depth explanation of invalidation events and how they differ from
  * change events, see the documentation of {@code ObservableValue}.
--- a/modules/javafx.base/src/main/java/javafx/beans/WeakInvalidationListener.java	Fri Aug 03 17:01:47 2018 +0530
+++ b/modules/javafx.base/src/main/java/javafx/beans/WeakInvalidationListener.java	Fri Aug 03 06:38:48 2018 -0700
@@ -29,18 +29,18 @@
 import javafx.beans.NamedArg;
 
 /**
- * A {@code WeakInvalidationListener} can be used, if an {@link Observable}
+ * A {@code WeakInvalidationListener} can be used if an {@link Observable}
  * should only maintain a weak reference to the listener. This helps to avoid
- * memory leaks, that can occur if observers are not unregistered from observed
+ * memory leaks that can occur if observers are not unregistered from observed
  * objects after use.
  * <p>
- * {@code WeakInvalidationListener} are created by passing in the original
+ * A {@code WeakInvalidationListener} is created by passing in the original
  * {@link InvalidationListener}. The {@code WeakInvalidationListener} should
  * then be registered to listen for changes of the observed object.
  * <p>
- * Note: You have to keep a reference to the {@code InvalidationListener}, that
- * was passed in as long as it is in use, otherwise it will be garbage collected
- * to soon.
+ * Note: You have to keep a reference to the {@code InvalidationListener} that
+ * was passed in as long as it is in use, otherwise it can be garbage collected
+ * too soon.
  *
  * @see InvalidationListener
  * @see Observable
--- a/modules/javafx.base/src/main/java/javafx/beans/value/ObservableValue.java	Fri Aug 03 17:01:47 2018 +0530
+++ b/modules/javafx.base/src/main/java/javafx/beans/value/ObservableValue.java	Fri Aug 03 06:38:48 2018 -0700
@@ -44,8 +44,8 @@
  * <p>
  * An {@code ObservableValue} generates two types of events: change events and
  * invalidation events. A change event indicates that the value has changed. An
- * invalidation event is generated, if the current value is not valid anymore.
- * This distinction becomes important, if the {@code ObservableValue} supports
+ * invalidation event is generated if the current value is not valid anymore.
+ * This distinction becomes important if the {@code ObservableValue} supports
  * lazy evaluation, because for a lazily evaluated value one does not know if an
  * invalid value really has changed until it is recomputed. For this reason,
  * generating change events requires eager evaluation while invalidation events
@@ -108,7 +108,7 @@
     void addListener(ChangeListener<? super T> listener);
 
     /**
-     * Removes the given listener from the list of listeners, that are notified
+     * Removes the given listener from the list of listeners that are notified
      * whenever the value of the {@code ObservableValue} changes.
      * <p>
      * If the given listener has not been previously registered (i.e. it was
--- a/modules/javafx.graphics/src/main/java/javafx/scene/Node.java	Fri Aug 03 17:01:47 2018 +0530
+++ b/modules/javafx.graphics/src/main/java/javafx/scene/Node.java	Fri Aug 03 06:38:48 2018 -0700
@@ -1857,11 +1857,15 @@
      * Defines how the picking computation is done for this node when
      * triggered by a {@code MouseEvent} or a {@code contains} function call.
      *
-     * If {@code pickOnBounds} is true, then picking is computed by
+     * If {@code pickOnBounds} is {@code true}, then picking is computed by
      * intersecting with the bounds of this node, else picking is computed
      * by intersecting with the geometric shape of this node.
      *
-     * @defaultValue false
+     * The default value of this property is {@code false} unless
+     * overridden by a subclass. The default value is {@code true}
+     * for {@link javafx.scene.layout.Region}.
+     *
+     * @defaultValue false; true for {@code Region}
      */
     private BooleanProperty pickOnBounds;
 
@@ -10079,4 +10083,3 @@
     }
 
 }
-
--- a/modules/javafx.graphics/src/main/java/javafx/scene/canvas/GraphicsContext.java	Fri Aug 03 17:01:47 2018 +0530
+++ b/modules/javafx.graphics/src/main/java/javafx/scene/canvas/GraphicsContext.java	Fri Aug 03 06:38:48 2018 -0700
@@ -706,7 +706,7 @@
         }
         buf.putByte(NGCanvas.PATHEND);
         // Transform needs to be updated for rendering attributes even though
-        // we have already trasnformed the points as we sent them.
+        // we have already transformed the points as we sent them.
         updateTransform();
         buf.putByte(command);
         // Now that we have changed the PG layer path, we need to mark our path dirty.
@@ -793,7 +793,7 @@
     private void reset() {
         GrowableDataBuffer buf = getBuffer();
         // Only reset if we have a significant amount of data to omit,
-        // this prevents a common occurence of "setFill(bg); fillRect();"
+        // this prevents a common occurrence of "setFill(bg); fillRect();"
         // at the start of a session from invoking a reset.
         // But, do a reset anyway if the rendering layer has been falling
         // behind because that lets the synchronization step throw out the
@@ -2609,7 +2609,7 @@
 
     /**
      * Draws an image into the given destination rectangle of the canvas. The
-     * Image is scaled to fit into the destination rectagnle.
+     * Image is scaled to fit into the destination rectangle.
      * A {@code null} image value or an image still in progress will be ignored.
      * <p>
      * This method will be affected by any of the
--- a/modules/javafx.graphics/src/main/java/javafx/scene/image/PixelFormat.java	Fri Aug 03 17:01:47 2018 +0530
+++ b/modules/javafx.graphics/src/main/java/javafx/scene/image/PixelFormat.java	Fri Aug 03 06:38:48 2018 -0700
@@ -209,7 +209,7 @@
      * <p>
      * Pixels in this format can be decoded using the following sample code:
      * <pre>{@code
-     *     int pixel = array[rowstart + x] &amp; 0xff;
+     *     int pixel = array[rowstart + x] & 0xff;
      *     int argb  = colors[pixel];
      *
      *     int alpha = ((argb >> 24) & 0xff);
@@ -237,7 +237,7 @@
      * <p>
      * Pixels in this format can be decoded using the following sample code:
      * <pre>{@code
-     *     int pixel = array[rowstart + x] &amp; 0xff;
+     *     int pixel = array[rowstart + x] & 0xff;
      *     int argb  = colors[pixel];
      *
      *     int alpha = ((argb >> 24) & 0xff);
--- a/modules/javafx.graphics/src/main/java/javafx/scene/layout/BorderStroke.java	Fri Aug 03 17:01:47 2018 +0530
+++ b/modules/javafx.graphics/src/main/java/javafx/scene/layout/BorderStroke.java	Fri Aug 03 06:38:48 2018 -0700
@@ -32,16 +32,17 @@
 import javafx.scene.shape.StrokeType;
 
 /**
- * Defines the stroke to use on a Border for styling a Region. The
- * stroke is a vector-based rendering that outlines the border area.
- * It can be inset (or outset) from the Region's edge, and the values
- * of the stroke are taken into account when computing the Region's
- * insets (for defining the content area). The stroke visuals are
- * not used when any BorderImages are in use.
+ * Defines the stroke to use on a {@link Border} for styling a {@code Region}.
+ * The stroke is a vector-based rendering that outlines the border area.
+ * It can be inset (or outset) from the {@code Region}'s edge, and the values
+ * of the stroke are taken into account when computing the {@code Region}'s
+ * insets (for defining the content area). The stroke visuals are not used when
+ * any {@link BorderImage}s are in use.
  * <p>
- * When applied to a Region with a defined shape, the border width
- * and stroking information for the {@code top} is used, while the
- * other attributes are ignored.
+ * When applied to a {@code Region} with a defined shape, the border width
+ * and stroking information for the {@code top} is used, while the other
+ * attributes are ignored.
+ *
  * @since JavaFX 8.0
  */
 public class BorderStroke {
@@ -70,7 +71,7 @@
      * Defines the fill of top side of this border.
      *
      * @return the fill of top side of this border
-     * @defaultValue black
+     * @defaultValue {@code Color.BLACK}
      */
     public final Paint getTopStroke() { return topStroke; }
     final Paint topStroke;
@@ -79,31 +80,31 @@
     // we use inherit. But first I'd like a performance analysis.
 
     /**
-     * Defines the fill of right side of this border. If {@code null} then the
-     * topFill is used.
+     * Defines the fill of right side of this border. If {@code null}, then the
+     * {@code topFill} is used.
      *
      * @return the fill of right side of this border
-     * @defaultValue null = same as topFill
+     * @defaultValue {@code null} (same as {@code topFill})
      */
     public final Paint getRightStroke() { return rightStroke; }
     final Paint rightStroke;
 
     /**
-     * Defines the fill of bottom side of this border. If {@code null} then the
-     * topFill is used.
+     * Defines the fill of bottom side of this border. If {@code null}, then the
+     * {@code topFill} is used.
      *
      * @return the fill of bottom side of this border
-     * @defaultValue null = same as topFill
+     * @defaultValue {@code null} (same as {@code topFill})
      */
     public final Paint getBottomStroke() { return bottomStroke; }
     final Paint bottomStroke;
 
     /**
-     * Defines the fill of left side of this border. If {@code null} then the
-     * rightFill is used.
+     * Defines the fill of left side of this border. If {@code null}, then the
+     * {@code rightFill} is used.
      *
      * @return the fill of left side of this border
-     * @defaultValue null = same sa rightFill
+     * @defaultValue {@code null} (same as {@code rightFill})
      */
     public final Paint getLeftStroke() { return leftStroke; }
     final Paint leftStroke;
@@ -112,55 +113,55 @@
      * Defines the style of top side of this border.
      *
      * @return the style of top side of this border
-     * @defaultValue none
+     * @defaultValue {@code BorderStrokeStyle.NONE}
      */
     public final BorderStrokeStyle getTopStyle() { return topStyle; }
     final BorderStrokeStyle topStyle;
 
     /**
-     * Defines the style of right side of this border. If {@code null} then
-     * topStyle is used;
+     * Defines the style of right side of this border. If {@code null}, then
+     * {@code topStyle} is used;
      *
      * @return the style of right side of this border
-     * @defaultValue null = same as topStyle
+     * @defaultValue {@code null} (same as {@code topStyle})
      */
     public final BorderStrokeStyle getRightStyle() { return rightStyle; }
     final BorderStrokeStyle rightStyle;
 
     /**
-     * Defines the style of bottom side of this border. If {@code null} then
-     * topStyle is used;  Use BorderStyle.NONE to set the border to
+     * Defines the style of bottom side of this border. If {@code null}, then
+     * {@code topStyle} is used;  Use {@code BorderStyle.NONE} to set the border to
      * have no border style.
      *
      * @return the style of bottom side of this border
-     * @defaultValue null = same as topStyle
+     * @defaultValue {@code null} (same as {@code topStyle})
      */
     public final BorderStrokeStyle getBottomStyle() { return bottomStyle; }
     final BorderStrokeStyle bottomStyle;
 
     /**
-     * Defines the style of left side of this border. If {@code null} then
-     * rightStyle is used. Use BorderStyle.NONE to set the border to
+     * Defines the style of left side of this border. If {@code null}, then
+     * {@code rightStyle} is used. Use {@code BorderStyle.NONE} to set the border to
      * have no border style.
      *
      * @return the style of left side of this border
-     * @defaultValue null = same as rightStyle
+     * @defaultValue {@code null} (same as {@code rightStyle})
      */
     public final BorderStrokeStyle getLeftStyle() { return leftStyle; }
     final BorderStrokeStyle leftStyle;
 
     /**
-     * Defines the thickness of each side of the BorderStroke. This will never
-     * be null, and defaults to DEFAULT_WIDTHS.
-     * @return the thickness of each side of the BorderStroke
+     * Defines the thickness of each side of the {@code BorderStroke}. This will never
+     * be {@code null}, and defaults to {@code DEFAULT_WIDTHS}.
+     * @return the thickness of each side of the {@code BorderStroke}
      */
     public final BorderWidths getWidths() { return widths; }
     final BorderWidths widths;
 
     /**
-     * Defines the insets of each side of the BorderStroke. This will never
-     * be null, and defaults to EMPTY.
-     * @return the insets of each side of the BorderStroke
+     * Defines the insets of each side of the {@code BorderStroke}. This will never
+     * be {@code null}, and defaults to {@code Insets.EMPTY}.
+     * @return the insets of each side of the {@code BorderStroke}
      */
     public final Insets getInsets() { return insets; }
     final Insets insets;
@@ -170,9 +171,9 @@
     final Insets outerEdge;
 
     /**
-     * Defines the radii for each corner of this BorderStroke. This will never
-     * be null, and defaults to CornerRadii.EMPTY.
-     * @return the radii for each corner of this BorderStroke
+     * Defines the radii for each corner of this {@code BorderStroke}. This will never
+     * be {@code null}, and defaults to {@code CornerRadii.EMPTY}.
+     * @return the radii for each corner of this {@code BorderStroke}
      */
     public final CornerRadii getRadii() { return radii; }
      /* TODO I should change CornerRadii to be 4 properties, one for each corner,
@@ -181,9 +182,9 @@
     private final CornerRadii radii;
 
     /**
-     * An uniform stroke has all (top, bottom, left, right) strokes of
-     * same color, width and style
-     * @return true if border stroke is uniform as defined above
+     * Checks if the stroke of this region is uniform. A uniform stroke has all its side
+     * strokes (top, bottom, left, right) of same color, width and style.
+     * @return {@code true} if border stroke is uniform
      */
     public final boolean isStrokeUniform() { return strokeUniform; }
     private final boolean strokeUniform;
@@ -194,12 +195,12 @@
     private final int hash;
 
     /**
-     * Creates a new BorderStroke.
+     * Creates a new {@code BorderStroke}.
      *
-     * @param stroke    The stroke to use for all sides. If null, we default to Color.BLACK.
-     * @param style     The style to use for all sides. If null, we default to BorderStrokeStyle.NONE
-     * @param radii     The radii to use. If null, we default to CornerRadii.EMPTY
-     * @param widths    The widths to use. If null, we default to DEFAULT_WIDTHS
+     * @param stroke    The stroke to use for all sides. If {@code null}, defaults to {@code Color.BLACK}.
+     * @param style     The style to use for all sides. If {@code null}, defaults to {@code BorderStrokeStyle.NONE}.
+     * @param radii     The radii to use. If {@code null}, defaults to {@code CornerRadii.EMPTY}.
+     * @param widths    The widths to use. If {@code null}, defaults to {@code DEFAULT_WIDTHS}.
      */
     public BorderStroke(@NamedArg("stroke") Paint stroke, @NamedArg("style") BorderStrokeStyle style, @NamedArg("radii") CornerRadii radii, @NamedArg("widths") BorderWidths widths) {
         // TODO: Note that we default to THIN, not to MEDIUM as the CSS spec says. So it will be
@@ -236,32 +237,34 @@
     }
 
     /**
-     * Creates a new BorderStroke.
+     * Creates a new {@code BorderStroke}.
      *
-     * @param stroke    The stroke to use for all sides. If null, we default to Color.BLACK.
-     * @param style     The style to use for all sides. If null, we default to BorderStrokeStyle.NONE
-     * @param radii     The radii to use. If null, we default to CornerRadii.EMPTY
-     * @param widths    The widths to use. If null, we default to DEFAULT_WIDTHS
+     * @param stroke    The stroke to use for all sides. If {@code null}, defaults to {@code Color.BLACK}.
+     * @param style     The style to use for all sides. If {@code null}, defaults to {@code BorderStrokeStyle.NONE}.
+     * @param radii     The radii to use. If {@code null}, defaults to {@code CornerRadii.EMPTY}.
+     * @param widths    The widths to use. If {@code null}, defaults to {@code DEFAULT_WIDTHS}.
      * @param insets    The insets indicating where to draw the border relative to the region edges.
+     *                  If {@code null}, defaults to {@code Insets.EMPTY}.
      */
     public BorderStroke(@NamedArg("stroke") Paint stroke, @NamedArg("style") BorderStrokeStyle style, @NamedArg("radii") CornerRadii radii, @NamedArg("widths") BorderWidths widths, @NamedArg("insets") Insets insets) {
         this(stroke, stroke, stroke, stroke, style, style, style, style, radii, widths, insets);
     }
 
     /**
-     * Create a new BorderStroke, specifying all construction parameters.
+     * Creates a new {@code BorderStroke}, specifying all construction parameters.
      *
-     * @param topStroke       The fill to use on the top. If null, defaults to BLACK.
-     * @param rightStroke     The fill to use on the right. If null, defaults to the same value as topStroke
-     * @param bottomStroke    The fill to use on the bottom. If null, defaults to the same value as bottomStroke
-     * @param leftStroke      The fill to use on the left. If null, defaults to the same value as rightStroke
-     * @param topStyle        The style to use on the top. If null, defaults to BorderStrokeStyle.NONE
-     * @param rightStyle      The style to use on the right. If null, defaults to the same value as topStyle
-     * @param bottomStyle     The style to use on the bottom. If null, defaults to the same value as topStyle
-     * @param leftStyle       The style to use on the left. If null, defaults to the same value as rightStyle
-     * @param radii           The radii. If null, we default to square corners by using CornerRadii.EMPTY
-     * @param widths          The thickness of each side. If null, we default to DEFAULT_WIDTHS.
-     * @param insets    The insets indicating where to draw the border relative to the region edges.
+     * @param topStroke       The fill to use on the top. If {@code null}, defaults to {@code Color.BLACK}.
+     * @param rightStroke     The fill to use on the right. If {@code null}, defaults to the same value as {@code topStroke}.
+     * @param bottomStroke    The fill to use on the bottom. If {@code null}, defaults to the same value as {@code topStroke}.
+     * @param leftStroke      The fill to use on the left. If {@code null}, defaults to the same value as {@code rightStroke}.
+     * @param topStyle        The style to use on the top. If {@code null}, defaults to {@code BorderStrokeStyle.NONE}.
+     * @param rightStyle      The style to use on the right. If {@code null}, defaults to the same value as {@code topStyle}.
+     * @param bottomStyle     The style to use on the bottom. If {@code null}, defaults to the same value as {@code topStyle}.
+     * @param leftStyle       The style to use on the left. If {@code null}, defaults to the same value as {@code rightStyle}.
+     * @param radii           The radii. If {@code null}, defaults to square corners by using {@code CornerRadii.EMPTY}.
+     * @param widths          The thickness of each side. If {@code null}, defaults to {@code DEFAULT_WIDTHS}.
+     * @param insets          The insets indicating where to draw the border relative to the region edges.
+     *                        If {@code null}, defaults to {@code Insets.EMPTY}.
      */
     public BorderStroke(
             @NamedArg("topStroke") Paint topStroke, @NamedArg("rightStroke") Paint rightStroke, @NamedArg("bottomStroke") Paint bottomStroke, @NamedArg("leftStroke") Paint leftStroke,
--- a/modules/javafx.graphics/src/main/java/javafx/scene/layout/Region.java	Fri Aug 03 17:01:47 2018 +0530
+++ b/modules/javafx.graphics/src/main/java/javafx/scene/layout/Region.java	Fri Aug 03 06:38:48 2018 -0700
@@ -114,12 +114,10 @@
  * By default a Region appears as a Rectangle. A BackgroundFill radii might cause the Rectangle to appear rounded.
  * This affects not only making the visuals look like a rounded rectangle, but it also causes the picking behavior
  * of the Region to act like a rounded rectangle, such that locations outside the corner radii are ignored. A
- * Region can be made to use any shape, however, by specifing the {@code shape} property. If a shape is specified,
+ * Region can be made to use any shape, however, by specifying the {@code shape} property. If a shape is specified,
  * then all BackgroundFills, BackgroundImages, and BorderStrokes will be applied to the shape. BorderImages are
  * not used for Regions which have a shape specified.
  * <p>
- * A Region with a shape
- * <p>
  * Although the layout bounds of a Region are not influenced by any Border or Background, the content area
  * insets and the picking area of the Region are. The {@code insets} of the Region define the distance
  * between the edge of the layout bounds and the edge of the content area. For example, if the Region
@@ -571,7 +569,7 @@
     };
 
     {
-        // To initialize the class helper at the begining each constructor of this class
+        // To initialize the class helper at the beginning each constructor of this class
         RegionHelper.initHelper(this);
     }
 
--- a/modules/javafx.graphics/src/main/java/javafx/scene/paint/Color.java	Fri Aug 03 17:01:47 2018 +0530
+++ b/modules/javafx.graphics/src/main/java/javafx/scene/paint/Color.java	Fri Aug 03 06:38:48 2018 -0700
@@ -715,7 +715,7 @@
      * @param hueShift the hue shift
      * @param saturationFactor the saturation factor
      * @param brightnessFactor the brightness factor
-     * @param opacityFactor the brightness factor
+     * @param opacityFactor the opacity factor
      * @return a {@code Color} based based on this {@code Color} with hue,
      * saturation, brightness and opacity values altered.
      */
--- a/modules/javafx.graphics/src/main/java/javafx/scene/shape/Mesh.java	Fri Aug 03 17:01:47 2018 +0530
+++ b/modules/javafx.graphics/src/main/java/javafx/scene/shape/Mesh.java	Fri Aug 03 06:38:48 2018 -0700
@@ -67,6 +67,9 @@
         });
     }
 
+    /**
+     * A constructor that is called by any {@code Mesh} implementation.
+     */
     protected Mesh() {
         if (!Platform.isSupported(ConditionalFeature.SCENE3D)) {
             String logname = Mesh.class.getName();
--- a/modules/javafx.graphics/src/main/java/javafx/scene/transform/Rotate.java	Fri Aug 03 17:01:47 2018 +0530
+++ b/modules/javafx.graphics/src/main/java/javafx/scene/transform/Rotate.java	Fri Aug 03 06:38:48 2018 -0700
@@ -44,14 +44,12 @@
  * intermediate origin is restored to the coordinates of the original
  * anchor point (S3).
  * <p>
- * For example, the matrix representing the returned transform of
- *    new Rotate (theta, x, y, z) around the Z-axis
- *
- * is :
+ * The matrix representing the rotation transformation around an axis {@code (x,y,z)}
+ * by an angle {@code t} is as follows:
  * <pre>
- *              [   cos(theta)    -sin(theta)   0    x-x*cos+y*sin  ]
- *              [   sin(theta)     cos(theta)   0    y-x*sin-y*cos  ]
- *              [      0               0        1          z        ]
+ *              [   cos(t)   -sin(t)   0   x-x*cos(t)+y*sin(t)   ]
+ *              [   sin(t)    cos(t)   0   y-x*sin(t)-y*cos(t)   ]
+ *              [     0         0      1           z             ]
  * </pre>
  * <p>
  * For example, to rotate a text 30 degrees around the Z-axis at
--- a/modules/javafx.graphics/src/main/java/javafx/scene/transform/Scale.java	Fri Aug 03 17:01:47 2018 +0530
+++ b/modules/javafx.graphics/src/main/java/javafx/scene/transform/Scale.java	Fri Aug 03 06:38:48 2018 -0700
@@ -37,11 +37,12 @@
 /**
  * This class represents an {@code Affine} object that scales coordinates
  * by the specified factors. The matrix representing the scaling transformation
- * is as follows:
+ * around a pivot point {@code (pivotX, pivotY, pivotZ)} with scaling factors
+ * {@code x}, {@code y} and {@code z} is as follows:
  * <pre>
- *              [ x   0   0   (1-x)*pivotX ]
- *              [ 0   y   0   (1-y)*pivotY ]
- *              [ 0   0   z   (1-z)*pivotZ ]
+ *              [   x   0   0   (1-x)*pivotX   ]
+ *              [   0   y   0   (1-y)*pivotY   ]
+ *              [   0   0   z   (1-z)*pivotZ   ]
  * </pre>
  * @since JavaFX 2.0
  */
--- a/modules/javafx.graphics/src/main/java/javafx/scene/transform/Shear.java	Fri Aug 03 17:01:47 2018 +0530
+++ b/modules/javafx.graphics/src/main/java/javafx/scene/transform/Shear.java	Fri Aug 03 06:38:48 2018 -0700
@@ -36,12 +36,13 @@
 
 /**
  * This class represents an {@code Affine} object that shears coordinates
- * by the specified multipliers. The matrix representing the shearing
- * transformation is as follows:
+ * by the specified multipliers. The matrix representing the shearing transformation
+ * around a pivot point {@code (pivotX, pivotY)} with multiplication factors {@code x}
+ * and {@code y} is as follows:
  * <pre>
- *      [   1   x   0   -x*pivotY ]
- *      [   y   1   0   -y*pivotX ]
- *      [   0   0   1   0   ]
+ *              [   1   x   0   -x*pivotY   ]
+ *              [   y   1   0   -y*pivotX   ]
+ *              [   0   0   1       0       ]
  * </pre>
  *
  * <p>
--- a/modules/javafx.graphics/src/main/java/javafx/scene/transform/Translate.java	Fri Aug 03 17:01:47 2018 +0530
+++ b/modules/javafx.graphics/src/main/java/javafx/scene/transform/Translate.java	Fri Aug 03 06:38:48 2018 -0700
@@ -37,11 +37,11 @@
 /**
  * This class represents an {@code Affine} object that translates coordinates
  * by the specified factors. The matrix representing the translating
- * transformation is as follows:
+ * transformation by distances {@code x}, {@code y} and {@code z} is as follows:
  * <pre>
- *              [   1    0    0    x  ]
- *              [   0    1    0    y  ]
- *              [   0    0    1    z  ]
+ *              [   1   0   0   x   ]
+ *              [   0   1   0   y   ]
+ *              [   0   0   1   z   ]
  * </pre>
  * @since JavaFX 2.0
  */