changeset 7594:8296c327320b

RT-37968: [Spinner] Review LocalDate and LocalTime spinner value factories
author jgiles
date Wed, 30 Jul 2014 16:21:31 +1200
parents dd2faaee52c9
children fd925446754b
files modules/controls/src/main/java/javafx/scene/control/Spinner.java modules/controls/src/main/java/javafx/scene/control/SpinnerValueFactory.java
diffstat 2 files changed, 6 insertions(+), 6 deletions(-) [+]
line wrap: on
line diff
--- a/modules/controls/src/main/java/javafx/scene/control/Spinner.java	Wed Jul 30 16:20:08 2014 +1200
+++ b/modules/controls/src/main/java/javafx/scene/control/Spinner.java	Wed Jul 30 16:21:31 2014 +1200
@@ -255,7 +255,7 @@
      *                     be within the bounds of the min and max arguments, or
      *                     else the min value will be used.
      */
-    public Spinner(@NamedArg("min") LocalDate min,
+    Spinner(@NamedArg("min") LocalDate min,
                    @NamedArg("max") LocalDate max,
                    @NamedArg("initialValue") LocalDate initialValue) {
         // This only works if the Spinner is of type LocalDate
@@ -277,7 +277,7 @@
      * @param amountToStepBy The amount to increment or decrement by, per step.
      * @param temporalUnit The size of each step (e.g. day, week, month, year, etc).
      */
-    public Spinner(@NamedArg("min") LocalDate min,
+    Spinner(@NamedArg("min") LocalDate min,
                    @NamedArg("max") LocalDate max,
                    @NamedArg("initialValue") LocalDate initialValue,
                    @NamedArg("amountToStepBy") long amountToStepBy,
@@ -299,7 +299,7 @@
      *                     be within the bounds of the min and max arguments, or
      *                     else the min value will be used.
      */
-    public Spinner(@NamedArg("min") LocalTime min,
+    Spinner(@NamedArg("min") LocalTime min,
                    @NamedArg("max") LocalTime max,
                    @NamedArg("initialValue") LocalTime initialValue) {
         // This only works if the Spinner is of type LocalTime
@@ -321,7 +321,7 @@
      * @param amountToStepBy The amount to increment or decrement by, per step.
      * @param temporalUnit The size of each step (e.g. hour, minute, second, etc).
      */
-    public Spinner(@NamedArg("min") LocalTime min,
+    Spinner(@NamedArg("min") LocalTime min,
                    @NamedArg("max") LocalTime max,
                    @NamedArg("initialValue") LocalTime initialValue,
                    @NamedArg("amountToStepBy") long amountToStepBy,
--- a/modules/controls/src/main/java/javafx/scene/control/SpinnerValueFactory.java	Wed Jul 30 16:20:08 2014 +1200
+++ b/modules/controls/src/main/java/javafx/scene/control/SpinnerValueFactory.java	Wed Jul 30 16:21:31 2014 +1200
@@ -870,7 +870,7 @@
      *     }
      * });</pre>
      */
-    public static class LocalDateSpinnerValueFactory extends SpinnerValueFactory<LocalDate> {
+    static class LocalDateSpinnerValueFactory extends SpinnerValueFactory<LocalDate> {
 
         /**
          * Creates a new instance of the LocalDateSpinnerValueFactory, using the
@@ -1125,7 +1125,7 @@
      *     }
      * });</pre>
      */
-    public static class LocalTimeSpinnerValueFactory extends SpinnerValueFactory<LocalTime> {
+    static class LocalTimeSpinnerValueFactory extends SpinnerValueFactory<LocalTime> {
 
         /**
          * Creates a new instance of the LocalTimepinnerValueFactory, using the