You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pivot.apache.org by sm...@apache.org on 2010/09/07 02:38:59 UTC

svn commit: r993186 - /pivot/trunk/wtk-terra/src/org/apache/pivot/wtk/skin/terra/TerraTheme.java

Author: smartini
Date: Tue Sep  7 00:38:59 2010
New Revision: 993186

URL: http://svn.apache.org/viewvc?rev=993186&view=rev
Log:
reverted to previous version (added check range on index in color get/set)

Modified:
    pivot/trunk/wtk-terra/src/org/apache/pivot/wtk/skin/terra/TerraTheme.java

Modified: pivot/trunk/wtk-terra/src/org/apache/pivot/wtk/skin/terra/TerraTheme.java
URL: http://svn.apache.org/viewvc/pivot/trunk/wtk-terra/src/org/apache/pivot/wtk/skin/terra/TerraTheme.java?rev=993186&r1=993185&r2=993186&view=diff
==============================================================================
--- pivot/trunk/wtk-terra/src/org/apache/pivot/wtk/skin/terra/TerraTheme.java (original)
+++ pivot/trunk/wtk-terra/src/org/apache/pivot/wtk/skin/terra/TerraTheme.java Tue Sep  7 00:38:59 2010
@@ -304,10 +304,6 @@ public final class TerraTheme extends Th
      * @param index
      */
     public Color getColor(int index) {
-        if (index < 0 || index >= colors.getLength()) {
-            throw new IllegalArgumentException("Index not in range [0, " + (colors.getLength() -1) + "]");
-        }
-
         return colors.get(index);
     }
 
@@ -318,9 +314,6 @@ public final class TerraTheme extends Th
      * @param color
      */
     public void setColor(int index, Color color) {
-        if (index < 0 || index >= colors.getLength()) {
-            throw new IllegalArgumentException("Index not in range [0, " + (colors.getLength() -1) + "]");
-        }
         if (color == null) {
             throw new IllegalArgumentException();
         }