]> source.dussan.org Git - poi.git/commitdiff
cleanup sxxsf temp files after tests, remove non-ascii characters from source code...
authorYegor Kozlov <yegor@apache.org>
Sat, 25 Jun 2011 10:56:46 +0000 (10:56 +0000)
committerYegor Kozlov <yegor@apache.org>
Sat, 25 Jun 2011 10:56:46 +0000 (10:56 +0000)
git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1139518 13f79535-47bb-0310-9956-ffa450edef68

16 files changed:
build.xml
src/java/org/apache/poi/hssf/record/common/UnicodeString.java
src/java/org/apache/poi/hssf/usermodel/HSSFSheet.java
src/java/org/apache/poi/poifs/storage/BlockAllocationTableReader.java
src/java/org/apache/poi/ss/usermodel/Sheet.java
src/ooxml/java/org/apache/poi/openxml4j/opc/PackagingURIHelper.java
src/ooxml/java/org/apache/poi/xssf/streaming/SXSSFSheet.java
src/ooxml/java/org/apache/poi/xssf/streaming/SXSSFWorkbook.java
src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFSheet.java
src/ooxml/testcases/org/apache/poi/TestPOIXMLDocument.java
src/scratchpad/src/org/apache/poi/hwpf/model/FontTable.java
src/scratchpad/src/org/apache/poi/hwpf/model/PropertyNode.java
src/scratchpad/src/org/apache/poi/hwpf/model/SectionTable.java
src/scratchpad/testcases/org/apache/poi/hdgf/extractor/TestVisioExtractor.java
src/testcases/org/apache/poi/hssf/usermodel/TestHSSFDataFormat.java
src/testcases/org/apache/poi/poifs/filesystem/TestEmptyDocument.java

index 3acca123a8542516042b2c7a124ce4af6f480ad4..182b88cd7794546059a7c1dad67d63538c059dda 100644 (file)
--- a/build.xml
+++ b/build.xml
@@ -66,6 +66,12 @@ under the License.
     <property name="jdk.version.class" value="1.5" description="JDK version of generated class files"/>
     <property name="compile.debug" value="true"/>
 
+    <!-- 
+        Logging is suppressed by default. 
+        To redirect log output to console, run ant with -Dorg.apache.poi.util.POILogger=org.apache.poi.util.SystemOutLogger  
+    -->
+    <property name="org.apache.poi.util.POILogger" value="org.apache.poi.util.NullLogger"/>
+
     <!-- issue warnings if source code contains unmappable characters for encoding ASCII  -->
     <property name="java.source.encoding" value="ASCII"/>
 
@@ -165,6 +171,7 @@ under the License.
     <propertyset id="junit.properties">
         <propertyref name="POI.testdata.path"/>
         <propertyref name="java.awt.headless"/>
+        <propertyref name="org.apache.poi.util.POILogger"/>
     </propertyset>
 
     <path id="main.classpath">
index fe16f0a355c63b3d3755ba47c2b1c7f9ff4984b3..bd9ecbac7b443a6097b77223826d017c21f8edfc 100644 (file)
@@ -25,11 +25,8 @@ import java.util.List;
 import org.apache.poi.hssf.record.cont.ContinuableRecordInput;
 import org.apache.poi.hssf.record.RecordInputStream;
 import org.apache.poi.hssf.record.cont.ContinuableRecordOutput;
-import org.apache.poi.util.BitField;
-import org.apache.poi.util.BitFieldFactory;
-import org.apache.poi.util.LittleEndianInput;
-import org.apache.poi.util.LittleEndianOutput;
-import org.apache.poi.util.StringUtil;
+import org.apache.poi.poifs.dev.POIFSLister;
+import org.apache.poi.util.*;
 
 /**
  * Title: Unicode String<p/>
@@ -40,6 +37,8 @@ import org.apache.poi.util.StringUtil;
  * REFERENCE:  PG 951 Excel Binary File Format (.xls) Structure Specification v20091214 
  */
 public class UnicodeString implements Comparable<UnicodeString> { // TODO - make this final when the compatibility version is removed
+    private static POILogger _logger = POILogFactory.getLogger(UnicodeString.class);
+
     private short             field_1_charCount;
     private byte              field_2_optionflags;
     private String            field_3_string;
@@ -138,7 +137,7 @@ public class UnicodeString implements Comparable<UnicodeString> { // TODO - make
           
           // Spot corrupt records
           if(reserved != 1) {
-             System.err.println("Warning - ExtRst was has wrong magic marker, expecting 1 but found " + reserved + " - ignoring");
+             _logger.log(POILogger.WARN, "Warning - ExtRst has wrong magic marker, expecting 1 but found " + reserved + " - ignoring");
              // Grab all the remaining data, and ignore it
              for(int i=0; i<expectedLength-2; i++) {
                 in.readByte();
@@ -438,7 +437,7 @@ public class UnicodeString implements Comparable<UnicodeString> { // TODO - make
         if (isExtendedText() && (extensionLength > 0)) {
           field_5_ext_rst = new ExtRst(new ContinuableRecordInput(in), extensionLength);
           if(field_5_ext_rst.getDataSize()+4 != extensionLength) {
-             System.err.println("ExtRst was supposed to be " + extensionLength + " bytes long, but seems to actually be " + (field_5_ext_rst.getDataSize()+4));
+             _logger.log(POILogger.WARN, "ExtRst was supposed to be " + extensionLength + " bytes long, but seems to actually be " + (field_5_ext_rst.getDataSize() + 4));
           }
         }
     }
index 59ef00cfe416dd5df56829d2bca4915392b43e25..ed8cc4e59fa5fc779b0da078885f87391879aeb4 100644 (file)
@@ -438,7 +438,7 @@ public final class HSSFSheet implements org.apache.poi.ss.usermodel.Sheet {
      *
      * <p>
      * Character width is defined as the maximum digit width
-     * of the numbers <code>0, 1, 2,  9</code> as rendered
+     * of the numbers <code>0, 1, 2, ... 9</code> as rendered
      * using the default font (first font in the workbook).
      * <br/>
      * Unless you are using a very special font, the default character is '0' (zero),
index 0d1b86dd4fc4af497e948d4159af90c6a1911984..5de62855755285de712efc454ef7b715098e41b9 100644 (file)
@@ -23,9 +23,7 @@ import java.util.*;
 
 import org.apache.poi.poifs.common.POIFSBigBlockSize;
 import org.apache.poi.poifs.common.POIFSConstants;
-import org.apache.poi.util.IntList;
-import org.apache.poi.util.LittleEndian;
-import org.apache.poi.util.LittleEndianConsts;
+import org.apache.poi.util.*;
 
 /**
  * This class manages and creates the Block Allocation Table, which is
@@ -43,7 +41,8 @@ import org.apache.poi.util.LittleEndianConsts;
  * @author Marc Johnson (mjohnson at apache dot org)
  */
 public final class BlockAllocationTableReader {
-    
+    private static final POILogger _logger = POILogFactory.getLogger(BlockAllocationTableReader.class);
+
     /**
      * Maximum number size (in blocks) of the allocation table as supported by
      * POI.<br/>
@@ -227,12 +226,12 @@ public final class BlockAllocationTableReader {
             } catch(IOException e) {
                 if(currentBlock == headerPropertiesStartBlock) {
                     // Special case where things are in the wrong order
-                    System.err.println("Warning, header block comes after data blocks in POIFS block listing");
+                    _logger.log(POILogger.WARN, "Warning, header block comes after data blocks in POIFS block listing");
                     currentBlock = POIFSConstants.END_OF_CHAIN;
                 } else if(currentBlock == 0 && firstPass) {
                     // Special case where the termination isn't done right
                     //  on an empty set
-                    System.err.println("Warning, incorrectly terminated empty data blocks in POIFS block listing (should end at -2, ended at 0)");
+                    _logger.log(POILogger.WARN, "Warning, incorrectly terminated empty data blocks in POIFS block listing (should end at -2, ended at 0)");
                     currentBlock = POIFSConstants.END_OF_CHAIN;
                 } else {
                     // Ripple up
index 4b82c08c8c27816da114f505c41a61c89d220e67..3532c2097021e4efdedc2dced7fc3adbcb6e4d1e 100644 (file)
@@ -127,7 +127,7 @@ public interface Sheet extends Iterable<Row> {
      *
      * <p>
      * Character width is defined as the maximum digit width
-     * of the numbers <code>0, 1, 2,  9</code> as rendered
+     * of the numbers <code>0, 1, 2, ... 9</code> as rendered
      * using the default font (first font in the workbook).
      * <br/>
      * Unless you are using a very special font, the default character is '0' (zero),
@@ -167,7 +167,7 @@ public interface Sheet extends Iterable<Row> {
      *
      * <p>
      * Character width is defined as the maximum digit width
-     * of the numbers <code>0, 1, 2,  9</code> as rendered
+     * of the numbers <code>0, 1, 2, ... 9</code> as rendered
      * using the default font (first font in the workbook)
      * </p>
      *
index 4d48d2bd7219fe52497a4bab6c63e9e5a2cd4581..7d33bdd022ee4052bb08d0c41e900912660d4d36 100644 (file)
@@ -24,6 +24,8 @@ import java.io.UnsupportedEncodingException;
 
 import org.apache.poi.openxml4j.exceptions.InvalidFormatException;
 import org.apache.poi.openxml4j.exceptions.InvalidOperationException;
+import org.apache.poi.util.POILogFactory;
+import org.apache.poi.util.POILogger;
 
 /**
  * Helper for part and pack URI.
@@ -32,6 +34,7 @@ import org.apache.poi.openxml4j.exceptions.InvalidOperationException;
  * @version 0.1
  */
 public final class PackagingURIHelper {
+    private final static POILogger _logger = POILogFactory.getLogger(PackagingURIHelper.class);
 
        /**
         * Package root URI.
@@ -293,7 +296,7 @@ public final class PackagingURIHelper {
                 try {
                     targetURI = new URI(path.substring(1));
                 } catch (Exception e) {
-                    System.err.println(e);
+                    _logger.log(POILogger.WARN, e);
                     return null;
                 }
             }
@@ -331,7 +334,7 @@ public final class PackagingURIHelper {
                        try {
                                return new URI(retVal.toString());
                        } catch (Exception e) {
-                               System.err.println(e);
+                               _logger.log(POILogger.WARN, e);
                                return null;
                        }
                }
@@ -373,7 +376,7 @@ public final class PackagingURIHelper {
                try {
                        return new URI(retVal.toString());
                } catch (Exception e) {
-                       System.err.println(e);
+                       _logger.log(POILogger.WARN, e);
                        return null;
                }
        }
index a7e7d1ce2cd902b69d089970114ced9abddf8bba..96af6932b0cdafebef1f440f7606ef2afe7cac40 100644 (file)
@@ -1261,7 +1261,7 @@ public class SXSSFSheet implements Sheet, Cloneable
 
         public SheetDataWriter() throws IOException 
         {
-            _fd = File.createTempFile("sheet", ".xml");
+            _fd = File.createTempFile("poi-sxxsf-sheet", ".xml");
             _fd.deleteOnExit();
             _out = new BufferedWriter(new FileWriter(_fd));
             _out.write("<sheetData>\n");
index 365e58b3370bd28068af65851e348c8c5ba62a36..0d2f39c38cc37591356374c7c909b79334c3e2e2 100644 (file)
@@ -533,7 +533,8 @@ public class SXSSFWorkbook implements Workbook
     public void write(OutputStream stream) throws IOException
     {
         //Save the template
-        File tmplFile = File.createTempFile("template", ".xlsx");
+        File tmplFile = File.createTempFile("poi-sxxsf-template", ".xlsx");
+        tmplFile.deleteOnExit();
         FileOutputStream os = new FileOutputStream(tmplFile);
         _wb.write(os);
         os.close();
index df3d9e455e3da66ab619e0447ea26032838306b1..6c9a0340ac980a955c44c31ffdf69382695557ea 100644 (file)
@@ -1935,7 +1935,7 @@ public class XSSFSheet extends POIXMLDocumentPart implements Sheet {
      *
      * <p>
      * Character width is defined as the maximum digit width
-     * of the numbers <code>0, 1, 2,  9</code> as rendered
+     * of the numbers <code>0, 1, 2, ... 9</code> as rendered
      * using the default font (first font in the workbook).
      * <br/>
      * Unless you are using a very special font, the default character is '0' (zero),
index e27708b1ce6b1fed1d1b83f47ed02d27391c277f..bba2f5bd15bcf5433de248d5b8585b4775bdc1d0 100644 (file)
@@ -148,7 +148,7 @@ public final class TestPOIXMLDocument extends TestCase {
         doc.parse(new TestFactory());
 
         for(POIXMLDocumentPart rel : doc.getRelations()){
-            System.out.println(rel);
+            //TODO finish me
         }
 
     }
index 2073be4bf9e4561e2b2a6b72ac24179c9b8fe0ee..7a2f29475233734b4d04a50b69fc7ddad59c8edc 100644 (file)
@@ -21,6 +21,8 @@ import java.io.IOException;
 import org.apache.poi.hwpf.model.io.HWPFFileSystem;
 import org.apache.poi.hwpf.model.io.HWPFOutputStream;
 import org.apache.poi.util.LittleEndian;
+import org.apache.poi.util.POILogFactory;
+import org.apache.poi.util.POILogger;
 
 /**
  * FontTable or in MS terminology sttbfffn is a common data structure written in all
@@ -32,6 +34,7 @@ import org.apache.poi.util.LittleEndian;
  */
 public final class FontTable
 {
+  private final static POILogger _logger = POILogFactory.getLogger(FontTable.class);
   private short _stringCount;// how many strings are included in the string table
   private short _extraDataSz;// size in bytes of the extra data
 
@@ -86,7 +89,7 @@ public final class FontTable
   {
     if(chpFtc >= _stringCount)
     {
-      System.out.println("Mismatch in chpFtc with stringCount");
+      _logger.log(POILogger.INFO, "Mismatch in chpFtc with stringCount");
       return null;
     }
 
@@ -97,7 +100,7 @@ public final class FontTable
   {
     if(chpFtc >= _stringCount)
     {
-      System.out.println("Mismatch in chpFtc with stringCount");
+      _logger.log(POILogger.INFO, "Mismatch in chpFtc with stringCount");
       return null;
     }
 
index c9b5fb9a7353a14dc3ffda5f1576d1a92c5ea7e3..e3338fd610a610067cf332decf077cb9801901ed 100644 (file)
@@ -17,6 +17,9 @@
 
 package org.apache.poi.hwpf.model;
 
+import org.apache.poi.util.POILogFactory;
+import org.apache.poi.util.POILogger;
+
 import java.util.Arrays;
 
 /**
@@ -30,6 +33,7 @@ import java.util.Arrays;
  */
 public abstract class PropertyNode implements Comparable, Cloneable
 {
+  private final static POILogger _logger = POILogFactory.getLogger(PropertyNode.class);
   protected Object _buf;
   /** The start, in characters */
   private int _cpStart;
@@ -49,7 +53,7 @@ public abstract class PropertyNode implements Comparable, Cloneable
       _buf = buf;
 
       if(_cpStart < 0) {
-         System.err.println("A property claimed to start before zero, at " + _cpStart + "! Resetting it to zero, and hoping for the best");
+         _logger.log(POILogger.WARN, "A property claimed to start before zero, at " + _cpStart + "! Resetting it to zero, and hoping for the best");
          _cpStart = 0;
       }
   }
index 987e49ac89effa3732006f089b038524648600db..8607b2109f28154c277fb0fc29ec63b4f966e9cd 100644 (file)
@@ -23,12 +23,15 @@ import java.util.List;
 
 import org.apache.poi.util.LittleEndian;
 import org.apache.poi.hwpf.model.io.*;
+import org.apache.poi.util.POILogFactory;
+import org.apache.poi.util.POILogger;
 
 /**
  * @author Ryan Ackley
  */
 public class SectionTable
 {
+  private final static POILogger _logger = POILogFactory.getLogger(SectionTable.class);
   private static final int SED_SIZE = 12;
 
   protected ArrayList<SEPX> _sections = new ArrayList<SEPX>();
@@ -92,7 +95,7 @@ public class SectionTable
        }
     }
     if(! matchAt && matchHalf) {
-       System.err.println("Your document seemed to be mostly unicode, but the section definition was in bytes! Trying anyway, but things may well go wrong!");
+       _logger.log(POILogger.WARN, "Your document seemed to be mostly unicode, but the section definition was in bytes! Trying anyway, but things may well go wrong!");
         for(int i=0; i<_sections.size(); i++) {
                SEPX s = _sections.get(i);
             GenericPropertyNode node = sedPlex.getProperty(i);
index 24b2a45b737b9c794cb90969d775243db4aec174..d36c5ba54fd1425cbd976a3807f8722f294362fc 100644 (file)
@@ -115,10 +115,12 @@ public final class TestVisioExtractor extends TestCase {
                // Check
                capture.flush();
                String text = baos.toString();
-               assertEquals(
+        // YK: stdout can contain lots of other stuff if logging is sent to console
+        // ( -Dorg.apache.poi.util.POILogger=org.apache.poi.util.SystemOutLogger)
+               assertTrue( text.contains(
                      "text\nView\n" +
                      "Test View\nI am a test view\n" +
-                     "Some random text, on a page\n"
-                     text);
+                     "Some random text, on a page\n"
+                     ));
        }
 }
index 74116219f5036d5c07bcfc8dfb1f8d223e83b21e..c96316744fd6e4b326a7c4f7bb61fab3002c2dde 100644 (file)
@@ -21,11 +21,14 @@ import org.apache.poi.hssf.HSSFITestDataProvider;
 import org.apache.poi.hssf.HSSFTestDataSamples;
 import org.apache.poi.ss.usermodel.*;
 import org.apache.poi.ss.util.CellReference;
+import org.apache.poi.util.POILogFactory;
+import org.apache.poi.util.POILogger;
 
 /**
  * Tests for {@link HSSFDataFormat}
  */
 public final class TestHSSFDataFormat extends BaseTestDataFormat {
+    private static POILogger _logger = POILogFactory.getLogger(TestHSSFDataFormat.class);
 
     public TestHSSFDataFormat() {
         super(HSSFITestDataProvider.instance);
@@ -61,8 +64,9 @@ public final class TestHSSFDataFormat extends BaseTestDataFormat {
                     CellStyle style = cell.getCellStyle();
 
                     String fmt = style.getDataFormatString();
-                    if(fmt == null)
-                        System.out.println(cell + ": " + fmt);
+                    if(fmt == null) {
+                        _logger.log(POILogger.WARN, cell + ": " + fmt);
+                    }
                 }
             }
         }
index 36886af9821e165e2f700fbc90dd836be7c9b979..d659f1654f72555f0ae453da8e253fc614fb6a10 100644 (file)
@@ -29,8 +29,11 @@ import org.apache.poi.poifs.filesystem.POIFSFileSystem;
 import org.apache.poi.poifs.filesystem.POIFSWriterEvent;
 import org.apache.poi.poifs.filesystem.POIFSWriterListener;
 import org.apache.poi.poifs.filesystem.DirectoryEntry;
+import org.apache.poi.util.POILogFactory;
+import org.apache.poi.util.POILogger;
 
 public final class TestEmptyDocument extends TestCase {
+    private static POILogger _logger = POILogFactory.getLogger(TestEmptyDocument.class);
 
        public void testSingleEmptyDocument() throws IOException {
                POIFSFileSystem fs = new POIFSFileSystem();
@@ -47,7 +50,7 @@ public final class TestEmptyDocument extends TestCase {
                DirectoryEntry dir = fs.getRoot();
                dir.createDocument("Foo", 0, new POIFSWriterListener() {
                        public void processPOIFSWriterEvent(POIFSWriterEvent event) {
-                               System.out.println("written");
+                               _logger.log(POILogger.WARN, "written");
                        }
                });