You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@xerces.apache.org by mr...@apache.org on 2007/12/12 18:38:32 UTC

svn commit: r603681 - /xerces/java/branches/xml-schema-1.1-dev/src/org/apache/xerces/dom/DOMStringListImpl.java

Author: mrglavas
Date: Wed Dec 12 09:38:32 2007
New Revision: 603681

URL: http://svn.apache.org/viewvc?rev=603681&view=rev
Log:
Fixing a possible IndexOutOfBoundsException which was introduced by the previous commit.

Modified:
    xerces/java/branches/xml-schema-1.1-dev/src/org/apache/xerces/dom/DOMStringListImpl.java

Modified: xerces/java/branches/xml-schema-1.1-dev/src/org/apache/xerces/dom/DOMStringListImpl.java
URL: http://svn.apache.org/viewvc/xerces/java/branches/xml-schema-1.1-dev/src/org/apache/xerces/dom/DOMStringListImpl.java?rev=603681&r1=603680&r2=603681&view=diff
==============================================================================
--- xerces/java/branches/xml-schema-1.1-dev/src/org/apache/xerces/dom/DOMStringListImpl.java (original)
+++ xerces/java/branches/xml-schema-1.1-dev/src/org/apache/xerces/dom/DOMStringListImpl.java Wed Dec 12 09:38:32 2007
@@ -61,12 +61,11 @@
 	 * @see org.w3c.dom.DOMStringList#item(int)
 	 */
 	public String item(int index) {
-        try {
-            return (String) fStrings.get(index);
-        } 
-        catch (ArrayIndexOutOfBoundsException e) {
-            return null;
-        }
+	    final int length = getLength();
+	    if (index >= 0 && index < length) {
+	        return (String) fStrings.get(index);
+	    }
+	    return null;
 	}
 
 	/**
@@ -80,7 +79,7 @@
 	 * @see org.w3c.dom.DOMStringList#contains(String)
 	 */
 	public boolean contains(String param) {
-		return fStrings.contains(param) ;
+		return fStrings.contains(param);
 	}
 
     /**



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@xerces.apache.org
For additional commands, e-mail: commits-help@xerces.apache.org