You are viewing a plain text version of this content. The canonical link for it is here.
Posted to log4j-dev@logging.apache.org by ce...@apache.org on 2004/12/28 17:33:58 UTC
cvs commit: logging-log4j/src/java/org/apache/log4j/joran/action PluginAction.java AppenderAction.java PropertyAction.java ConversionRuleAction.java RootLoggerAction.java LayoutAction.java LoggerAction.java AppenderRefAction.java ConfigurationAction.java
ceki 2004/12/28 08:33:58
Modified: src/java/org/apache/log4j/joran/action PluginAction.java
AppenderAction.java PropertyAction.java
ConversionRuleAction.java RootLoggerAction.java
LayoutAction.java LoggerAction.java
AppenderRefAction.java ConfigurationAction.java
Log:
Removed use of static loggers in Actions. Replaced by the getLogger() method.
Revision Changes Path
1.5 +8 -9 logging-log4j/src/java/org/apache/log4j/joran/action/PluginAction.java
Index: PluginAction.java
===================================================================
RCS file: /home/cvs/logging-log4j/src/java/org/apache/log4j/joran/action/PluginAction.java,v
retrieving revision 1.4
retrieving revision 1.5
diff -u -r1.4 -r1.5
--- PluginAction.java 19 Nov 2004 15:20:06 -0000 1.4
+++ PluginAction.java 28 Dec 2004 16:33:57 -0000 1.5
@@ -19,7 +19,6 @@
import org.apache.joran.action.Action;
import org.apache.joran.helper.Option;
-import org.apache.log4j.Logger;
import org.apache.log4j.helpers.OptionConverter;
import org.apache.log4j.plugins.Plugin;
import org.apache.log4j.spi.ErrorItem;
@@ -29,7 +28,7 @@
import org.xml.sax.Attributes;
public class PluginAction extends Action {
- static final Logger logger = Logger.getLogger(PluginAction.class);
+
Plugin plugin;
boolean inError = false;
@@ -43,7 +42,7 @@
String className = attributes.getValue(CLASS_ATTRIBUTE);
try {
- logger.debug(
+ getLogger().debug(
"About to instantiate plugin of type [" + className + "]");
Object instance =
@@ -54,11 +53,11 @@
String pluginName = attributes.getValue(NAME_ATTRIBUTE);
if (Option.isEmpty(pluginName)) {
- logger.warn(
+ getLogger().warn(
"No plugin name given for plugin of type " + className + "].");
} else {
plugin.setName(pluginName);
- logger.debug("plugin named as [" + pluginName + "]");
+ getLogger().debug("plugin named as [" + pluginName + "]");
}
LoggerRepository repository = (LoggerRepository) ec.getObject(0);
@@ -66,11 +65,11 @@
repository.getPluginRegistry().addPlugin(plugin);
plugin.setLoggerRepository(repository);
- logger.debug("Pushing plugin on to the object stack.");
+ getLogger().debug("Pushing plugin on to the object stack.");
ec.pushObject(plugin);
} catch (Exception oops) {
inError = true;
- logger.error(
+ getLogger().error(
"Could not create a plugin. Reported error follows.", oops);
ec.addError(
new ErrorItem(
@@ -94,11 +93,11 @@
Object o = ec.peekObject();
if (o != plugin) {
- logger.warn(
+ getLogger().warn(
"The object at the of the stack is not the plugin named ["
+ plugin.getName() + "] pushed earlier.");
} else {
- logger.warn(
+ getLogger().warn(
"Popping plugin named [" + plugin.getName()
+ "] from the object stack");
ec.popObject();
1.13 +8 -9 logging-log4j/src/java/org/apache/log4j/joran/action/AppenderAction.java
Index: AppenderAction.java
===================================================================
RCS file: /home/cvs/logging-log4j/src/java/org/apache/log4j/joran/action/AppenderAction.java,v
retrieving revision 1.12
retrieving revision 1.13
diff -u -r1.12 -r1.13
--- AppenderAction.java 19 Nov 2004 15:20:06 -0000 1.12
+++ AppenderAction.java 28 Dec 2004 16:33:57 -0000 1.13
@@ -20,7 +20,6 @@
import org.apache.joran.helper.Option;
import org.apache.log4j.Appender;
-import org.apache.log4j.Logger;
import org.apache.log4j.helpers.OptionConverter;
import org.apache.log4j.spi.ErrorItem;
import org.apache.log4j.spi.OptionHandler;
@@ -31,7 +30,7 @@
public class AppenderAction extends Action {
- static final Logger logger = Logger.getLogger(AppenderAction.class);
+
Appender appender;
private boolean inError = false;
@@ -46,7 +45,7 @@
String className = attributes.getValue(CLASS_ATTRIBUTE);
try {
- logger.debug(
+ getLogger().debug(
"About to instantiate appender of type [" + className + "]");
Object instance =
@@ -57,11 +56,11 @@
String appenderName = attributes.getValue(NAME_ATTRIBUTE);
if (Option.isEmpty(appenderName)) {
- logger.warn(
+ getLogger().warn(
"No appender name given for appender of type " + className + "].");
} else {
appender.setName(appenderName);
- logger.debug("Appender named as [" + appenderName + "]");
+ getLogger().debug("Appender named as [" + appenderName + "]");
}
// The execution context contains a bag which contains the appenders
@@ -71,11 +70,11 @@
// add the appender just created to the appender bag.
appenderBag.put(appenderName, appender);
- logger.debug("Pushing appender on to the object stack.");
+ getLogger().debug("Pushing appender on to the object stack.");
ec.pushObject(appender);
} catch (Exception oops) {
inError = true;
- logger.error(
+ getLogger().error(
"Could not create an Appender. Reported error follows.", oops);
ec.addError(
new ErrorItem(
@@ -99,11 +98,11 @@
Object o = ec.peekObject();
if (o != appender) {
- logger.warn(
+ getLogger().warn(
"The object at the of the stack is not the appender named ["
+ appender.getName() + "] pushed earlier.");
} else {
- logger.debug(
+ getLogger().debug(
"Popping appender named [" + appender.getName()
+ "] from the object stack");
ec.popObject();
1.8 +0 -2 logging-log4j/src/java/org/apache/log4j/joran/action/PropertyAction.java
Index: PropertyAction.java
===================================================================
RCS file: /home/cvs/logging-log4j/src/java/org/apache/log4j/joran/action/PropertyAction.java,v
retrieving revision 1.7
retrieving revision 1.8
diff -u -r1.7 -r1.8
--- PropertyAction.java 20 Nov 2004 12:47:06 -0000 1.7
+++ PropertyAction.java 28 Dec 2004 16:33:57 -0000 1.8
@@ -20,7 +20,6 @@
import org.apache.joran.action.Action;
import org.apache.joran.helper.Option;
-import org.apache.log4j.Logger;
import org.apache.log4j.helpers.OptionConverter;
import org.apache.log4j.spi.ErrorItem;
@@ -38,7 +37,6 @@
* @author Ceki Gülcü
*/
abstract public class PropertyAction extends Action {
- static final Logger logger = Logger.getLogger(PropertyAction.class);
static String INVALID_ATTRIBUTES =
"In <property> element, either the \"file\" attribute or both the \"name\" and \"value\" attributes must be set.";
1.11 +4 -6 logging-log4j/src/java/org/apache/log4j/joran/action/ConversionRuleAction.java
Index: ConversionRuleAction.java
===================================================================
RCS file: /home/cvs/logging-log4j/src/java/org/apache/log4j/joran/action/ConversionRuleAction.java,v
retrieving revision 1.10
retrieving revision 1.11
diff -u -r1.10 -r1.11
--- ConversionRuleAction.java 19 Nov 2004 15:20:06 -0000 1.10
+++ ConversionRuleAction.java 28 Dec 2004 16:33:57 -0000 1.11
@@ -21,7 +21,6 @@
import org.apache.joran.helper.Option;
import org.apache.log4j.Layout;
-import org.apache.log4j.Logger;
import org.apache.log4j.PatternLayout;
import org.apache.log4j.spi.ErrorItem;
@@ -30,7 +29,6 @@
public class ConversionRuleAction extends Action {
- static final Logger logger = Logger.getLogger(ConversionRuleAction.class);
Layout layout;
boolean inError = false;
@@ -51,7 +49,7 @@
if (Option.isEmpty(conversionWord)) {
inError = true;
errorMsg = "No 'conversionWord' attribute in <conversionRule>";
- logger.warn(errorMsg);
+ getLogger().warn(errorMsg);
ec.addError(new ErrorItem(errorMsg));
return;
@@ -60,14 +58,14 @@
if (Option.isEmpty(converterClass)) {
inError = true;
errorMsg = "No 'converterClass' attribute in <conversionRule>";
- logger.warn(errorMsg);
+ getLogger().warn(errorMsg);
ec.addError(new ErrorItem(errorMsg));
return;
}
try {
- logger.debug(
+ getLogger().debug(
"About to add conversion rule [" + conversionWord + ", "
+ converterClass + "] to layout");
@@ -80,7 +78,7 @@
} catch (Exception oops) {
inError = true;
errorMsg = "Could not add conversion rule to PatternLayout.";
- logger.error(errorMsg, oops);
+ getLogger().error(errorMsg, oops);
ec.addError(new ErrorItem(errorMsg));
}
}
1.9 +1 -1 logging-log4j/src/java/org/apache/log4j/joran/action/RootLoggerAction.java
Index: RootLoggerAction.java
===================================================================
RCS file: /home/cvs/logging-log4j/src/java/org/apache/log4j/joran/action/RootLoggerAction.java,v
retrieving revision 1.8
retrieving revision 1.9
diff -u -r1.8 -r1.9
--- RootLoggerAction.java 20 Nov 2004 12:47:06 -0000 1.8
+++ RootLoggerAction.java 28 Dec 2004 16:33:57 -0000 1.9
@@ -30,7 +30,7 @@
static final String ADDITIVITY_ATTR = "additivity";
static final String EMPTY_STR = "";
static final Class[] ONE_STRING_PARAM = new Class[] { String.class };
- //Logger logger = Logger.getLogger(RootLoggerAction.class);
+
Logger root;
boolean inError = false;
1.12 +7 -9 logging-log4j/src/java/org/apache/log4j/joran/action/LayoutAction.java
Index: LayoutAction.java
===================================================================
RCS file: /home/cvs/logging-log4j/src/java/org/apache/log4j/joran/action/LayoutAction.java,v
retrieving revision 1.11
retrieving revision 1.12
diff -u -r1.11 -r1.12
--- LayoutAction.java 19 Nov 2004 15:20:06 -0000 1.11
+++ LayoutAction.java 28 Dec 2004 16:33:57 -0000 1.12
@@ -21,7 +21,6 @@
import org.apache.log4j.Appender;
import org.apache.log4j.Layout;
-import org.apache.log4j.Logger;
import org.apache.log4j.helpers.OptionConverter;
import org.apache.log4j.spi.ErrorItem;
import org.apache.log4j.spi.OptionHandler;
@@ -30,7 +29,6 @@
public class LayoutAction extends Action {
- static final Logger logger = Logger.getLogger(LayoutAction.class);
Layout layout;
boolean inError = false;
@@ -45,18 +43,18 @@
String className =
attributes.getValue(CLASS_ATTRIBUTE);
try {
- logger.debug("About to instantiate layout of type [" + className + "]");
+ getLogger().debug("About to instantiate layout of type [" + className + "]");
Object instance =
OptionConverter.instantiateByClassName(
className, org.apache.log4j.Layout.class, null);
layout = (Layout) instance;
- logger.debug("Pushing layout on top of the object stack.");
+ getLogger().debug("Pushing layout on top of the object stack.");
ec.pushObject(layout);
} catch (Exception oops) {
inError = true;
- logger.error(
+ getLogger().error(
"Could not create an Layout. Reported error follows.", oops);
ec.addError(new ErrorItem("Could not create layout of type " + className + "]."));
}
@@ -78,18 +76,18 @@
Object o = ec.peekObject();
if (o != layout) {
- logger.warn(
+ getLogger().warn(
"The object on the top the of the stack is not the layout pushed earlier.");
} else {
- logger.debug("Popping layout from the object stack");
+ getLogger().debug("Popping layout from the object stack");
ec.popObject();
try {
- logger.debug("About to set the layout of the containing appender.");
+ getLogger().debug("About to set the layout of the containing appender.");
Appender appender = (Appender) ec.peekObject();
appender.setLayout(layout);
} catch(Exception ex) {
- logger.error("Could not set the layout for containing appender.", ex);
+ getLogger().error("Could not set the layout for containing appender.", ex);
}
}
}
1.10 +9 -10 logging-log4j/src/java/org/apache/log4j/joran/action/LoggerAction.java
Index: LoggerAction.java
===================================================================
RCS file: /home/cvs/logging-log4j/src/java/org/apache/log4j/joran/action/LoggerAction.java,v
retrieving revision 1.9
retrieving revision 1.10
diff -u -r1.9 -r1.10
--- LoggerAction.java 19 Nov 2004 15:20:06 -0000 1.9
+++ LoggerAction.java 28 Dec 2004 16:33:57 -0000 1.10
@@ -32,7 +32,6 @@
public class LoggerAction extends Action {
- Logger logger = Logger.getLogger(LoggerAction.class);
boolean inError = false;
public void begin(ExecutionContext ec, String name, Attributes attributes) {
@@ -53,23 +52,23 @@
String errorMsg = "No 'name' attribute in element " + name + line;
- logger.warn(errorMsg);
+ getLogger().warn(errorMsg);
ec.addError(new ErrorItem(errorMsg));
return;
}
- logger.debug("Logger name is [" + loggerName + "].");
+ getLogger().debug("Logger name is [" + loggerName + "].");
Logger l;
String className = attributes.getValue(CLASS_ATTRIBUTE);
if (Option.isEmpty(className)) {
- logger.debug("Retreiving an instance of org.apache.log4j.Logger.");
+ getLogger().debug("Retreiving an instance of org.apache.log4j.getLogger().");
l = repository.getLogger(loggerName);
} else {
- logger.debug("Desired logger sub-class: [" + className + ']');
+ getLogger().debug("Desired logger sub-class: [" + className + ']');
try {
Class clazz = Loader.loadClass(className);
@@ -78,7 +77,7 @@
l = (Logger) getInstanceMethod.invoke(
null, new Object[] { loggerName });
} catch (Exception oops) {
- logger.error(
+ getLogger().error(
"Could not retrieve category [" + loggerName
+ "]. Reported error follows.", oops);
@@ -89,19 +88,19 @@
boolean additivity =
OptionConverter.toBoolean(
attributes.getValue(ActionConst.ADDITIVITY_ATTRIBUTE), true);
- logger.debug(
+ getLogger().debug(
"Setting [" + l.getName() + "] additivity to [" + additivity + "].");
l.setAdditivity(additivity);
- logger.debug("Pushing logger named [" + loggerName + "].");
+ getLogger().debug("Pushing logger named [" + loggerName + "].");
ec.pushObject(l);
}
public void end(ExecutionContext ec, String e) {
- logger.debug("end() called.");
+ getLogger().debug("end() called.");
if (!inError) {
- logger.debug("Removing logger from stack.");
+ getLogger().debug("Removing logger from stack.");
ec.popObject();
}
}
1.15 +0 -2 logging-log4j/src/java/org/apache/log4j/joran/action/AppenderRefAction.java
Index: AppenderRefAction.java
===================================================================
RCS file: /home/cvs/logging-log4j/src/java/org/apache/log4j/joran/action/AppenderRefAction.java,v
retrieving revision 1.14
retrieving revision 1.15
diff -u -r1.14 -r1.15
--- AppenderRefAction.java 17 Dec 2004 19:58:30 -0000 1.14
+++ AppenderRefAction.java 28 Dec 2004 16:33:57 -0000 1.15
@@ -32,8 +32,6 @@
public class AppenderRefAction extends Action {
- //static final Logger logger = Logger.getLogger(AppenderRefAction.class);
-
boolean inError = false;
public void begin(
1.9 +0 -2 logging-log4j/src/java/org/apache/log4j/joran/action/ConfigurationAction.java
Index: ConfigurationAction.java
===================================================================
RCS file: /home/cvs/logging-log4j/src/java/org/apache/log4j/joran/action/ConfigurationAction.java,v
retrieving revision 1.8
retrieving revision 1.9
diff -u -r1.8 -r1.9
--- ConfigurationAction.java 28 Dec 2004 16:28:59 -0000 1.8
+++ ConfigurationAction.java 28 Dec 2004 16:33:57 -0000 1.9
@@ -21,14 +21,12 @@
import org.apache.joran.ExecutionContext;
import org.apache.joran.action.Action;
-import org.apache.log4j.Logger;
import org.apache.log4j.config.ConfiguratorBase;
import org.apache.log4j.spi.LoggerRepository;
import org.xml.sax.Attributes;
public class ConfigurationAction extends Action {
- static final Logger logger = Logger.getLogger(ConfigurationAction.class);
static final String INTERNAL_DEBUG_ATTR = "debug";
boolean attachment = false;
---------------------------------------------------------------------
To unsubscribe, e-mail: log4j-dev-unsubscribe@logging.apache.org
For additional commands, e-mail: log4j-dev-help@logging.apache.org