Author: markt
Date: Sat Jan 15 13:53:06 2011
New Revision: 1059330
URL: http://svn.apache.org/viewvc?rev=1059330&view=rev
Log:
Fix FindBugs and Eclipse warnings
Modified:
tomcat/trunk/java/org/apache/catalina/manager/JMXProxyServlet.java
Modified: tomcat/trunk/java/org/apache/catalina/manager/JMXProxyServlet.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/manager/JMXProxyServlet.java?rev=1059330&r1=1059329&r2=1059330&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/manager/JMXProxyServlet.java
(original)
+++ tomcat/trunk/java/org/apache/catalina/manager/JMXProxyServlet.java Sat Jan
15 13:53:06 2011
@@ -52,8 +52,8 @@ public class JMXProxyServlet extends Htt
/**
* MBean server.
*/
- protected MBeanServer mBeanServer = null;
- protected Registry registry;
+ protected transient MBeanServer mBeanServer = null;
+ protected transient Registry registry;
// --------------------------------------------------------- Public Methods
/**
@@ -227,8 +227,8 @@ public class JMXProxyServlet extends Htt
ExceptionUtils.handleThrowable(t);
}
}
- } catch (Exception e) {
- // Ignore
+ } catch (Throwable t) {
+ ExceptionUtils.handleThrowable(t);
}
writer.println();
}
@@ -269,7 +269,13 @@ public class JMXProxyServlet extends Htt
sb.append( value.substring(pos,end));
}
- public boolean isSupported( String type ) {
+ /**
+ * Determines if a type is supported by the {@link JMXProxyServlet}.
+ *
+ * @param type The type to check
+ * @return Always returns <code>true</code>
+ */
+ public boolean isSupported(String type) {
return true;
}
}
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org
