You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2011/09/22 21:12:02 UTC
svn commit: r1174325 - in /tomcat/trunk/java/org/apache/tomcat/util/modeler:
ManagedBean.java modules/MbeansDescriptorsDigesterSource.java
modules/MbeansSource.java
Author: markt
Date: Thu Sep 22 19:12:01 2011
New Revision: 1174325
URL: http://svn.apache.org/viewvc?rev=1174325&view=rev
Log:
Fix a small number of Eclipse warnings.
- Unused code
- Unnecessary null checks
Modified:
tomcat/trunk/java/org/apache/tomcat/util/modeler/ManagedBean.java
tomcat/trunk/java/org/apache/tomcat/util/modeler/modules/MbeansDescriptorsDigesterSource.java
tomcat/trunk/java/org/apache/tomcat/util/modeler/modules/MbeansSource.java
Modified: tomcat/trunk/java/org/apache/tomcat/util/modeler/ManagedBean.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/modeler/ManagedBean.java?rev=1174325&r1=1174324&r2=1174325&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/util/modeler/ManagedBean.java (original)
+++ tomcat/trunk/java/org/apache/tomcat/util/modeler/ManagedBean.java Thu Sep 22 19:12:01 2011
@@ -483,148 +483,141 @@ public class ManagedBean implements java
}
Method getGetter(String aname, BaseModelMBean mbean, Object resource)
- throws AttributeNotFoundException, MBeanException, ReflectionException {
- // TODO: do we need caching ? JMX is for management, it's not supposed to require lots of performance.
- Method m=null; // (Method)getAttMap.get( name );
-
- if( m==null ) {
- AttributeInfo attrInfo = attributes.get(aname);
- // Look up the actual operation to be used
- if (attrInfo == null)
- throw new AttributeNotFoundException(" Cannot find attribute " + aname + " for " + resource);
-
- String getMethod = attrInfo.getGetMethod();
- if (getMethod == null)
- throw new AttributeNotFoundException("Cannot find attribute " + aname + " get method name");
+ throws AttributeNotFoundException, ReflectionException {
- Object object = null;
- NoSuchMethodException exception = null;
+ Method m = null;
+
+ AttributeInfo attrInfo = attributes.get(aname);
+ // Look up the actual operation to be used
+ if (attrInfo == null)
+ throw new AttributeNotFoundException(" Cannot find attribute " + aname + " for " + resource);
+
+ String getMethod = attrInfo.getGetMethod();
+ if (getMethod == null)
+ throw new AttributeNotFoundException("Cannot find attribute " + aname + " get method name");
+
+ Object object = null;
+ NoSuchMethodException exception = null;
+ try {
+ object = mbean;
+ m = object.getClass().getMethod(getMethod, NO_ARGS_PARAM_SIG);
+ } catch (NoSuchMethodException e) {
+ exception = e;
+ }
+ if( m== null && resource != null ) {
try {
- object = mbean;
+ object = resource;
m = object.getClass().getMethod(getMethod, NO_ARGS_PARAM_SIG);
+ exception=null;
} catch (NoSuchMethodException e) {
exception = e;
}
- if( m== null && resource != null ) {
- try {
- object = resource;
- m = object.getClass().getMethod(getMethod, NO_ARGS_PARAM_SIG);
- exception=null;
- } catch (NoSuchMethodException e) {
- exception = e;
- }
- }
- if( exception != null )
- throw new ReflectionException(exception,
- "Cannot find getter method " + getMethod);
- //getAttMap.put( name, m );
}
+ if( exception != null )
+ throw new ReflectionException(exception,
+ "Cannot find getter method " + getMethod);
return m;
}
public Method getSetter(String aname, BaseModelMBean bean, Object resource)
- throws AttributeNotFoundException, MBeanException, ReflectionException {
- // Cache may be needed for getters, but it is a really bad idea for setters, this is far
- // less frequent.
- Method m=null;//(Method)setAttMap.get( name );
-
- if( m==null ) {
- AttributeInfo attrInfo = attributes.get(aname);
- if (attrInfo == null)
- throw new AttributeNotFoundException(" Cannot find attribute " + aname);
-
- // Look up the actual operation to be used
- String setMethod = attrInfo.getSetMethod();
- if (setMethod == null)
- throw new AttributeNotFoundException("Cannot find attribute " + aname + " set method name");
+ throws AttributeNotFoundException, ReflectionException {
- String argType=attrInfo.getType();
+ Method m = null;
- Class<?> signature[] =
- new Class[] { BaseModelMBean.getAttributeClass( argType ) };
-
- Object object = null;
- NoSuchMethodException exception = null;
+ AttributeInfo attrInfo = attributes.get(aname);
+ if (attrInfo == null)
+ throw new AttributeNotFoundException(" Cannot find attribute " + aname);
+
+ // Look up the actual operation to be used
+ String setMethod = attrInfo.getSetMethod();
+ if (setMethod == null)
+ throw new AttributeNotFoundException("Cannot find attribute " + aname + " set method name");
+
+ String argType=attrInfo.getType();
+
+ Class<?> signature[] =
+ new Class[] { BaseModelMBean.getAttributeClass( argType ) };
+
+ Object object = null;
+ NoSuchMethodException exception = null;
+ try {
+ object = bean;
+ m = object.getClass().getMethod(setMethod, signature);
+ } catch (NoSuchMethodException e) {
+ exception = e;
+ }
+ if( m== null && resource != null ) {
try {
- object = bean;
+ object = resource;
m = object.getClass().getMethod(setMethod, signature);
+ exception=null;
} catch (NoSuchMethodException e) {
exception = e;
}
- if( m== null && resource != null ) {
- try {
- object = resource;
- m = object.getClass().getMethod(setMethod, signature);
- exception=null;
- } catch (NoSuchMethodException e) {
- exception = e;
- }
- }
- if( exception != null )
- throw new ReflectionException(exception,
- "Cannot find setter method " + setMethod +
- " " + resource);
- //setAttMap.put( name, m );
}
+ if( exception != null )
+ throw new ReflectionException(exception,
+ "Cannot find setter method " + setMethod +
+ " " + resource);
return m;
}
public Method getInvoke(String aname, Object[] params, String[] signature, BaseModelMBean bean, Object resource)
throws MBeanException, ReflectionException {
+
Method method = null;
- if (method == null) {
- if (params == null)
- params = new Object[0];
- if (signature == null)
- signature = new String[0];
- if (params.length != signature.length)
- throw new RuntimeOperationsException(
- new IllegalArgumentException(
- "Inconsistent arguments and signature"),
- "Inconsistent arguments and signature");
-
- // Acquire the ModelMBeanOperationInfo information for
- // the requested operation
- OperationInfo opInfo = operations.get(aname);
- if (opInfo == null)
- throw new MBeanException(new ServiceNotFoundException(
- "Cannot find operation " + aname),
- "Cannot find operation " + aname);
-
- // Prepare the signature required by Java reflection APIs
- // FIXME - should we use the signature from opInfo?
- Class<?> types[] = new Class[signature.length];
- for (int i = 0; i < signature.length; i++) {
- types[i] = BaseModelMBean.getAttributeClass(signature[i]);
- }
+
+ if (params == null)
+ params = new Object[0];
+ if (signature == null)
+ signature = new String[0];
+ if (params.length != signature.length)
+ throw new RuntimeOperationsException(
+ new IllegalArgumentException(
+ "Inconsistent arguments and signature"),
+ "Inconsistent arguments and signature");
+
+ // Acquire the ModelMBeanOperationInfo information for
+ // the requested operation
+ OperationInfo opInfo = operations.get(aname);
+ if (opInfo == null)
+ throw new MBeanException(new ServiceNotFoundException(
+ "Cannot find operation " + aname),
+ "Cannot find operation " + aname);
+
+ // Prepare the signature required by Java reflection APIs
+ // FIXME - should we use the signature from opInfo?
+ Class<?> types[] = new Class[signature.length];
+ for (int i = 0; i < signature.length; i++) {
+ types[i] = BaseModelMBean.getAttributeClass(signature[i]);
+ }
- // Locate the method to be invoked, either in this MBean itself
- // or in the corresponding managed resource
- // FIXME - Accessible methods in superinterfaces?
- Object object = null;
- Exception exception = null;
- try {
- object = bean;
+ // Locate the method to be invoked, either in this MBean itself
+ // or in the corresponding managed resource
+ // FIXME - Accessible methods in superinterfaces?
+ Object object = null;
+ Exception exception = null;
+ try {
+ object = bean;
+ method = object.getClass().getMethod(aname, types);
+ } catch (NoSuchMethodException e) {
+ exception = e;
+ }
+ try {
+ if ((method == null) && (resource != null)) {
+ object = resource;
method = object.getClass().getMethod(aname, types);
- } catch (NoSuchMethodException e) {
- exception = e;
}
- try {
- if ((method == null) && (resource != null)) {
- object = resource;
- method = object.getClass().getMethod(aname, types);
- }
- } catch (NoSuchMethodException e) {
- exception = e;
- }
- if (method == null) {
- throw new ReflectionException(exception, "Cannot find method "
- + aname + " with this signature");
- }
- // invokeAttMap.put(mkey, method);
+ } catch (NoSuchMethodException e) {
+ exception = e;
}
+ if (method == null) {
+ throw new ReflectionException(exception, "Cannot find method "
+ + aname + " with this signature");
+ }
+
return method;
}
Modified: tomcat/trunk/java/org/apache/tomcat/util/modeler/modules/MbeansDescriptorsDigesterSource.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/modeler/modules/MbeansDescriptorsDigesterSource.java?rev=1174325&r1=1174324&r2=1174325&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/util/modeler/modules/MbeansDescriptorsDigesterSource.java (original)
+++ tomcat/trunk/java/org/apache/tomcat/util/modeler/modules/MbeansDescriptorsDigesterSource.java Thu Sep 22 19:12:01 2011
@@ -44,7 +44,7 @@ public class MbeansDescriptorsDigesterSo
List<ObjectName> mbeans = new ArrayList<ObjectName>();
protected static volatile Digester digester = null;
- protected static Digester createDigester(Registry registry) {
+ protected static Digester createDigester() {
Digester digester = new Digester();
digester.setNamespaceAware(false);
@@ -223,7 +223,7 @@ public class MbeansDescriptorsDigesterSo
InputStream stream = (InputStream) source;
if (digester == null) {
- digester = createDigester(registry);
+ digester = createDigester();
}
ArrayList<ManagedBean> loadedMbeans = new ArrayList<ManagedBean>();
Modified: tomcat/trunk/java/org/apache/tomcat/util/modeler/modules/MbeansSource.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/modeler/modules/MbeansSource.java?rev=1174325&r1=1174324&r2=1174325&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/util/modeler/modules/MbeansSource.java (original)
+++ tomcat/trunk/java/org/apache/tomcat/util/modeler/modules/MbeansSource.java Thu Sep 22 19:12:01 2011
@@ -340,8 +340,7 @@ public class MbeansSource extends Modele
" " + value);
ObjectName oname=new ObjectName(objectName);
// find the type
- if( type==null )
- type=registry.getType( oname, attName );
+ type=registry.getType( oname, attName );
if( type==null ) {
log.info("Can't find attribute " + objectName + " " + attName );
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org