cvs commit: jakarta-tomcat-4.0/catalina/src/share/org/apache/catalina/logger FileLogger.java

[remm]

remm        01/10/23 16:03:28

  Modified:    catalina/src/share/org/apache/catalina/logger
                        FileLogger.java
  Log:
  - Cosmetic change: it seems to me that there could be a double creation
    of the new writer (which should be harmless, so I won't roll back the change to
    the 4.0 branch).
  
  Revision  Changes    Path
  1.4       +9 -7      
jakarta-tomcat-4.0/catalina/src/share/org/apache/catalina/logger/FileLogger.java
  
  Index: FileLogger.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-tomcat-4.0/catalina/src/share/org/apache/catalina/logger/FileLogger.java,v
  retrieving revision 1.3
  retrieving revision 1.4
  diff -u -r1.3 -r1.4
  --- FileLogger.java   2001/08/27 19:10:25     1.3
  +++ FileLogger.java   2001/10/23 23:03:28     1.4
  @@ -1,7 +1,7 @@
   /*
  - * $Header: 
/home/cvs/jakarta-tomcat-4.0/catalina/src/share/org/apache/catalina/logger/FileLogger.java,v
 1.3 2001/08/27 19:10:25 craigmcc Exp $
  - * $Revision: 1.3 $
  - * $Date: 2001/08/27 19:10:25 $
  + * $Header: 
/home/cvs/jakarta-tomcat-4.0/catalina/src/share/org/apache/catalina/logger/FileLogger.java,v
 1.4 2001/10/23 23:03:28 remm Exp $
  + * $Revision: 1.4 $
  + * $Date: 2001/10/23 23:03:28 $
    *
    * ====================================================================
    *
  @@ -84,7 +84,7 @@
    * optional preceding timestamp.
    *
    * @author Craig R. McClanahan
  - * @version $Revision: 1.3 $ $Date: 2001/08/27 19:10:25 $
  + * @version $Revision: 1.4 $ $Date: 2001/10/23 23:03:28 $
    */
   
   public final class FileLogger
  @@ -279,9 +279,11 @@
           // If the date has changed, switch log files
           if (!date.equals(tsDate)) {
               synchronized (this) {
  -                close();
  -                date = tsDate;
  -                open();
  +                if (!date.equals(tsDate)) {
  +                    close();
  +                    date = tsDate;
  +                    open();
  +                }
               }
           }