svn commit: r1820334 - in /pivot/trunk/wtk/src/org/apache/pivot/wtk/content: CalendarDateSpinnerData.java TableViewDateCellRenderer.java

[rwhitcomb]

Author: rwhitcomb
Date: Fri Jan  5 17:42:10 2018
New Revision: 1820334

URL: http://svn.apache.org/viewvc?rev=1820334&view=rev
Log:
PIVOT-1012: Update several classes to do their parameter validation
using the Utils methods.

Modified:
    
pivot/trunk/wtk/src/org/apache/pivot/wtk/content/CalendarDateSpinnerData.java
    
pivot/trunk/wtk/src/org/apache/pivot/wtk/content/TableViewDateCellRenderer.java

Modified: 
pivot/trunk/wtk/src/org/apache/pivot/wtk/content/CalendarDateSpinnerData.java
URL: 
http://svn.apache.org/viewvc/pivot/trunk/wtk/src/org/apache/pivot/wtk/content/CalendarDateSpinnerData.java?rev=1820334&r1=1820333&r2=1820334&view=diff
==============================================================================
--- 
pivot/trunk/wtk/src/org/apache/pivot/wtk/content/CalendarDateSpinnerData.java 
(original)
+++ 
pivot/trunk/wtk/src/org/apache/pivot/wtk/content/CalendarDateSpinnerData.java 
Fri Jan  5 17:42:10 2018
@@ -28,6 +28,7 @@ import org.apache.pivot.collections.List
 import org.apache.pivot.collections.Sequence;
 import org.apache.pivot.util.CalendarDate;
 import org.apache.pivot.util.ListenerList;
+import org.apache.pivot.util.Utils;
 
 /**
  * Spinner data model that presents a bounded list of calendar dates. <p> This
@@ -89,13 +90,8 @@ public class CalendarDateSpinnerData imp
      * @param upperBound The latest date to include in this spinner data.
      */
     public CalendarDateSpinnerData(CalendarDate lowerBound, CalendarDate 
upperBound) {
-        if (lowerBound == null) {
-            throw new IllegalArgumentException("lowerBound is null.");
-        }
-
-        if (upperBound == null) {
-            throw new IllegalArgumentException("upperBound is null.");
-        }
+        Utils.checkNull(lowerBound, "lowerBound");
+        Utils.checkNull(upperBound, "upperBound");
 
         if (lowerBound.compareTo(upperBound) > 0) {
             throw new IllegalArgumentException("lowerBound is after 
upperBound.");

Modified: 
pivot/trunk/wtk/src/org/apache/pivot/wtk/content/TableViewDateCellRenderer.java
URL: 
http://svn.apache.org/viewvc/pivot/trunk/wtk/src/org/apache/pivot/wtk/content/TableViewDateCellRenderer.java?rev=1820334&r1=1820333&r2=1820334&view=diff
==============================================================================
--- 
pivot/trunk/wtk/src/org/apache/pivot/wtk/content/TableViewDateCellRenderer.java 
(original)
+++ 
pivot/trunk/wtk/src/org/apache/pivot/wtk/content/TableViewDateCellRenderer.java 
Fri Jan  5 17:42:10 2018
@@ -23,6 +23,7 @@ import java.util.Date;
 
 import org.apache.pivot.json.JSON;
 import org.apache.pivot.util.CalendarDate;
+import org.apache.pivot.util.Utils;
 
 /**
  * Default renderer for table view cells that contain date data. Renders cell
@@ -36,13 +37,16 @@ public class TableViewDateCellRenderer e
     }
 
     public void setDateFormat(DateFormat dateFormat) {
-        if (dateFormat == null) {
-            throw new IllegalArgumentException("dateFormat is null.");
-        }
+        Utils.checkNull(dateFormat, "dateFormat");
 
         this.dateFormat = dateFormat;
     }
 
+    /**
+     * Sets the date format to the given pattern string.
+     * @param dateFormat A pattern string for use with {@link 
SimpleDateFormat}.
+     * @see #setDateFormat(DateFormat)
+     */
     public void setDateFormat(String dateFormat) {
         setDateFormat(new SimpleDateFormat(dateFormat));
     }