Author: markt
Date: Wed May 25 14:35:58 2016
New Revision: 1745496
URL: http://svn.apache.org/viewvc?rev=1745496&view=rev
Log:
Remove unnecessary field
Modified:
tomcat/trunk/java/org/apache/catalina/core/NamingContextListener.java
Modified: tomcat/trunk/java/org/apache/catalina/core/NamingContextListener.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/core/NamingContextListener.java?rev=1745496&r1=1745495&r2=1745496&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/core/NamingContextListener.java
(original)
+++ tomcat/trunk/java/org/apache/catalina/core/NamingContextListener.java Wed
May 25 14:35:58 2016
@@ -78,17 +78,13 @@ import org.apache.tomcat.util.res.String
* @author Remy Maucherat
*/
public class NamingContextListener
- implements LifecycleListener, ContainerListener, PropertyChangeListener {
+ implements LifecycleListener, ContainerListener,
PropertyChangeListener {
private static final Log log =
LogFactory.getLog(NamingContextListener.class);
// ----------------------------------------------------- Instance Variables
-
- protected Log logger = log;
-
-
/**
* Name of the associated naming context.
*/
@@ -217,7 +213,6 @@ public class NamingContextListener
if (container instanceof Context) {
namingResources = ((Context) container).getNamingResources();
- logger = log;
token = ((Context) container).getNamingToken();
} else if (container instanceof Server) {
namingResources = ((Server) container).getGlobalNamingResources();
@@ -251,7 +246,7 @@ public class NamingContextListener
try {
createNamingContext();
} catch (NamingException e) {
- logger.error
+ log.error
(sm.getString("naming.namingContextCreationFailed",
e));
}
@@ -265,7 +260,7 @@ public class NamingContextListener
ContextBindings.bindClassLoader(container, token,
((Context)
container).getLoader().getClassLoader());
} catch (NamingException e) {
- logger.error(sm.getString("naming.bindFailed", e));
+ log.error(sm.getString("naming.bindFailed", e));
}
}
@@ -276,7 +271,7 @@ public class NamingContextListener
ContextBindings.bindClassLoader(container, token,
this.getClass().getClassLoader());
} catch (NamingException e) {
- logger.error(sm.getString("naming.bindFailed", e));
+ log.error(sm.getString("naming.bindFailed", e));
}
if (container instanceof StandardServer) {
((StandardServer) container).setGlobalNamingContext
@@ -703,7 +698,7 @@ public class NamingContextListener
// Ignore because UserTransaction was obviously
// added via ResourceLink
} catch (NamingException e) {
- logger.error(sm.getString("naming.bindFailed", e));
+ log.error(sm.getString("naming.bindFailed", e));
}
}
@@ -713,7 +708,7 @@ public class NamingContextListener
compCtx.bind("Resources",
((Context) container).getResources());
} catch (NamingException e) {
- logger.error(sm.getString("naming.bindFailed", e));
+ log.error(sm.getString("naming.bindFailed", e));
}
}
@@ -786,7 +781,7 @@ public class NamingContextListener
createSubcontexts(envCtx, ejb.getName());
envCtx.bind(ejb.getName(), ref);
} catch (NamingException e) {
- logger.error(sm.getString("naming.bindFailed", e));
+ log.error(sm.getString("naming.bindFailed", e));
}
}
@@ -857,25 +852,25 @@ public class NamingContextListener
} else {
value = constructEnvEntry(env.getType(), env.getValue());
if (value == null) {
- logger.error(sm.getString(
+ log.error(sm.getString(
"naming.invalidEnvEntryType", env.getName()));
}
}
} catch (NumberFormatException e) {
- logger.error(sm.getString("naming.invalidEnvEntryValue",
env.getName()));
+ log.error(sm.getString("naming.invalidEnvEntryValue",
env.getName()));
} catch (IllegalArgumentException e) {
- logger.error(sm.getString("naming.invalidEnvEntryValue",
env.getName()));
+ log.error(sm.getString("naming.invalidEnvEntryValue",
env.getName()));
}
// Binding the object to the appropriate name
if (value != null) {
try {
- if (logger.isDebugEnabled())
- logger.debug(" Adding environment entry " +
env.getName());
+ if (log.isDebugEnabled())
+ log.debug(" Adding environment entry " + env.getName());
createSubcontexts(envCtx, env.getName());
envCtx.bind(env.getName(), value);
} catch (NamingException e) {
- logger.error(sm.getString("naming.invalidEnvEntryValue", e));
+ log.error(sm.getString("naming.invalidEnvEntryValue", e));
}
}
@@ -948,10 +943,10 @@ public class NamingContextListener
wsdlURL = ((Context) container).
getServletContext().
getResource("/" +
service.getWsdlfile());
- logger.debug(" Changing service ref wsdl file for /"
+ log.debug(" Changing service ref wsdl file for /"
+ service.getWsdlfile());
} catch (MalformedURLException e) {
- logger.error(sm.getString("naming.wsdlFailed", e));
+ log.error(sm.getString("naming.wsdlFailed", e));
}
}
if (wsdlURL == null)
@@ -982,10 +977,10 @@ public class NamingContextListener
jaxrpcURL = ((Context) container).
getServletContext().
getResource("/" +
service.getJaxrpcmappingfile());
- logger.debug(" Changing service ref jaxrpc file for /"
+ log.debug(" Changing service ref jaxrpc file for /"
+ service.getJaxrpcmappingfile());
} catch (MalformedURLException e) {
- logger.error(sm.getString("naming.wsdlFailed", e));
+ log.error(sm.getString("naming.wsdlFailed", e));
}
}
if (jaxrpcURL == null)
@@ -1038,14 +1033,14 @@ public class NamingContextListener
}
try {
- if (logger.isDebugEnabled()) {
- logger.debug(" Adding service ref "
+ if (log.isDebugEnabled()) {
+ log.debug(" Adding service ref "
+ service.getName() + " " + ref);
}
createSubcontexts(envCtx, service.getName());
envCtx.bind(service.getName(), ref);
} catch (NamingException e) {
- logger.error(sm.getString("naming.bindFailed", e));
+ log.error(sm.getString("naming.bindFailed", e));
}
}
@@ -1072,14 +1067,14 @@ public class NamingContextListener
ref.add(refAddr);
}
try {
- if (logger.isDebugEnabled()) {
- logger.debug(" Adding resource ref "
+ if (log.isDebugEnabled()) {
+ log.debug(" Adding resource ref "
+ resource.getName() + " " + ref);
}
createSubcontexts(envCtx, resource.getName());
envCtx.bind(resource.getName(), ref);
} catch (NamingException e) {
- logger.error(sm.getString("naming.bindFailed", e));
+ log.error(sm.getString("naming.bindFailed", e));
}
if ("javax.sql.DataSource".equals(ref.getClassName()) &&
@@ -1090,7 +1085,7 @@ public class NamingContextListener
Registry.getRegistry(null,
null).registerComponent(actualResource, on, null);
objectNames.put(resource.getName(), on);
} catch (Exception e) {
- logger.warn(sm.getString("naming.jmxRegistrationFailed", e));
+ log.warn(sm.getString("naming.jmxRegistrationFailed", e));
}
}
@@ -1115,12 +1110,12 @@ public class NamingContextListener
ref.add(refAddr);
}
try {
- if (logger.isDebugEnabled())
+ if (log.isDebugEnabled())
log.debug(" Adding resource env ref " +
resourceEnvRef.getName());
createSubcontexts(envCtx, resourceEnvRef.getName());
envCtx.bind(resourceEnvRef.getName(), ref);
} catch (NamingException e) {
- logger.error(sm.getString("naming.bindFailed", e));
+ log.error(sm.getString("naming.bindFailed", e));
}
}
@@ -1149,12 +1144,12 @@ public class NamingContextListener
"UserTransaction".equals(resourceLink.getName())
? compCtx : envCtx;
try {
- if (logger.isDebugEnabled())
+ if (log.isDebugEnabled())
log.debug(" Adding resource link " + resourceLink.getName());
createSubcontexts(envCtx, resourceLink.getName());
ctx.bind(resourceLink.getName(), ref);
} catch (NamingException e) {
- logger.error(sm.getString("naming.bindFailed", e));
+ log.error(sm.getString("naming.bindFailed", e));
}
}
@@ -1170,7 +1165,7 @@ public class NamingContextListener
try {
envCtx.unbind(name);
} catch (NamingException e) {
- logger.error(sm.getString("naming.unbindFailed", e));
+ log.error(sm.getString("naming.unbindFailed", e));
}
}
@@ -1186,7 +1181,7 @@ public class NamingContextListener
try {
envCtx.unbind(name);
} catch (NamingException e) {
- logger.error(sm.getString("naming.unbindFailed", e));
+ log.error(sm.getString("naming.unbindFailed", e));
}
}
@@ -1202,7 +1197,7 @@ public class NamingContextListener
try {
envCtx.unbind(name);
} catch (NamingException e) {
- logger.error(sm.getString("naming.unbindFailed", e));
+ log.error(sm.getString("naming.unbindFailed", e));
}
}
@@ -1218,7 +1213,7 @@ public class NamingContextListener
try {
envCtx.unbind(name);
} catch (NamingException e) {
- logger.error(sm.getString("naming.unbindFailed", e));
+ log.error(sm.getString("naming.unbindFailed", e));
}
}
@@ -1234,7 +1229,7 @@ public class NamingContextListener
try {
envCtx.unbind(name);
} catch (NamingException e) {
- logger.error(sm.getString("naming.unbindFailed", e));
+ log.error(sm.getString("naming.unbindFailed", e));
}
ObjectName on = objectNames.get(name);
@@ -1255,7 +1250,7 @@ public class NamingContextListener
try {
envCtx.unbind(name);
} catch (NamingException e) {
- logger.error(sm.getString("naming.unbindFailed", e));
+ log.error(sm.getString("naming.unbindFailed", e));
}
}
@@ -1271,7 +1266,7 @@ public class NamingContextListener
try {
envCtx.unbind(name);
} catch (NamingException e) {
- logger.error(sm.getString("naming.unbindFailed", e));
+ log.error(sm.getString("naming.unbindFailed", e));
}
}
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org
