You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@xalan.apache.org by cu...@locus.apache.org on 2000/07/19 20:14:26 UTC

cvs commit: xml-xalan/src/org/apache/xalan/xpath/dtm DTMLiaison.java

curcuru     00/07/19 11:14:26

  Modified:    src/org/apache/xalan/xpath/dtm DTMLiaison.java
  Log:
  remove unneeded else clause that did dynamic validation
  PR:DMAN4LSRDL
  Submitted by:<md...@us.britannica.com>
  
  Revision  Changes    Path
  1.17      +5 -5      xml-xalan/src/org/apache/xalan/xpath/dtm/DTMLiaison.java
  
  Index: DTMLiaison.java
  ===================================================================
  RCS file: /home/cvs/xml-xalan/src/org/apache/xalan/xpath/dtm/DTMLiaison.java,v
  retrieving revision 1.16
  retrieving revision 1.17
  diff -u -r1.16 -r1.17
  --- DTMLiaison.java	2000/05/26 05:50:21	1.16
  +++ DTMLiaison.java	2000/07/19 18:14:25	1.17
  @@ -225,11 +225,11 @@
   
           if(getUseValidation())
             parser.setFeature("http://xml.org/sax/features/validation", true);
  -        else
  -        {
  -          // This doesn't seem to work, but I'm a setting it anyhow.
  -          parser.setFeature("http://apache.org/xml/features/validation/dynamic", true);
  -        }
  +
  +        // PR:DMAN4LSRDL Submitted by:<md...@us.britannica.com> remove unneeded else clause
  +        // "http://xml.org/sax/features/validation/dynamic" is and should be false
  +        // by default.  It incorrectly decides to validate documents that declare
  +        // internal entities. -<md...@us.britannica.com>
   
           if(m_doThreading)
           {