You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@santuario.apache.org by mu...@apache.org on 2005/12/19 21:34:08 UTC
svn commit: r357786 -
/xml/security/branches/jsr105_0_16/src/javax/xml/crypto/dsig/spec/XPathFilter2ParameterSpec.java
Author: mullan
Date: Mon Dec 19 12:34:02 2005
New Revision: 357786
URL: http://svn.apache.org/viewcvs?rev=357786&view=rev
Log:
Avoid calling size() method inside loop.
Modified:
xml/security/branches/jsr105_0_16/src/javax/xml/crypto/dsig/spec/XPathFilter2ParameterSpec.java
Modified: xml/security/branches/jsr105_0_16/src/javax/xml/crypto/dsig/spec/XPathFilter2ParameterSpec.java
URL: http://svn.apache.org/viewcvs/xml/security/branches/jsr105_0_16/src/javax/xml/crypto/dsig/spec/XPathFilter2ParameterSpec.java?rev=357786&r1=357785&r2=357786&view=diff
==============================================================================
--- xml/security/branches/jsr105_0_16/src/javax/xml/crypto/dsig/spec/XPathFilter2ParameterSpec.java (original)
+++ xml/security/branches/jsr105_0_16/src/javax/xml/crypto/dsig/spec/XPathFilter2ParameterSpec.java Mon Dec 19 12:34:02 2005
@@ -61,7 +61,8 @@
if (xPathListCopy.isEmpty()) {
throw new IllegalArgumentException("xPathList cannot be empty");
}
- for (int i = 0; i < xPathListCopy.size(); i++) {
+ int size = xPathListCopy.size();
+ for (int i = 0; i < size; i++) {
if (!(xPathListCopy.get(i) instanceof XPathType)) {
throw new ClassCastException
("xPathList["+i+"] is not a valid type");