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/09 20:33:25 UTC

svn commit: r1396170 - /logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/appender/FileManager.java

Author: ggregory
Date: Tue Oct  9 18:33:24 2012
New Revision: 1396170

URL: http://svn.apache.org/viewvc?rev=1396170&view=rev
Log:
Make svar and ivars final.

Modified:
    logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/appender/FileManager.java

Modified: logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/appender/FileManager.java
URL: http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/appender/FileManager.java?rev=1396170&r1=1396169&r2=1396170&view=diff
==============================================================================
--- logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/appender/FileManager.java (original)
+++ logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/appender/FileManager.java Tue Oct  9 18:33:24 2012
@@ -31,10 +31,10 @@ import java.nio.channels.FileLock;
  */
 public class FileManager extends OutputStreamManager {
 
-    private static ManagerFactory factory = new FileManagerFactory();
+    private static final ManagerFactory factory = new FileManagerFactory();
 
-    private boolean isAppend;
-    private boolean isLocking;
+    private final boolean isAppend;
+    private final boolean isLocking;
 
     protected FileManager(String fileName, OutputStream os, boolean append, boolean locking) {
         super(os, fileName);