You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@xalan.apache.org by zo...@apache.org on 2002/08/24 13:33:49 UTC

cvs commit: xml-xalan/java/src/org/apache/xml/dtm/ref/sax2dtm SAX2DTM.java

zongaro     2002/08/24 04:33:49

  Modified:    java/src/org/apache/xml/dtm/ref/sax2dtm Tag: XSLTC_DTM
                        SAX2DTM.java
  Log:
  Removed bit of code in startElement that was changing the local name and QName
  of a namespace node to be equal to the prefix being declared.  That was causing
  getNodeName to return the value that getNodeNameX should be used to retrieve.
  
  Revision  Changes    Path
  No                   revision
  
  
  No                   revision
  
  
  1.28.2.4  +0 -1      xml-xalan/java/src/org/apache/xml/dtm/ref/sax2dtm/SAX2DTM.java
  
  Index: SAX2DTM.java
  ===================================================================
  RCS file: /home/cvs/xml-xalan/java/src/org/apache/xml/dtm/ref/sax2dtm/SAX2DTM.java,v
  retrieving revision 1.28.2.3
  retrieving revision 1.28.2.4
  diff -u -r1.28.2.3 -r1.28.2.4
  --- SAX2DTM.java	29 Jul 2002 00:01:31 -0000	1.28.2.3
  +++ SAX2DTM.java	24 Aug 2002 11:33:49 -0000	1.28.2.4
  @@ -1876,7 +1876,6 @@
           if (declAlreadyDeclared(prefix))
             continue;  // go to the next attribute.
   
  -        attrLocalName = attrQName = prefix;
           nodeType = DTM.NAMESPACE_NODE;
         }
         else
  
  
  

---------------------------------------------------------------------
To unsubscribe, e-mail: xalan-cvs-unsubscribe@xml.apache.org
For additional commands, e-mail: xalan-cvs-help@xml.apache.org


Re: cvs commit: xml-xalan/java/src/org/apache/xml/dtm/ref/sax2dtm SAX2DTM.java

Posted by Joseph Kesselman <ke...@us.ibm.com>.
Be careful here.  There was a deliberate decision -- which may be open to 
debate, admittedly -- that namespace nodes in the DTM are NOT simply 
attribute nodes, and do not store their names in the same way attributes 
do. 

It's certainly possible that this difference has introduced some confusion 
in the code -- but we need to be very careful about deciding whether the 
bug is in model construction or model retrieval.

______________________________________
Joe Kesselman  / IBM Research