Apache
/
poi
mirror of https://github.com/apache/poi.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fix bug #44627 - improve the thread safety of POILogFactory 

git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@638815 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
Nick Burch 2008-03-19 12:49:35 +00:00
parent e33dbf93a0
commit b43afc9428
5 changed files with 76 additions and 44 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
src/documentation/content/xdocs/changes.xml
View File

@ -36,6 +36,7 @@
<!-- Don't forget to update status.xml too! -->
<release version="3.1-beta1" date="2008-??-??">
<action dev="POI-DEVELOPERS" type="fix">44627 - Improve the thread safety of POILogFactory</action>
<action dev="POI-DEVELOPERS" type="add">30311 - Initial support for Conditional Formatting</action>
<action dev="POI-DEVELOPERS" type="fix">44609 - Handle leading spaces in formulas, such as '= 4'</action>
<action dev="POI-DEVELOPERS" type="add">44608 - Support for PercentPtg in the formula evaluator</action>

1
src/documentation/content/xdocs/status.xml
View File

@ -33,6 +33,7 @@
<!-- Don't forget to update changes.xml too! -->
<changes>
<release version="3.1-beta1" date="2008-??-??">
<action dev="POI-DEVELOPERS" type="fix">44627 - Improve the thread safety of POILogFactory</action>
<action dev="POI-DEVELOPERS" type="add">30311 - Initial support for Conditional Formatting</action>
<action dev="POI-DEVELOPERS" type="fix">44609 - Handle leading spaces in formulas, such as '= 4'</action>
<action dev="POI-DEVELOPERS" type="add">44608 - Support for PercentPtg in the formula evaluator</action>

73
src/java/org/apache/poi/util/POILogFactory.java
View File

@ -33,14 +33,25 @@ import java.util.*;
public class POILogFactory
{
// map of POILogger instances, with classes as keys
private static Map _loggers = new HashMap();;
/**
* Map of POILogger instances, with classes as keys
*/
private static Map _loggers = new HashMap();;
/**
* construct a POILogFactory.
* A common instance of NullLogger, as it does nothing
* we only need the one
*/
private static POILogger _nullLogger = new NullLogger();
/**
* The name of the class to use. Initialised the
* first time we need it
*/
private static String _loggerClassName = null;
/**
* Construct a POILogFactory.
*/
private POILogFactory()
{
}
@ -69,28 +80,48 @@ public class POILogFactory
public static POILogger getLogger(final String cat)
{
POILogger logger = null;
if (_loggers.containsKey(cat))
{
logger = ( POILogger ) _loggers.get(cat);
// If we haven't found out what logger to use yet,
// then do so now
// Don't look it up until we're first asked, so
// that our users can set the system property
// between class loading and first use
if(_loggerClassName == null) {
try {
_loggerClassName = System.getProperty("org.apache.poi.util.POILogger");
} catch(Exception e) {}
// Use the default logger if none specified,
// or none could be fetched
if(_loggerClassName == null) {
_loggerClassName = _nullLogger.getClass().getName();
}
}
else
{
try{
String loggerClassName = System.getProperty("org.apache.poi.util.POILogger");
Class loggerClass = Class.forName(loggerClassName);
// Short circuit for the null logger, which
// ignores all categories
if(_loggerClassName.equals(_nullLogger.getClass().getName())) {
return _nullLogger;
}
// Fetch the right logger for them, creating
// it if that's required
if (_loggers.containsKey(cat)) {
logger = ( POILogger ) _loggers.get(cat);
} else {
try {
Class loggerClass = Class.forName(_loggerClassName);
logger = ( POILogger ) loggerClass.newInstance();
}
catch(Exception e){
logger = new NullLogger();
logger.initialize(cat);
} catch(Exception e) {
// Give up and use the null logger
logger = _nullLogger;
}
logger.initialize(cat);
// Save for next time
_loggers.put(cat, logger);
}
return logger;
}
} // end public class POILogFactory
} // end public class POILogFactory

22
src/testcases/org/apache/poi/poifs/storage/TestRawDataBlock.java
View File

@ -36,23 +36,23 @@ import junit.framework.*;
public class TestRawDataBlock
extends TestCase
{
/**
* Constructor TestRawDataBlock
*
* @param name
*/
public TestRawDataBlock(String name)
{
super(name);
static {
// We always want to use our own
// logger
System.setProperty(
"org.apache.poi.util.POILogger",
"org.apache.poi.util.DummyPOILogger"
);
}
/**
* Constructor TestRawDataBlock
*
* @param name
*/
public TestRawDataBlock(String name)
{
super(name);
}
/**

23
src/testcases/org/apache/poi/poifs/storage/TestRawDataBlockList.java
View File

@ -36,23 +36,23 @@ import junit.framework.*;
public class TestRawDataBlockList
extends TestCase
{
/**
* Constructor TestRawDataBlockList
*
* @param name
*/
public TestRawDataBlockList(String name)
{
super(name);
static {
// We always want to use our own
// logger
System.setProperty(
"org.apache.poi.util.POILogger",
"org.apache.poi.util.DummyPOILogger"
);
}
/**
* Constructor TestRawDataBlockList
*
* @param name
*/
public TestRawDataBlockList(String name)
{
super(name);
}
/**
@ -60,7 +60,6 @@ public class TestRawDataBlockList
*
* @exception IOException
*/
public void testNormalConstructor()
throws IOException
{