Remove old FIXME comment that was fixed long ago.

Project: http://git-wip-us.apache.org/repos/asf/logging-log4j2/repo
Commit: http://git-wip-us.apache.org/repos/asf/logging-log4j2/commit/08e70e77
Tree: http://git-wip-us.apache.org/repos/asf/logging-log4j2/tree/08e70e77
Diff: http://git-wip-us.apache.org/repos/asf/logging-log4j2/diff/08e70e77

Branch: refs/heads/LOG4J2-435-delete-on-rollover
Commit: 08e70e7710f85a1fc3327b29d7ce53f544cd4d79
Parents: 7273e66
Author: Matt Sicker <[email protected]>
Authored: Sun Nov 1 18:10:40 2015 -0600
Committer: Ralph Goers <[email protected]>
Committed: Fri Nov 20 17:40:13 2015 -0700

----------------------------------------------------------------------
 .../java/org/apache/logging/log4j/core/layout/PatternLayout.java  | 3 ---
 1 file changed, 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/logging-log4j2/blob/08e70e77/log4j-core/src/main/java/org/apache/logging/log4j/core/layout/PatternLayout.java
----------------------------------------------------------------------
diff --git 
a/log4j-core/src/main/java/org/apache/logging/log4j/core/layout/PatternLayout.java
 
b/log4j-core/src/main/java/org/apache/logging/log4j/core/layout/PatternLayout.java
index 33e6d71..cda7915 100644
--- 
a/log4j-core/src/main/java/org/apache/logging/log4j/core/layout/PatternLayout.java
+++ 
b/log4j-core/src/main/java/org/apache/logging/log4j/core/layout/PatternLayout.java
@@ -355,9 +355,6 @@ public final class PatternLayout extends 
AbstractStringLayout {
      */
     public static class Builder implements 
org.apache.logging.log4j.core.util.Builder<PatternLayout> {
 
-        // FIXME: it seems rather redundant to repeat default values (same 
goes for field names)
-        // perhaps introduce a @PluginBuilderAttribute that has no values of 
its own and uses reflection?
-
         @PluginBuilderAttribute
         private String pattern = PatternLayout.DEFAULT_CONVERSION_PATTERN;
 

Reply via email to