sboag       00/12/20 08:34:16

  Modified:    java/src/org/apache/xalan/transformer ResultTreeHandler.java
  Log:
  In addAttribute, don't do ensurePrefixIsDeclared  for the xmlns
  name itself.
  Uncomment code that checks if the prefix is declared already
  before adding the attribute.
  This fixes bug with duplicate xmlns attributes reported
  by "Jarno Elovirta" <[EMAIL PROTECTED]>
  on 12/20/2000 02:35 AM.  May cause fairly minor performance
  regression.
  
  Revision  Changes    Path
  1.29      +4 -3      
xml-xalan/java/src/org/apache/xalan/transformer/ResultTreeHandler.java
  
  Index: ResultTreeHandler.java
  ===================================================================
  RCS file: 
/home/cvs/xml-xalan/java/src/org/apache/xalan/transformer/ResultTreeHandler.java,v
  retrieving revision 1.28
  retrieving revision 1.29
  diff -u -r1.28 -r1.29
  --- ResultTreeHandler.java    2000/12/13 17:23:01     1.28
  +++ ResultTreeHandler.java    2000/12/20 16:34:16     1.29
  @@ -1137,7 +1137,8 @@
   
       try
       {
  -      ensurePrefixIsDeclared(uri, rawName);
  +      if(!rawName.equals("xmlns")) // don't handle xmlns default namespace.
  +        ensurePrefixIsDeclared(uri, rawName);
       }
       catch(org.xml.sax.SAXException se)
       {
  @@ -1147,8 +1148,8 @@
       if (DEBUG)
         System.out.println("Adding attr: " + localName + ", " + uri);
   
  -    // if(!isDefinedNSDecl(rawName, value))
  -    qe.addAttribute(uri, localName, rawName, type, value);
  +    if(!isDefinedNSDecl(rawName, value))
  +      qe.addAttribute(uri, localName, rawName, type, value);
     }
   
     /**
  
  
  

Reply via email to