Attachment 34562 Details for Bug 60526 – [PATCH] update lastLog

[PATCH] update lastLog

patch.txt (text/plain), 1.07 KB, created by PJ Fanning on 2016-12-28 21:32:31 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: PJ Fanning

Created: 2016-12-28 21:32:31 UTC

Size: 1.07 KB

patch

obsolete

>diff --git a/src/ooxml/java/org/apache/poi/util/SAXHelper.java b/src/ooxml/java/org/apache/poi/util/SAXHelper.java
>index d83b814..3ff70a5 100644
>--- a/src/ooxml/java/org/apache/poi/util/SAXHelper.java
>+++ b/src/ooxml/java/org/apache/poi/util/SAXHelper.java
>@@ -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();
>                 }
>             }
>         }

Actions: View

Attachments on bug 60526: 34562 | 34563