]> source.dussan.org Git - poi.git/commitdiff
bug 60526: Make loggers final and make throttled log actually work
authorDominik Stadler <centic@apache.org>
Fri, 30 Dec 2016 13:09:28 +0000 (13:09 +0000)
committerDominik Stadler <centic@apache.org>
Fri, 30 Dec 2016 13:09:28 +0000 (13:09 +0000)
git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1776555 13f79535-47bb-0310-9956-ffa450edef68

24 files changed:
src/java/org/apache/poi/hssf/eventusermodel/FormatTrackingHSSFListener.java
src/ooxml/java/org/apache/poi/openxml4j/opc/PackageRelationshipCollection.java
src/ooxml/java/org/apache/poi/openxml4j/opc/internal/ZipContentTypeManager.java
src/ooxml/java/org/apache/poi/openxml4j/opc/internal/marshallers/ZipPartMarshaller.java
src/ooxml/java/org/apache/poi/openxml4j/util/ZipSecureFile.java
src/ooxml/java/org/apache/poi/util/SAXHelper.java
src/ooxml/java/org/apache/poi/xslf/usermodel/XMLSlideShow.java
src/ooxml/java/org/apache/poi/xslf/usermodel/XSLFColor.java
src/ooxml/java/org/apache/poi/xslf/usermodel/XSLFGroupShape.java
src/ooxml/java/org/apache/poi/xssf/extractor/XSSFImportFromXML.java
src/ooxml/java/org/apache/poi/xssf/streaming/SXSSFFormulaEvaluator.java
src/scratchpad/src/org/apache/poi/hdgf/streams/ChunkStream.java
src/scratchpad/src/org/apache/poi/hmef/attribute/MAPIDateAttribute.java
src/scratchpad/src/org/apache/poi/hmef/attribute/MAPIStringAttribute.java
src/scratchpad/src/org/apache/poi/hmef/attribute/TNEFDateAttribute.java
src/scratchpad/src/org/apache/poi/hmef/attribute/TNEFStringAttribute.java
src/scratchpad/src/org/apache/poi/hslf/record/CurrentUserAtom.java
src/scratchpad/src/org/apache/poi/hsmf/parsers/POIFSChunkParser.java
src/scratchpad/src/org/apache/poi/hwmf/record/HwmfBitmapDib.java
src/scratchpad/src/org/apache/poi/hwpf/model/FIBFieldHandler.java
src/scratchpad/src/org/apache/poi/hwpf/model/ListTables.java
src/scratchpad/src/org/apache/poi/hwpf/model/PlfLfo.java
src/scratchpad/src/org/apache/poi/hwpf/usermodel/HWPFList.java
src/scratchpad/src/org/apache/poi/hwpf/usermodel/Paragraph.java

index 4a786962c9f04a34a272d9ccd590cb7377052efc..de9814e3a2a6cfc53b6a61471aa59a1cd18e14be 100644 (file)
@@ -41,7 +41,7 @@ import org.apache.poi.util.POILogger;
  * ids.
  */
 public class FormatTrackingHSSFListener implements HSSFListener {
-       private static POILogger logger = POILogFactory.getLogger(FormatTrackingHSSFListener.class);
+       private final static POILogger logger = POILogFactory.getLogger(FormatTrackingHSSFListener.class);
        private final HSSFListener _childListener;
        private final HSSFDataFormatter _formatter;
        private final NumberFormat _defaultFormat;
index 665c51f6c419cceb02207d501e0209e1b97a57bf..232b38b382fd987bb454c228cafd6002ad18013d 100644 (file)
@@ -43,7 +43,7 @@ import org.w3c.dom.NodeList;
 public final class PackageRelationshipCollection implements
         Iterable<PackageRelationship> {
 
-    private static POILogger logger = POILogFactory.getLogger(PackageRelationshipCollection.class);
+    private final static POILogger logger = POILogFactory.getLogger(PackageRelationshipCollection.class);
 
     /**
      * Package relationships ordered by ID.
index 1e00d93081da885a16b8865880a043dd9067eb0c..70bdb39e12ea902d716f0b3c9869b3a74bc43e5b 100644 (file)
@@ -38,7 +38,7 @@ import org.w3c.dom.Document;
  * @see ContentTypeManager
  */
 public class ZipContentTypeManager extends ContentTypeManager {
-    private static POILogger logger = POILogFactory.getLogger(ZipContentTypeManager.class);
+    private final static POILogger logger = POILogFactory.getLogger(ZipContentTypeManager.class);
 
        /**
         * Delegate constructor to the super constructor.
index 8860ae37e5d6b22d08fb38bc183bf6611102ee55..37abb442934ed92c85e4c4158616f16281147ccc 100644 (file)
@@ -48,7 +48,7 @@ import org.w3c.dom.Element;
  * @author Julien Chable
  */
 public final class ZipPartMarshaller implements PartMarshaller {
-       private static POILogger logger = POILogFactory.getLogger(ZipPartMarshaller.class);
+       private final static POILogger logger = POILogFactory.getLogger(ZipPartMarshaller.class);
 
        /**
         * Save the specified part.
index 9000656e5726ffe9b2554de028ea56aa954bdbb4..d1fc0cb1df31d156b31b3fd618e245a56796ede6 100644 (file)
@@ -45,7 +45,7 @@ import org.apache.poi.util.SuppressForbidden;
  * and {@link #setMinInflateRatio(double)}.\r
  */\r
 public class ZipSecureFile extends ZipFile {\r
-    private static POILogger logger = POILogFactory.getLogger(ZipSecureFile.class);\r
+    private final static POILogger logger = POILogFactory.getLogger(ZipSecureFile.class);\r
     \r
     private static double MIN_INFLATE_RATIO = 0.01d;\r
     private static long MAX_ENTRY_SIZE = 0xFFFFFFFFL;\r
index d83b814328d288e71fc068dd744008dd73ff3d5f..3ff70a5be35fba362038b4a32547e0e08daa32f9 100644 (file)
@@ -36,7 +36,7 @@ import org.xml.sax.XMLReader;
  * Provides handy methods for working with SAX parsers and readers
  */
 public final class SAXHelper {
-    private static POILogger logger = POILogFactory.getLogger(SAXHelper.class);
+    private static final POILogger logger = POILogFactory.getLogger(SAXHelper.class);
     private static long lastLog = 0;
 
     private SAXHelper() {}
@@ -94,6 +94,7 @@ public final class SAXHelper {
                 // throttle the log somewhat as it can spam the log otherwise
                 if(System.currentTimeMillis() > lastLog + TimeUnit.MINUTES.toMillis(5)) {
                     logger.log(POILogger.WARN, "SAX Security Manager could not be setup [log suppressed for 5 minutes]", e);
+                    lastLog = System.currentTimeMillis();
                 }
             }
         }
index 14b656b53e436d5f2988478b0594257ff27370e3..cee2d0eb04eb8c4cdb319e918deeb18bb0c59123 100644 (file)
@@ -72,7 +72,7 @@ import org.openxmlformats.schemas.presentationml.x2006.main.PresentationDocument
 @Beta
 public class XMLSlideShow extends POIXMLDocument
 implements SlideShow<XSLFShape,XSLFTextParagraph> {
-    private static POILogger _logger = POILogFactory.getLogger(XMLSlideShow.class);
+    private final static POILogger _logger = POILogFactory.getLogger(XMLSlideShow.class);
 
     private CTPresentation _presentation;
     private List<XSLFSlide> _slides;
index f519f7ec205c828d4166fbcfe13c583c0fef2944..bff6e0140aa485db40874de0e986b01367680b7b 100644 (file)
@@ -46,7 +46,7 @@ import org.w3c.dom.Node;
 @Beta\r
 @Internal\r
 public class XSLFColor {\r
-    private static POILogger LOGGER = POILogFactory.getLogger(XSLFColor.class);\r
+    private final static POILogger LOGGER = POILogFactory.getLogger(XSLFColor.class);\r
     \r
     private XmlObject _xmlObject;\r
     private Color _color;\r
index 08022e976cf007ccb541e3a9be6447bc450a47bc..bb904be56ae154e13c26c8a3b7d5972a2766e00f 100644 (file)
@@ -55,7 +55,7 @@ import org.openxmlformats.schemas.presentationml.x2006.main.CTShape;
 @Beta\r
 public class XSLFGroupShape extends XSLFShape\r
 implements XSLFShapeContainer, GroupShape<XSLFShape,XSLFTextParagraph> {\r
-    private static POILogger _logger = POILogFactory.getLogger(XSLFGroupShape.class);\r
+    private final static POILogger _logger = POILogFactory.getLogger(XSLFGroupShape.class);\r
     \r
     private final List<XSLFShape> _shapes;\r
     private final CTGroupShapeProperties _grpSpPr;\r
index 0ffbd0b88c18aaee758d3563f113a4d8ab8340e7..478ed54172918402e70c7cff78451a93db36aee8 100644 (file)
@@ -71,7 +71,7 @@ public class XSSFImportFromXML {
 
     private final XSSFMap _map;
 
-    private static POILogger logger = POILogFactory.getLogger(XSSFImportFromXML.class);
+    private final static POILogger logger = POILogFactory.getLogger(XSSFImportFromXML.class);
 
     public XSSFImportFromXML(XSSFMap map) {
         _map = map;
index 0ac776ffcafae87606e192433fee17792fa2b1ae..d6e258e9c6a430ff615a485c518ee0b769c586e4 100644 (file)
@@ -34,7 +34,7 @@ import org.apache.poi.xssf.usermodel.BaseXSSFFormulaEvaluator;
  *  lookup cells within the current Window.
  */
 public final class SXSSFFormulaEvaluator extends BaseXSSFFormulaEvaluator {
-    private static POILogger logger = POILogFactory.getLogger(SXSSFFormulaEvaluator.class);
+    private final static POILogger logger = POILogFactory.getLogger(SXSSFFormulaEvaluator.class);
     
     private SXSSFWorkbook wb;
     
index 9f15e40b9fa14c6af49e7bbfde5acf9ebe8d867e..f58a4c85102cb3097dc9c343ba626b27a4b5870c 100644 (file)
@@ -27,7 +27,7 @@ import org.apache.poi.util.POILogFactory;
 import org.apache.poi.util.POILogger;
 
 public final class ChunkStream extends Stream {
-       private static POILogger logger = POILogFactory.getLogger(ChunkStream.class);
+       private final static POILogger logger = POILogFactory.getLogger(ChunkStream.class);
        
        private ChunkFactory chunkFactory;
        /** All the Chunks we contain */
index 53eb575e6f17fc2977fca67734874fb466442d58..0669723de504c65902f4256a02db967bfe694eda 100644 (file)
@@ -42,7 +42,7 @@ import org.apache.poi.util.POILogger;
  * @see <a href="https://msdn.microsoft.com/en-us/library/cc433490(v=exchg.80).aspx">[MS-OXPROPS]: Exchange Server Protocols Master Property List</a>
  */
 public final class MAPIDateAttribute extends MAPIAttribute {
-   private static POILogger logger = POILogFactory.getLogger(MAPIDateAttribute.class);
+   private final static POILogger logger = POILogFactory.getLogger(MAPIDateAttribute.class);
    private Date data;
    
    /**
index 5307e71c74af178ebb4a3268aebb3bb3f8e64a95..ef4e16b6e751f57e59387c7c0de1737c26026997 100644 (file)
@@ -32,7 +32,7 @@ import org.apache.poi.util.StringUtil;
  *  to a {@link HMEFMessage} or one of its {@link Attachment}s.
  */
 public final class MAPIStringAttribute extends MAPIAttribute {
-   private static POILogger logger = POILogFactory.getLogger(MAPIStringAttribute.class);
+   private final static POILogger logger = POILogFactory.getLogger(MAPIStringAttribute.class);
    private static final String CODEPAGE = "CP1252";
    private final String data;
    
index 1df0d969161e5c47c6b3d59a4b480f14afab09a1..faedc4c7cc595a0057d2be55e8807323cb933214 100644 (file)
@@ -39,7 +39,7 @@ import org.apache.poi.util.POILogger;
  *  or one of its {@link Attachment}s.
  */
 public final class TNEFDateAttribute extends TNEFAttribute {
-   private static POILogger logger = POILogFactory.getLogger(TNEFDateAttribute.class);
+   private final static POILogger logger = POILogFactory.getLogger(TNEFDateAttribute.class);
    private Date data;
    
    /**
index 34e1ef36916fdd4f546cf54ecc62d08c57960bee..a63b0d12b6799988bd4dfc82a005f7f78f7b3e4e 100644 (file)
@@ -31,7 +31,7 @@ import org.apache.poi.util.StringUtil;
  *  or one of its {@link Attachment}s.
  */
 public final class TNEFStringAttribute extends TNEFAttribute {
-   private static POILogger logger = POILogFactory.getLogger(TNEFStringAttribute.class);
+   private final static POILogger logger = POILogFactory.getLogger(TNEFStringAttribute.class);
    private String data;
    
    /**
index 2d688603a0909cecb013dbda1318c895f88329ed..d05c5b2d1456aa3d4f90368b036734abd9140d5d 100644 (file)
@@ -43,7 +43,7 @@ import org.apache.poi.util.StringUtil;
  */
 public class CurrentUserAtom
 {
-       private static POILogger logger = POILogFactory.getLogger(CurrentUserAtom.class);
+       private final static POILogger logger = POILogFactory.getLogger(CurrentUserAtom.class);
 
        /** Standard Atom header */
        public static final byte[] atomHeader = new byte[] { 0, 0, -10, 15 };
index c44f4947f3fba3ca367402bdcec250c213ac5b69..ffe44ad936a95d5de125461db31c08f468ead711 100644 (file)
@@ -50,7 +50,7 @@ import org.apache.poi.util.POILogger;
  * data and so on.
  */
 public final class POIFSChunkParser {
-   private static POILogger logger = POILogFactory.getLogger(POIFSChunkParser.class);
+   private final static POILogger logger = POILogFactory.getLogger(POIFSChunkParser.class);
 
    public static ChunkGroup[] parse(NPOIFSFileSystem fs) throws IOException {
       return parse(fs.getRoot());
index ba935f39f16597e9e357b10a700aee0a865edacf..da49d2ebd412e61b6761c41e37e1491b98b43068 100644 (file)
@@ -189,7 +189,7 @@ public class HwmfBitmapDib {
         }\r
     }\r
 \r
-    private static POILogger logger = POILogFactory.getLogger(HwmfBitmapDib.class);\r
+    private final static POILogger logger = POILogFactory.getLogger(HwmfBitmapDib.class);\r
     private static final int BMP_HEADER_SIZE = 14;\r
     \r
     private int headerSize;\r
index ec422d9ffde521bdac38f79a7eb9758342a79a33..ae830f62cd31ca16a5dd2dad81fee4fcc577e843 100644 (file)
@@ -144,7 +144,7 @@ public final class FIBFieldHandler
   public static final int STTBLISTNAMES = 91;
   public static final int STTBFUSSR = 92;
 
-  private static POILogger log = POILogFactory.getLogger(FIBFieldHandler.class);
+  private final static POILogger log = POILogFactory.getLogger(FIBFieldHandler.class);
 
   private static final int FIELD_SIZE = LittleEndian.INT_SIZE * 2;
 
index 5ec02537f58d01c66fc68afa190ee1db998e7e68..01b9577421e4a447197d4239f65ff6449e96b105 100644 (file)
@@ -34,7 +34,7 @@ import org.apache.poi.util.POILogger;
 @Internal
 public final class ListTables
 {
-  private static POILogger log = POILogFactory.getLogger(ListTables.class);
+  private final static POILogger log = POILogFactory.getLogger(ListTables.class);
 
     /**
      * Both PlfLst and the following LVLs
index 90618da7435995e975bef5bbd9ffb091ece77e56..b387e8f31503911854553cbd8da117c6d6a20d2f 100644 (file)
@@ -37,7 +37,7 @@ import org.apache.poi.util.POILogger;
  */
 public class PlfLfo
 {
-    private static POILogger log = POILogFactory.getLogger( PlfLfo.class );
+    private final static POILogger log = POILogFactory.getLogger( PlfLfo.class );
 
     /**
      * An unsigned integer that specifies the count of elements in both the
index 6f69f71726e0bea16188146838486800745168a8..3364aecb080b097a1be6edf380c7131c0a9b8f46 100644 (file)
@@ -48,14 +48,11 @@ import org.apache.poi.hwpf.sprm.ParagraphSprmCompressor;
  */
 public final class HWPFList
 {
-    //private static POILogger log = POILogFactory.getLogger( HWPFList.class );
-
     private boolean _ignoreLogicalLeftIdentation = false;
     private LFO _lfo;
     private LFOData _lfoData;
     private ListData _listData;
     private ListTables _listTables;
-    private boolean _registered;
     private StyleSheet _styleSheet;
 
     /**
@@ -80,7 +77,6 @@ public final class HWPFList
     {
         _listTables = listTables;
         _styleSheet = styleSheet;
-        _registered = true;
 
         /* See documentation for sprmPIlfo (0x460B) */
         if ( ilfo == 0 || ilfo == 0xF801 )
@@ -214,8 +210,7 @@ public final class HWPFList
         int styleIndex = _listData.getLevelStyle( level );
         CharacterProperties base = _styleSheet.getCharacterStyle( styleIndex );
 
-        byte[] grpprl = CharacterSprmCompressor.compressCharacterProperty( chp,
-                base );
+        byte[] grpprl = CharacterSprmCompressor.compressCharacterProperty( chp, base );
         listLevel.setNumberProperties( grpprl );
     }
 
@@ -233,8 +228,7 @@ public final class HWPFList
         int styleIndex = _listData.getLevelStyle( level );
         ParagraphProperties base = _styleSheet.getParagraphStyle( styleIndex );
 
-        byte[] grpprl = ParagraphSprmCompressor.compressParagraphProperty( pap,
-                base );
+        byte[] grpprl = ParagraphSprmCompressor.compressParagraphProperty( pap, base );
         listLevel.setLevelProperties( grpprl );
     }
 
index 6e5f688160ce0f6a0aee8c0c8f12da6189319ffd..9da19fd3b917113a317a8ab5a92a454d5711dac3 100644 (file)
@@ -33,7 +33,7 @@ import org.apache.poi.util.POILogFactory;
 import org.apache.poi.util.POILogger;
 
 public class Paragraph extends Range implements Cloneable {
-    private static POILogger log = POILogFactory.getLogger( Paragraph.class );
+    private final static POILogger log = POILogFactory.getLogger( Paragraph.class );
 
     public final static short SPRM_JC = 0x2403;
     public final static short SPRM_FSIDEBYSIDE = 0x2404;