You are viewing a plain text version of this content. The canonical link for it is here.
Posted to server-dev@james.apache.org by rd...@apache.org on 2008/04/05 12:32:23 UTC

svn commit: r645064 - /james/server/trunk/experimental-seda-imap-function/src/main/java/org/apache/james/experimental/imapserver/ImapHandler.java

Author: rdonkin
Date: Sat Apr  5 03:32:19 2008
New Revision: 645064

URL: http://svn.apache.org/viewvc?rev=645064&view=rev
Log:
Remove unnecessary variable

Modified:
    james/server/trunk/experimental-seda-imap-function/src/main/java/org/apache/james/experimental/imapserver/ImapHandler.java

Modified: james/server/trunk/experimental-seda-imap-function/src/main/java/org/apache/james/experimental/imapserver/ImapHandler.java
URL: http://svn.apache.org/viewvc/james/server/trunk/experimental-seda-imap-function/src/main/java/org/apache/james/experimental/imapserver/ImapHandler.java?rev=645064&r1=645063&r2=645064&view=diff
==============================================================================
--- james/server/trunk/experimental-seda-imap-function/src/main/java/org/apache/james/experimental/imapserver/ImapHandler.java (original)
+++ james/server/trunk/experimental-seda-imap-function/src/main/java/org/apache/james/experimental/imapserver/ImapHandler.java Sat Apr  5 03:32:19 2008
@@ -56,8 +56,6 @@
      * The per-service configuration data that applies to all handlers
      */
     private ImapHandlerConfigurationData theConfigData;
-    
-    private boolean handlerIsUp=false;
 
     /**
      * The session termination status
@@ -113,10 +111,6 @@
      * Resets the handler data to a basic state.
      */
     public void resetHandler() {
-        if (handlerIsUp == false) {
-            return;
-        }
-        handlerIsUp = false;
         
         endSession();
         
@@ -126,21 +120,10 @@
                 session.closeMailbox();
             }
         } catch (Exception e) {
-            getLogger().error("session.cleanUp", e);
+            getLogger().warn("Failed to close mailbox: " + e.getMessage());
+            getLogger().debug(e.getMessage(), e);
         }
         session = null;
-
-        // Clear config data
-        // Removed: we should never clean this one:
-        // theConfigData = null;
-    }
-    
-    protected void initHandler( Socket connection ) throws IOException {
-        handlerIsUp=true;
-        getLogger().debug("Accepting connection for "+connection.toString());
-        // DEBUG
-        
-        super.initHandler(connection);
     }
 
     /**
@@ -163,7 +146,7 @@
             setupLogger(session);
 
             theWatchdog.start();
-            while ( handlerIsUp && !sessionEnded && handleRequest() ) {
+            while ( !sessionEnded && handleRequest() ) {
                 theWatchdog.reset();
             }
             theWatchdog.stop();



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