You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by tr...@apache.org on 2005/09/22 06:00:20 UTC

svn commit: r290883 - /directory/apacheds/trunk/core/src/main/java/org/apache/ldap/server/jndi/DefaultContextFactoryService.java

Author: trustin
Date: Wed Sep 21 21:00:15 2005
New Revision: 290883

URL: http://svn.apache.org/viewcvs?rev=290883&view=rev
Log:
Fixed afterStartup hook method is invoked even if startup process failed.

Modified:
    directory/apacheds/trunk/core/src/main/java/org/apache/ldap/server/jndi/DefaultContextFactoryService.java

Modified: directory/apacheds/trunk/core/src/main/java/org/apache/ldap/server/jndi/DefaultContextFactoryService.java
URL: http://svn.apache.org/viewcvs/directory/apacheds/trunk/core/src/main/java/org/apache/ldap/server/jndi/DefaultContextFactoryService.java?rev=290883&r1=290882&r2=290883&view=diff
==============================================================================
--- directory/apacheds/trunk/core/src/main/java/org/apache/ldap/server/jndi/DefaultContextFactoryService.java (original)
+++ directory/apacheds/trunk/core/src/main/java/org/apache/ldap/server/jndi/DefaultContextFactoryService.java Wed Sep 21 21:00:15 2005
@@ -197,18 +197,13 @@
         this.startupConfiguration = cfg;
         
         listener.beforeStartup( this );
-        try
-        {
-            initialize();
-            firstStart = createBootstrapEntries();
-            createTestEntries();
-            this.serviceListener = listener;
-            started = true;
-        }
-        finally
-        {
-            listener.afterStartup( this );
-        }
+
+        initialize();
+        firstStart = createBootstrapEntries();
+        createTestEntries();
+        this.serviceListener = listener;
+        started = true;
+        listener.afterStartup( this );
     }
 
     public synchronized void sync() throws NamingException