zongaro 2004/11/19 07:51:27
Modified: java/src/org/apache/xpath/objects XStringForFSB.java
Log:
Patch for Jira bug 1955. The equals(Object) method contains a list of
instanceof checks, and then casts the Object argument to the tested type in
order to dispatch to a more specific equals method. In the case where the
Object argument was an XMLString, the code was actually casting "this" to
XMLString - thus testing this.equals(this), which will always be true.
Reviewd by Joanne Tong (joannet () ca ! ibm ! com).
Revision Changes Path
1.23 +2 -2
xml-xalan/java/src/org/apache/xpath/objects/XStringForFSB.java
Index: XStringForFSB.java
===================================================================
RCS file:
/home/cvs/xml-xalan/java/src/org/apache/xpath/objects/XStringForFSB.java,v
retrieving revision 1.22
retrieving revision 1.23
diff -u -r1.22 -r1.23
--- XStringForFSB.java 21 Oct 2004 15:29:10 -0000 1.22
+++ XStringForFSB.java 19 Nov 2004 15:51:27 -0000 1.23
@@ -432,7 +432,7 @@
else if (obj2 instanceof XNodeSet)
return obj2.equals(this);
else if (obj2 instanceof XStringForFSB)
- return equals((XMLString) this);
+ return equals((XMLString) obj2);
else
return equals(obj2.toString());
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]