"[EMAIL PROTECTED]" wrote : anonymous wrote : 
  |   | -snip-
  |   | Correct me if I am wrong, but it looks like the current implementation 
doesn't use a threadlocal implementation, but instead just spawns multiple 
document instances. It looks like the Axis implementation attempts to solve the 
problem I mentioned by maintaining a back reference to the SOAPPart. By 
glancing through the code it does look like 4.0 could suffer from the problem 
of assigning one document element to another document.
  |   | 
  |   | The design method I mention could still work by importing nodes if they 
are from a different document (this is what the RI appears to do).
  |   | 
  | 
  | Alternatively we could do the same the thing that the axis implementation 
does, and hold a reference to the SOAPPart in every Node, and then when 
getOwnerDocument is called we return the SOAPPart. The SOAPPart implementation 
would then always return the envelope as the owner. Then the fact that all 
nodes are allocated via a threadlocal document doesn't matter.  It is probably 
better implemented this way anyways because returning the actual DOM document 
can be problematic since each the NodeImpl is not actually stored in the 
document (its aggregate org.w3c.Node) is, which could lead to confusion.
  | 
  | What do you think?
  | 
  | -Jason 
  | 
  | 
  | 

View the original post : 
http://www.jboss.org/index.html?module=bb&op=viewtopic&p=3875298#3875298

Reply to the post : 
http://www.jboss.org/index.html?module=bb&op=posting&mode=reply&p=3875298


-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now.
http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click
_______________________________________________
JBoss-Development mailing list
JBoss-Development@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jboss-development

Reply via email to