Author: markt
Date: Thu Jul 12 21:45:37 2012
New Revision: 1360969
URL: http://svn.apache.org/viewvc?rev=1360969&view=rev
Log:
Fix FindBugs warnings (Serializable classes with non-serializabe fields)
Modified:
tomcat/trunk/java/org/apache/catalina/deploy/InjectionTarget.java
tomcat/trunk/java/org/apache/catalina/deploy/SecurityRoleRef.java
Modified: tomcat/trunk/java/org/apache/catalina/deploy/InjectionTarget.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/deploy/InjectionTarget.java?rev=1360969&r1=1360968&r2=1360969&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/deploy/InjectionTarget.java (original)
+++ tomcat/trunk/java/org/apache/catalina/deploy/InjectionTarget.java Thu Jul
12 21:45:37 2012
@@ -16,10 +16,15 @@
*/
package org.apache.catalina.deploy;
+import java.io.Serializable;
+
/**
* @version $Id$
*/
-public class InjectionTarget {
+public class InjectionTarget implements Serializable {
+
+ private static final long serialVersionUID = 1L;
+
private String targetClass;
private String targetName;
Modified: tomcat/trunk/java/org/apache/catalina/deploy/SecurityRoleRef.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/deploy/SecurityRoleRef.java?rev=1360969&r1=1360968&r2=1360969&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/deploy/SecurityRoleRef.java (original)
+++ tomcat/trunk/java/org/apache/catalina/deploy/SecurityRoleRef.java Thu Jul
12 21:45:37 2012
@@ -18,6 +18,8 @@
package org.apache.catalina.deploy;
+import java.io.Serializable;
+
/**
* <p>Representation of a security role reference for a web application, as
@@ -29,11 +31,12 @@ package org.apache.catalina.deploy;
* @since Tomcat 5.5
*/
-public class SecurityRoleRef {
+public class SecurityRoleRef implements Serializable {
+ private static final long serialVersionUID = 1L;
- // ------------------------------------------------------------- Properties
+ // ------------------------------------------------------------- Properties
/**
* The (required) role name.
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
