Hi dims,
If there's a test depending on the ordering of the attributes staying
intact, is the test really appropriate? A parser can report the
attributes in any order it wants. Switching to LinkedHashMap is going to
add more overhead (not a lot, but every bit hurts).
- Dennis
[EMAIL PROTECTED] wrote:
Author: dims
Date: Tue Jul 11 07:02:51 2006
New Revision: 420851
URL: http://svn.apache.org/viewvc?rev=420851&view=rev
Log:
prevent org.apache.axis2.addressing.EndpointReferenceTypeTest from failing in
JDK 1.6.0-beta2 - basically by keeping the ordering of the attributes intact
Modified:
webservices/commons/trunk/modules/axiom/src/org/apache/axiom/om/impl/llom/OMElementImpl.java
Modified:
webservices/commons/trunk/modules/axiom/src/org/apache/axiom/om/impl/llom/OMElementImpl.java
URL:
http://svn.apache.org/viewvc/webservices/commons/trunk/modules/axiom/src/org/apache/axiom/om/impl/llom/OMElementImpl.java?rev=420851&r1=420850&r2=420851&view=diff
==============================================================================
---
webservices/commons/trunk/modules/axiom/src/org/apache/axiom/om/impl/llom/OMElementImpl.java
(original)
+++
webservices/commons/trunk/modules/axiom/src/org/apache/axiom/om/impl/llom/OMElementImpl.java
Tue Jul 11 07:02:51 2006
@@ -45,6 +45,7 @@
import java.io.ByteArrayOutputStream;
import java.util.HashMap;
import java.util.Iterator;
+import java.util.LinkedHashMap;
/**
* Class OMElementImpl
@@ -487,7 +488,7 @@
*/
public OMAttribute addAttribute(OMAttribute attr) {
if (attributes == null) {
- this.attributes = new HashMap(5);
+ this.attributes = new LinkedHashMap(5);
}
OMNamespace namespace = attr.getNamespace();
if (namespace != null && this.findNamespace(namespace.getName(),
namespace.getPrefix()) == null)
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
