curcuru     02/05/03 14:56:53

  Modified:    test/java/src/org/apache/qetest XMLFileLogger.java
  Log:
  Arrrg - hashitems also should really be escapeString'd, since
  we're never expecting (in current usage) element content there,
  and some tests might have oddly encoded chars as values
  
  Revision  Changes    Path
  1.10      +3 -3      
xml-xalan/test/java/src/org/apache/qetest/XMLFileLogger.java
  
  Index: XMLFileLogger.java
  ===================================================================
  RCS file: 
/home/cvs/xml-xalan/test/java/src/org/apache/qetest/XMLFileLogger.java,v
  retrieving revision 1.9
  retrieving revision 1.10
  diff -u -r1.9 -r1.10
  --- XMLFileLogger.java        2 May 2002 19:42:19 -0000       1.9
  +++ XMLFileLogger.java        3 May 2002 21:56:53 -0000       1.10
  @@ -77,7 +77,7 @@
    * Logger that saves output to a simple XML-format file.
    * @todo improve escapeString so it's more rigorous about escaping
    * @author [EMAIL PROTECTED]
  - * @version $Id: XMLFileLogger.java,v 1.9 2002/05/02 19:42:19 curcuru Exp $
  + * @version $Id: XMLFileLogger.java,v 1.10 2002/05/03 21:56:53 curcuru Exp $
    */
   public class XMLFileLogger implements Logger
   {
  @@ -754,9 +754,9 @@
                       Object key = enum.nextElement();
   
                       // Ensure we'll have clean output by pre-fetching value 
before outputting anything
  -                    String value = hash.get(key).toString();
  +                    String value = escapeString(hash.get(key).toString());
   
  -                    reportPrinter.print(HASHITEM_HDR + key.toString()
  +                    reportPrinter.print(HASHITEM_HDR + 
escapeString(key.toString())
                                           + "\">");
                       reportPrinter.print(value);
                       reportPrinter.println("</" + ELEM_HASHITEM + ">");
  
  
  

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to