You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by gp...@apache.org on 2010/06/22 15:01:24 UTC
svn commit: r956873 -
/myfaces/extensions/cdi/trunk/jse-modules/message-module/impl/src/main/java/org/apache/myfaces/extensions/cdi/message/impl/DefaultFormatterFactory.java
Author: gpetracek
Date: Tue Jun 22 13:01:24 2010
New Revision: 956873
URL: http://svn.apache.org/viewvc?rev=956873&view=rev
Log:
EXTCDI-21 minor change
Modified:
myfaces/extensions/cdi/trunk/jse-modules/message-module/impl/src/main/java/org/apache/myfaces/extensions/cdi/message/impl/DefaultFormatterFactory.java
Modified: myfaces/extensions/cdi/trunk/jse-modules/message-module/impl/src/main/java/org/apache/myfaces/extensions/cdi/message/impl/DefaultFormatterFactory.java
URL: http://svn.apache.org/viewvc/myfaces/extensions/cdi/trunk/jse-modules/message-module/impl/src/main/java/org/apache/myfaces/extensions/cdi/message/impl/DefaultFormatterFactory.java?rev=956873&r1=956872&r2=956873&view=diff
==============================================================================
--- myfaces/extensions/cdi/trunk/jse-modules/message-module/impl/src/main/java/org/apache/myfaces/extensions/cdi/message/impl/DefaultFormatterFactory.java (original)
+++ myfaces/extensions/cdi/trunk/jse-modules/message-module/impl/src/main/java/org/apache/myfaces/extensions/cdi/message/impl/DefaultFormatterFactory.java Tue Jun 22 13:01:24 2010
@@ -27,7 +27,6 @@ import java.io.Serializable;
import java.util.List;
import java.util.ArrayList;
import java.util.Map;
-import java.util.HashMap;
import java.util.Collections;
import java.util.Locale;
import java.util.concurrent.ConcurrentHashMap;
@@ -50,16 +49,7 @@ public class DefaultFormatterFactory imp
private Map<FormatterConfigKey, GenericConfig> formatterConfigs =
new ConcurrentHashMap<FormatterConfigKey, GenericConfig>();
- private Logger getLogger()
- {
- if(this.logger == null)
- {
- this.logger = Logger.getLogger(getClass().getName());
- }
- return this.logger;
- }
-
- public FormatterFactory add(Formatter formatter)
+ public synchronized FormatterFactory add(Formatter formatter)
{
this.formatters.add(formatter);
formatterCache = null;
@@ -84,7 +74,7 @@ public class DefaultFormatterFactory imp
{
if (this.formatterCache == null)
{
- this.formatterCache = new HashMap<Class<?>, Formatter>();
+ this.formatterCache = new ConcurrentHashMap<Class<?>, Formatter>();
}
Formatter found = findFormatterFor(type, this.formatters);
@@ -138,6 +128,15 @@ public class DefaultFormatterFactory imp
return new FormatterConfigKey(type, locale);
}
+ private Logger getLogger()
+ {
+ if(this.logger == null)
+ {
+ this.logger = Logger.getLogger(getClass().getName());
+ }
+ return this.logger;
+ }
+
class FormatterConfigKey implements Serializable
{
private static final long serialVersionUID = -6430653319283563370L;