Comment on @doubt entries
git-svn-id: https://svn.apache.org/repos/asf/logging/log4j/branches/BRANCH_2_0_EXPERIMENTAL/rgoers@949490 13f79535-47bb-0310-9956-ffa450edef68
diff --git a/log4j2-core/src/main/java/org/apache/logging/log4j/core/Logger.java b/log4j2-core/src/main/java/org/apache/logging/log4j/core/Logger.java
index 29feaeb..6a7e523 100644
--- a/log4j2-core/src/main/java/org/apache/logging/log4j/core/Logger.java
+++ b/log4j2-core/src/main/java/org/apache/logging/log4j/core/Logger.java
@@ -28,7 +28,7 @@
import java.util.Map;
/**
- * @doubt All the isEnabled methods could be pushed into a filter interface. Not sure of the utility of having isEnabled
+ * @doubt All the isEnabled methods could be pushed into a filter interface. Not sure of the utility of having isEnabled
* be able to examine the message pattern and parameters.
*/
public class Logger extends AbstractLogger {
@@ -165,7 +165,7 @@
* volatile.
*
* @param config The new Configuration.
- * @doubt lost me on the comment, this.config is declared volatile.
+ * @doubt lost me on the comment, this.config is declared volatile. (RG) Me too.
*/
void updateConfiguration(Configuration config) {
this.config = new PrivateConfig(config, this);
@@ -175,7 +175,7 @@
* @doubt class is not immutable, so it should not be shared between threads.
*/
protected class PrivateConfig {
- /** @doubt public member variables? **/
+ /** @doubt public member variables? (RG) Should be changed. Did this while making the Log4j 1.2 API **/
public final LoggerConfig loggerConfig;
public final Configuration config;
public Level level;
