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:14 UTC
svn commit: r603680 -
/xerces/java/trunk/src/org/apache/xerces/dom/DOMStringListImpl.java
Author: mrglavas
Date: Wed Dec 12 09:38:13 2007
New Revision: 603680
URL: http://svn.apache.org/viewvc?rev=603680&view=rev
Log:
Fixing a possible IndexOutOfBoundsException which was introduced by the previous commit.
Modified:
xerces/java/trunk/src/org/apache/xerces/dom/DOMStringListImpl.java
Modified: xerces/java/trunk/src/org/apache/xerces/dom/DOMStringListImpl.java
URL: http://svn.apache.org/viewvc/xerces/java/trunk/src/org/apache/xerces/dom/DOMStringListImpl.java?rev=603680&r1=603679&r2=603680&view=diff
==============================================================================
--- xerces/java/trunk/src/org/apache/xerces/dom/DOMStringListImpl.java (original)
+++ xerces/java/trunk/src/org/apache/xerces/dom/DOMStringListImpl.java Wed Dec 12 09:38:13 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