You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@logging.apache.org by gg...@apache.org on 2012/10/10 00:00:07 UTC

svn commit: r1396369 - /logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/plugins/ResolverUtil.java

Author: ggregory
Date: Tue Oct  9 22:00:07 2012
New Revision: 1396369

URL: http://svn.apache.org/viewvc?rev=1396369&view=rev
Log:
Make ivar final.

Modified:
    logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/plugins/ResolverUtil.java

Modified: logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/plugins/ResolverUtil.java
URL: http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/plugins/ResolverUtil.java?rev=1396369&r1=1396368&r2=1396369&view=diff
==============================================================================
--- logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/plugins/ResolverUtil.java (original)
+++ logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/plugins/ResolverUtil.java Tue Oct  9 22:00:07 2012
@@ -448,7 +448,7 @@ public class ResolverUtil<T> {
      * that this test will match the parent type itself if it is presented for matching.
      */
     public static class IsA extends ClassTest {
-        private Class parent;
+        private final Class parent;
 
         /**
          * Constructs an IsA test using the supplied Class as the parent class/interface.
@@ -475,7 +475,7 @@ public class ResolverUtil<T> {
      * A Test that checks to see if each class name ends with the provided suffix.
      */
     public static class NameEndsWith extends ClassTest {
-        private String suffix;
+        private final String suffix;
 
         /**
          * Constructs a NameEndsWith test using the supplied suffix.
@@ -503,7 +503,7 @@ public class ResolverUtil<T> {
      * is, then the test returns true, otherwise false.
      */
     public static class AnnotatedWith extends ClassTest {
-        private Class<? extends Annotation> annotation;
+        private final Class<? extends Annotation> annotation;
 
         /**
          * Constructs an AnnotatedWith test for the specified annotation type.
@@ -532,7 +532,7 @@ public class ResolverUtil<T> {
      * A Test that checks to see if the class name matches.
      */
     public static class NameIs extends ResourceTest {
-        private String name;
+        private final String name;
 
         public NameIs(String name) { this.name = "/" + name; }