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: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to