You are viewing a plain text version of this content. The canonical link for it is here.
Posted to log4j-dev@logging.apache.org by ce...@apache.org on 2005/01/22 19:18:34 UTC

cvs commit: logging-log4j/src/java/org/apache/log4j AppenderSkeleton.java

ceki        2005/01/22 10:18:34

  Modified:    src/java/org/apache/log4j AppenderSkeleton.java
  Log:
  an appender should not flood other appenders with indiscriminate logging.
  
  Revision  Changes    Path
  1.35      +5 -2      logging-log4j/src/java/org/apache/log4j/AppenderSkeleton.java
  
  Index: AppenderSkeleton.java
  ===================================================================
  RCS file: /home/cvs/logging-log4j/src/java/org/apache/log4j/AppenderSkeleton.java,v
  retrieving revision 1.34
  retrieving revision 1.35
  diff -u -r1.34 -r1.35
  --- AppenderSkeleton.java	6 Jan 2005 17:03:11 -0000	1.34
  +++ AppenderSkeleton.java	22 Jan 2005 18:18:33 -0000	1.35
  @@ -210,8 +210,11 @@
         guard = true;
   
         if (closed) {
  -        getLogger().error(
  -          "Attempted to append to closed appender named [{}].", name);
  +        // FIXME: We should not flood other appenders but at the same time
  +        // should output something meaningful (only once though).
  +        // Logging not allowed:
  +        //getLogger().error(
  +        //  "Attempted to append to closed appender named [{}].", name);
   
           return;
         }
  
  
  

---------------------------------------------------------------------
To unsubscribe, e-mail: log4j-dev-unsubscribe@logging.apache.org
For additional commands, e-mail: log4j-dev-help@logging.apache.org