You are viewing a plain text version of this content. The canonical link for it is here.
Posted to java-dev@axis.apache.org by sa...@apache.org on 2006/08/08 17:31:31 UTC

svn commit: r429691 [2/2] - in /webservices/axis2/trunk/c: include/axis2_conf.h modules/core/engine/addr_disp.c modules/core/engine/conf.c

Modified: webservices/axis2/trunk/c/modules/core/engine/conf.c
URL: http://svn.apache.org/viewvc/webservices/axis2/trunk/c/modules/core/engine/conf.c?rev=429691&r1=429690&r2=429691&view=diff
==============================================================================
--- webservices/axis2/trunk/c/modules/core/engine/conf.c (original)
+++ webservices/axis2/trunk/c/modules/core/engine/conf.c Tue Aug  8 08:31:30 2006
@@ -1,18 +1,18 @@
 /*
- * Copyright 2004,2005 The Apache Software Foundation.
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *      http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
+* Copyright 2004,2005 The Apache Software Foundation.
+*
+* Licensed under the Apache License, Version 2.0 (the "License");
+* you may not use this file except in compliance with the License.
+* You may obtain a copy of the License at
+*
+*      http://www.apache.org/licenses/LICENSE-2.0
+*
+* Unless required by applicable law or agreed to in writing, software
+* distributed under the License is distributed on an "AS IS" BASIS,
+* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+* See the License for the specific language governing permissions and
+* limitations under the License.
+*/
 
 #include <string.h>
 #include <axis2_conf.h>
@@ -24,10 +24,10 @@
 
 typedef struct axis2_conf_impl axis2_conf_impl_t;
 
-/** 
+/**
  * @brief Engine configuration struct impl
- * Axis2 Engine Configuration impl  
- */  
+ * Axis2 Engine Configuration impl
+ */
 struct axis2_conf_impl
 {
     axis2_conf_t conf;
@@ -42,14 +42,13 @@
      * Field modules
      */
     /*axis2_hash_t *modules;*/
-    /*to store all the availble modules (including version)*/
+    /*to store all the available modules (including version)*/
     axis2_hash_t *all_modules;
     /*to store mapping between default version to module name*/
     axis2_hash_t *name_to_version_map;
-    /** private ArrayList inPhases; */
     axis2_array_list_t *out_phases;
-    axis2_array_list_t *in_faultphases;
-    axis2_array_list_t *out_faultphases;
+    axis2_array_list_t *in_fault_phases;
+    axis2_array_list_t *out_fault_phases;
 
     axis2_array_list_t *in_phases_upto_and_including_post_dispatch;
     axis2_phases_info_t *phases_info;
@@ -57,7 +56,7 @@
     axis2_hash_t *msg_recvs;
     axis2_hash_t *faulty_svcs;
     axis2_hash_t *faulty_modules;
-    axis2_char_t *axis2_repos;
+    axis2_char_t *axis2_repo;
     axis2_dep_engine_t *dep_engine;
     axis2_array_list_t *handlers;
 };
@@ -66,89 +65,89 @@
 
 /***************************** Function prototypes ****************************/
 
-axis2_status_t AXIS2_CALL 
+axis2_status_t AXIS2_CALL
 axis2_conf_free(
-    axis2_conf_t *conf, 
+    axis2_conf_t *conf,
     const axis2_env_t *env);
 
-axis2_status_t AXIS2_CALL 
+axis2_status_t AXIS2_CALL
 axis2_conf_add_svc_grp(
-    axis2_conf_t *conf, 
+    axis2_conf_t *conf,
     const axis2_env_t *env,
     axis2_svc_grp_t *svc_grp);
 
-axis2_svc_grp_t *AXIS2_CALL 
+axis2_svc_grp_t *AXIS2_CALL
 axis2_conf_get_svc_grp(
-    const axis2_conf_t *conf, 
+    const axis2_conf_t *conf,
     const axis2_env_t *env,
-    axis2_char_t *svc_grp_name);
+    const axis2_char_t *svc_grp_name);
 
 axis2_hash_t *AXIS2_CALL
-axis2_conf_get_svc_grps(
-    const axis2_conf_t *conf, 
+axis2_conf_get_all_svc_grps(
+    const axis2_conf_t *conf,
     const axis2_env_t *env);
 
-axis2_status_t AXIS2_CALL 
+axis2_status_t AXIS2_CALL
 axis2_conf_add_svc(
-    axis2_conf_t *conf, 
+    axis2_conf_t *conf,
     const axis2_env_t *env,
     axis2_svc_t *svc);
 
-axis2_svc_t *AXIS2_CALL 
+axis2_svc_t *AXIS2_CALL
 axis2_conf_get_svc(
-    const axis2_conf_t *conf, 
+    const axis2_conf_t *conf,
     const axis2_env_t *env,
-    axis2_char_t *svc_name);
+    const axis2_char_t *svc_name);
 
-axis2_status_t AXIS2_CALL 
+axis2_status_t AXIS2_CALL
 axis2_conf_remove_svc(
-    axis2_conf_t *conf, 
+    axis2_conf_t *conf,
     const axis2_env_t *env,
     const axis2_char_t *name);
 
 axis2_status_t AXIS2_CALL
 axis2_conf_add_param(
-    axis2_conf_t *conf, 
-    const axis2_env_t *env, 
+    axis2_conf_t *conf,
+    const axis2_env_t *env,
     axis2_param_t *param);
 
 axis2_param_t *AXIS2_CALL
 axis2_conf_get_param(
-    const axis2_conf_t *conf, 
+    const axis2_conf_t *conf,
     const axis2_env_t *env,
     const axis2_char_t *name);
 
 axis2_array_list_t *AXIS2_CALL
-axis2_conf_get_params(
-    const axis2_conf_t *conf, 
+axis2_conf_get_all_params(
+    const axis2_conf_t *conf,
     const axis2_env_t *env);
 
 axis2_bool_t AXIS2_CALL
 axis2_conf_is_param_locked(
-    axis2_conf_t *conf, 
+    const axis2_conf_t *conf,
     const axis2_env_t *env,
-    axis2_char_t *param_name);
-                            
+    const axis2_char_t *param_name);
+
 /**
  * To split a given svc name into it's svc grp name and svc name.
- * if the svc name is foo:bar then svc grp name is "foo" and 
- * svc name is "bar" but if the svc name is only the "foo" we asume 
+ * if the svc name is foo:bar then svc grp name is "foo" and
+ * svc name is "bar" but if the svc name is only the "foo" we assume
  * svc grp name is "foo" and svc name is "foo"
  * meaning foo := foo:foo
  * @param svc_name
- * @return svc name and grp name 
+ * @return svc name and grp name
  */
-axis2_status_t 
+axis2_status_t
 split_svc_name(
     const axis2_env_t *env,
-    axis2_char_t *svc_name, 
+    axis2_char_t *svc_name,
     axis2_char_t **svc_name_st);
-      
+
 axis2_transport_in_desc_t *AXIS2_CALL
 axis2_conf_get_transport_in(
     const axis2_conf_t *conf,
     const axis2_env_t *env,
-    axis2_qname_t *qname);
+    const axis2_qname_t *qname);
 
 axis2_status_t AXIS2_CALL
 axis2_conf_add_transport_in(
@@ -160,7 +159,7 @@
 axis2_conf_get_transport_out(
     const axis2_conf_t *conf,
     const axis2_env_t *env,
-    axis2_qname_t *qname);
+    const axis2_qname_t *qname);
 
 axis2_status_t AXIS2_CALL
 axis2_conf_add_transport_out(
@@ -169,23 +168,23 @@
     axis2_transport_out_desc_t *transport);
 
 axis2_hash_t *AXIS2_CALL
-axis2_conf_get_transports_in(
+axis2_conf_get_all_in_transports(
     const axis2_conf_t *conf,
     const axis2_env_t *env);
 
 axis2_hash_t *AXIS2_CALL
-axis2_conf_get_transports_out(
+axis2_conf_get_all_out_transports(
     const axis2_conf_t *conf,
-    const axis2_env_t *env);   
-                                        
+    const axis2_env_t *env);
+
 axis2_module_desc_t *AXIS2_CALL
 axis2_conf_get_module(
     const axis2_conf_t *conf,
     const axis2_env_t *env,
     const axis2_qname_t *qname);
-                                        
+
 axis2_array_list_t *AXIS2_CALL
-axis2_conf_get_engaged_modules(
+axis2_conf_get_all_engaged_modules(
     const axis2_conf_t *conf,
     const axis2_env_t *env);
 
@@ -195,33 +194,33 @@
     const axis2_env_t *env);
 
 axis2_array_list_t *AXIS2_CALL
-axis2_conf_get_outflow(
+axis2_conf_get_out_flow(
     const axis2_conf_t *conf,
     const axis2_env_t *env);
 
 axis2_array_list_t *AXIS2_CALL
-axis2_conf_get_in_faultflow(
+axis2_conf_get_in_fault_flow(
     const axis2_conf_t *conf,
     const axis2_env_t *env);
 
 axis2_array_list_t *AXIS2_CALL
-axis2_conf_get_out_faultflow(
+axis2_conf_get_out_fault_flow(
     const axis2_conf_t *conf,
-    const axis2_env_t *env);                                        
- 
+    const axis2_env_t *env);
+
 axis2_hash_t *AXIS2_CALL
-axis2_conf_get_faulty_svcs(
+axis2_conf_get_all_faulty_svcs(
     const axis2_conf_t *conf,
     const axis2_env_t *env);
 
-axis2_hash_t *AXIS2_CALL 
-axis2_conf_get_faulty_modules(
+axis2_hash_t *AXIS2_CALL
+axis2_conf_get_all_faulty_modules(
     const axis2_conf_t *conf,
-     const axis2_env_t *env);
-    
+    const axis2_env_t *env);
+
 /*to get all the services in the system */
 axis2_hash_t *AXIS2_CALL
-axis2_conf_get_svcs(
+axis2_conf_get_all_svcs(
     const axis2_conf_t *conf,
     const axis2_env_t *env);
 
@@ -241,12 +240,12 @@
     axis2_conf_t *conf,
     const axis2_env_t *env,
     axis2_phases_info_t *phases_info);
-    axis2_status_t AXIS2_CALL
+axis2_status_t AXIS2_CALL
 
 axis2_conf_add_msg_recv(
     axis2_conf_t *conf,
     const axis2_env_t *env,
-    axis2_char_t *key,
+    const axis2_char_t *key,
     axis2_msg_recv_t *msg_recv);
 
 axis2_msg_recv_t *AXIS2_CALL
@@ -267,21 +266,21 @@
     const axis2_env_t *env);
 
 axis2_status_t AXIS2_CALL
-axis2_conf_set_in_faultphases(
+axis2_conf_set_in_fault_phases(
     axis2_conf_t *conf,
     const axis2_env_t *env,
     axis2_array_list_t *list);
 
 axis2_status_t AXIS2_CALL
-axis2_conf_set_out_faultphases(
+axis2_conf_set_out_fault_phases(
     axis2_conf_t *conf,
     const axis2_env_t *env,
     axis2_array_list_t *list);
-    
+
 axis2_hash_t *AXIS2_CALL
-axis2_conf_get_modules(
+axis2_conf_get_all_modules(
     const axis2_conf_t *conf,
-    const axis2_env_t *env); 
+    const axis2_env_t *env);
 
 axis2_status_t AXIS2_CALL
 axis2_conf_add_module(
@@ -292,7 +291,7 @@
 axis2_status_t AXIS2_CALL
 axis2_conf_set_default_dispatchers(
     axis2_conf_t *conf,
-    const axis2_env_t *env); 
+    const axis2_env_t *env);
 
 axis2_status_t AXIS2_CALL
 axis2_conf_set_dispatch_phase(
@@ -313,46 +312,44 @@
     const axis2_env_t *env,
     const axis2_qname_t *module_ref);
 
-axis2_char_t *AXIS2_CALL
-axis2_conf_get_repos(
+const axis2_char_t *AXIS2_CALL
+axis2_conf_get_repo(
     const axis2_conf_t *conf,
     const axis2_env_t *env);
-                            
+
 axis2_status_t AXIS2_CALL
-axis2_conf_set_repos(
+axis2_conf_set_repo(
     axis2_conf_t *conf,
     const axis2_env_t *env,
-    axis2_char_t *repos_path);
-                        
-axis2_char_t *AXIS2_CALL
+    axis2_char_t *repo_path);
+
+const axis2_char_t *AXIS2_CALL
 axis2_conf_get_default_module_version(
-    const axis2_conf_t *conf, 
-    const axis2_env_t *env, 
+    const axis2_conf_t *conf,
+    const axis2_env_t *env,
     const axis2_char_t *module_name);
-                                    
+
 axis2_module_desc_t *AXIS2_CALL
 axis2_conf_get_default_module(
     const axis2_conf_t *conf,
-    const axis2_env_t *env, 
+    const axis2_env_t *env,
     const axis2_char_t *module_name);
-                                
+
 axis2_status_t AXIS2_CALL
 axis2_conf_add_default_module_version(
-    axis2_conf_t *conf, 
+    axis2_conf_t *conf,
     const axis2_env_t *env,
-    axis2_char_t *module_name, 
-    axis2_char_t *module_version);
-                                        
+    const axis2_char_t *module_name,
+    const axis2_char_t *module_version);
+
 axis2_status_t AXIS2_CALL
 axis2_conf_engage_module_with_version(
-    axis2_conf_t *conf, 
+    axis2_conf_t *conf,
     const axis2_env_t *env,
     const axis2_char_t *module_name,
     const axis2_char_t *version_id);
 
-/************************** End of function prototypes ************************/
-
-axis2_conf_t *AXIS2_CALL 
+axis2_conf_t *AXIS2_CALL
 axis2_conf_create (
     const axis2_env_t *env)
 {
@@ -361,182 +358,148 @@
     axis2_phase_t *phase = NULL;
 
     AXIS2_ENV_CHECK(env, NULL);
-    
-   config_impl = (axis2_conf_impl_t *) AXIS2_MALLOC (env->allocator
-      , sizeof(axis2_conf_impl_t));
-   
-   if(NULL == config_impl)
+
+    config_impl = (axis2_conf_impl_t *) AXIS2_MALLOC (env->allocator
+            , sizeof(axis2_conf_impl_t));
+
+    if (NULL == config_impl)
     {
-       AXIS2_ERROR_SET(env->error, AXIS2_ERROR_NO_MEMORY, AXIS2_FAILURE);
+        AXIS2_ERROR_SET(env->error, AXIS2_ERROR_NO_MEMORY, AXIS2_FAILURE);
         return NULL;
     }
-   
+
     config_impl->conf.param_container = NULL;
     config_impl->svc_grps = NULL;
-    /*config_impl->modules = NULL;*/
     config_impl->engaged_modules = NULL;
     config_impl->in_phases_upto_and_including_post_dispatch = NULL;
-   
     config_impl->out_phases = NULL;
-    config_impl->in_faultphases = NULL;
-    config_impl->out_faultphases = NULL;
+    config_impl->in_fault_phases = NULL;
+    config_impl->out_fault_phases = NULL;
     config_impl->phases_info = NULL;
     config_impl->all_svcs = NULL;
     config_impl->msg_recvs = NULL;
     config_impl->faulty_svcs = NULL;
     config_impl->faulty_modules = NULL;
-    config_impl->axis2_repos = NULL;
+    config_impl->axis2_repo = NULL;
     config_impl->dep_engine = NULL;
     config_impl->all_modules = NULL;
     config_impl->name_to_version_map = NULL;
     config_impl->conf.ops = NULL;
-    
-    config_impl->conf.param_container = (axis2_param_container_t *) 
-        axis2_param_container_create(env);      
-   if(NULL == config_impl->conf.param_container)
-   {
-        axis2_conf_free(&(config_impl->conf), env);
-      AXIS2_ERROR_SET(env->error, AXIS2_ERROR_NO_MEMORY, AXIS2_FAILURE);
-        return NULL;
-   }
-    
-    config_impl->svc_grps = axis2_hash_make(env);      
-   if(NULL == config_impl->svc_grps)
-   {
+
+    config_impl->conf.param_container = (axis2_param_container_t *)
+            axis2_param_container_create(env);
+    if (NULL == config_impl->conf.param_container)
+    {
         axis2_conf_free(&(config_impl->conf), env);
-      AXIS2_ERROR_SET(env->error, AXIS2_ERROR_NO_MEMORY, AXIS2_FAILURE);
+        AXIS2_ERROR_SET(env->error, AXIS2_ERROR_NO_MEMORY, AXIS2_FAILURE);
         return NULL;
-   }
+    }
 
-    config_impl->transports_in = axis2_hash_make(env);      
-   if(NULL == config_impl->transports_in)
-   {
+    config_impl->svc_grps = axis2_hash_make(env);
+    if (NULL == config_impl->svc_grps)
+    {
         axis2_conf_free(&(config_impl->conf), env);
-      AXIS2_ERROR_SET(env->error, AXIS2_ERROR_NO_MEMORY, AXIS2_FAILURE);
+        AXIS2_ERROR_SET(env->error, AXIS2_ERROR_NO_MEMORY, AXIS2_FAILURE);
         return NULL;
-   }
-    
-   config_impl->transports_out = axis2_hash_make(env);      
-   if(NULL == config_impl->transports_out)
-   {
+    }
+
+    config_impl->transports_in = axis2_hash_make(env);
+    if (NULL == config_impl->transports_in)
+    {
         axis2_conf_free(&(config_impl->conf), env);
-      AXIS2_ERROR_SET(env->error, AXIS2_ERROR_NO_MEMORY, AXIS2_FAILURE);
+        AXIS2_ERROR_SET(env->error, AXIS2_ERROR_NO_MEMORY, AXIS2_FAILURE);
         return NULL;
-   }
-    
-    /*config_impl->modules = axis2_hash_make(env);      
-   if(NULL == config_impl->modules)
-   {
+    }
+
+    config_impl->transports_out = axis2_hash_make(env);
+    if (NULL == config_impl->transports_out)
+    {
         axis2_conf_free(&(config_impl->conf), env);
-      AXIS2_ERROR_SET(env->error, AXIS2_ERROR_NO_MEMORY, AXIS2_FAILURE);
+        AXIS2_ERROR_SET(env->error, AXIS2_ERROR_NO_MEMORY, AXIS2_FAILURE);
         return NULL;
-   }*/
-    
-    config_impl->engaged_modules = axis2_array_list_create(env, 0);      
-   if(NULL == config_impl->engaged_modules)
-   {
+    }
+
+    config_impl->engaged_modules = axis2_array_list_create(env, 0);
+    if (NULL == config_impl->engaged_modules)
+    {
         axis2_conf_free(&(config_impl->conf), env);
-      AXIS2_ERROR_SET(env->error, AXIS2_ERROR_NO_MEMORY, AXIS2_FAILURE);
+        AXIS2_ERROR_SET(env->error, AXIS2_ERROR_NO_MEMORY, AXIS2_FAILURE);
         return NULL;
-   }
-    
-    config_impl->in_phases_upto_and_including_post_dispatch = 
-        axis2_array_list_create(env, 0);      
-   if(NULL == config_impl->in_phases_upto_and_including_post_dispatch)
-   {
+    }
+
+    config_impl->in_phases_upto_and_including_post_dispatch =
+        axis2_array_list_create(env, 0);
+    if (NULL == config_impl->in_phases_upto_and_including_post_dispatch)
+    {
         axis2_conf_free(&(config_impl->conf), env);
-      AXIS2_ERROR_SET(env->error, AXIS2_ERROR_NO_MEMORY, AXIS2_FAILURE);
+        AXIS2_ERROR_SET(env->error, AXIS2_ERROR_NO_MEMORY, AXIS2_FAILURE);
         return NULL;
-   }
+    }
     else
     {
         phase = axis2_phase_create(env, AXIS2_PHASE_TRANSPORTIN);
-        if(NULL == phase)
+        if (NULL == phase)
         {
             axis2_conf_free(&(config_impl->conf), env);
             return NULL;
         }
         status = AXIS2_ARRAY_LIST_ADD(config_impl->
-            in_phases_upto_and_including_post_dispatch, env, phase);
-        if(AXIS2_FAILURE == status)
+                in_phases_upto_and_including_post_dispatch, env, phase);
+        if (AXIS2_FAILURE == status)
         {
             axis2_conf_free(&(config_impl->conf), env);
             return NULL;
-            
+
         }
         phase = axis2_phase_create(env, AXIS2_PHASE_PRE_DISPATCH);
-        if(NULL == phase)
+        if (NULL == phase)
         {
             axis2_conf_free(&(config_impl->conf), env);
             return NULL;
         }
         status = AXIS2_ARRAY_LIST_ADD(config_impl->
-            in_phases_upto_and_including_post_dispatch, env, phase);
-        if(AXIS2_FAILURE == status)
+                in_phases_upto_and_including_post_dispatch, env, phase);
+        if (AXIS2_FAILURE == status)
         {
             axis2_conf_free(&(config_impl->conf), env);
-            return NULL;   
+            return NULL;
         }
     }
-    
-    /*config_impl->out_phases = axis2_array_list_create(env, 0);      
-   if(NULL == config_impl->out_phases)
-   {
-        axis2_conf_free(&(config_impl->conf), env);
-      AXIS2_ERROR_SET(env->error, AXIS2_ERROR_NO_MEMORY, AXIS2_FAILURE);
-        return NULL;
-   }
-    
-    config_impl->in_faultphases = axis2_array_list_create(env, 0);      
-   if(NULL == config_impl->in_faultphases)
-   {
-        axis2_conf_free(&(config_impl->conf), env);
-      AXIS2_ERROR_SET(env->error, AXIS2_ERROR_NO_MEMORY, AXIS2_FAILURE);
-        return NULL;
-   }
-    
-    config_impl->out_faultphases = axis2_array_list_create(env, 0);      
-   if(NULL == config_impl->out_faultphases)
-   {
-        axis2_conf_free(&(config_impl->conf), env);
-      AXIS2_ERROR_SET(env->error, AXIS2_ERROR_NO_MEMORY, AXIS2_FAILURE);
-        return NULL;
-   }*/
-    
-    config_impl->handlers = axis2_array_list_create(env, 0);      
-   if(NULL == config_impl->handlers)
-   {
+
+    config_impl->handlers = axis2_array_list_create(env, 0);
+    if (NULL == config_impl->handlers)
+    {
         axis2_conf_free(&(config_impl->conf), env);
-      AXIS2_ERROR_SET(env->error, AXIS2_ERROR_NO_MEMORY, AXIS2_FAILURE);
+        AXIS2_ERROR_SET(env->error, AXIS2_ERROR_NO_MEMORY, AXIS2_FAILURE);
         return NULL;
-   }
-    
+    }
+
     config_impl->all_svcs = axis2_hash_make(env);
-    if(NULL == config_impl->all_svcs)
+    if (NULL == config_impl->all_svcs)
     {
         axis2_conf_free(&(config_impl->conf), env);
         AXIS2_ERROR_SET(env->error, AXIS2_ERROR_NO_MEMORY, AXIS2_FAILURE);
         return NULL;
-    } 
-    
+    }
+
     config_impl->msg_recvs = axis2_hash_make(env);
-    if(NULL == config_impl->msg_recvs)
+    if (NULL == config_impl->msg_recvs)
     {
         axis2_conf_free(&(config_impl->conf), env);
         AXIS2_ERROR_SET(env->error, AXIS2_ERROR_NO_MEMORY, AXIS2_FAILURE);
         return NULL;
-    } 
+    }
 
     config_impl->faulty_svcs = axis2_hash_make(env);
-    if(NULL == config_impl->faulty_svcs)
+    if (NULL == config_impl->faulty_svcs)
     {
         axis2_conf_free(&(config_impl->conf), env);
         AXIS2_ERROR_SET(env->error, AXIS2_ERROR_NO_MEMORY, AXIS2_FAILURE);
         return NULL;
-    } 
+    }
 
     config_impl->faulty_modules = axis2_hash_make(env);
-    if(NULL == config_impl->faulty_modules)
+    if (NULL == config_impl->faulty_modules)
     {
         axis2_conf_free(&(config_impl->conf), env);
         AXIS2_ERROR_SET(env->error, AXIS2_ERROR_NO_MEMORY, AXIS2_FAILURE);
@@ -544,7 +507,7 @@
     }
 
     config_impl->all_modules = axis2_hash_make(env);
-    if(NULL == config_impl->all_modules)
+    if (NULL == config_impl->all_modules)
     {
         axis2_conf_free(&(config_impl->conf), env);
         AXIS2_ERROR_SET(env->error, AXIS2_ERROR_NO_MEMORY, AXIS2_FAILURE);
@@ -552,160 +515,158 @@
     }
 
     config_impl->name_to_version_map = axis2_hash_make(env);
-    if(NULL == config_impl->name_to_version_map)
+    if (NULL == config_impl->name_to_version_map)
     {
         axis2_conf_free(&(config_impl->conf), env);
         AXIS2_ERROR_SET(env->error, AXIS2_ERROR_NO_MEMORY, AXIS2_FAILURE);
         return NULL;
     }
-    
+
     config_impl->conf.ops = AXIS2_MALLOC(env->allocator,
-        sizeof(axis2_conf_ops_t));
-    if(NULL == config_impl->conf.ops)
+            sizeof(axis2_conf_ops_t));
+    if (NULL == config_impl->conf.ops)
     {
         axis2_conf_free(&(config_impl->conf), env);
         AXIS2_ERROR_SET(env->error, AXIS2_ERROR_NO_MEMORY, AXIS2_FAILURE);
         return NULL;
     }
-    
-   config_impl->conf.ops->free = axis2_conf_free;
-   config_impl->conf.ops->add_svc_grp = 
+
+    config_impl->conf.ops->free = axis2_conf_free;
+    config_impl->conf.ops->add_svc_grp =
         axis2_conf_add_svc_grp;
-   config_impl->conf.ops->get_svc_grp = 
+    config_impl->conf.ops->get_svc_grp =
         axis2_conf_get_svc_grp;
-    config_impl->conf.ops->get_svc_grps = 
-        axis2_conf_get_svc_grps;
-   config_impl->conf.ops->add_svc = axis2_conf_add_svc;
-   config_impl->conf.ops->get_svc = axis2_conf_get_svc;
-   config_impl->conf.ops->remove_svc = 
+    config_impl->conf.ops->get_all_svc_grps =
+        axis2_conf_get_all_svc_grps;
+    config_impl->conf.ops->add_svc = axis2_conf_add_svc;
+    config_impl->conf.ops->get_svc = axis2_conf_get_svc;
+    config_impl->conf.ops->remove_svc =
         axis2_conf_remove_svc;
-    config_impl->conf.ops->add_param = 
+    config_impl->conf.ops->add_param =
         axis2_conf_add_param;
-   config_impl->conf.ops->get_param = 
+    config_impl->conf.ops->get_param =
         axis2_conf_get_param;
-   config_impl->conf.ops->get_params = 
-        axis2_conf_get_params;
-    config_impl->conf.ops->is_param_locked = 
-            axis2_conf_is_param_locked;
-    config_impl->conf.ops->get_transport_in = 
-            axis2_conf_get_transport_in;
-    config_impl->conf.ops->add_transport_in = 
-            axis2_conf_add_transport_in;    
-    config_impl->conf.ops->get_transport_out = 
-            axis2_conf_get_transport_out;
-    config_impl->conf.ops->add_transport_out = 
-            axis2_conf_add_transport_out;
-    config_impl->conf.ops->get_transports_in = 
-            axis2_conf_get_transports_in;
-    config_impl->conf.ops->get_transports_out = 
-            axis2_conf_get_transports_out;
-    config_impl->conf.ops->get_module = 
-            axis2_conf_get_module;
-    config_impl->conf.ops->get_engaged_modules =
-            axis2_conf_get_engaged_modules;
+    config_impl->conf.ops->get_all_params =
+        axis2_conf_get_all_params;
+    config_impl->conf.ops->is_param_locked =
+        axis2_conf_is_param_locked;
+    config_impl->conf.ops->get_transport_in =
+        axis2_conf_get_transport_in;
+    config_impl->conf.ops->add_transport_in =
+        axis2_conf_add_transport_in;
+    config_impl->conf.ops->get_transport_out =
+        axis2_conf_get_transport_out;
+    config_impl->conf.ops->add_transport_out =
+        axis2_conf_add_transport_out;
+    config_impl->conf.ops->get_all_in_transports =
+        axis2_conf_get_all_in_transports;
+    config_impl->conf.ops->get_all_out_transports =
+        axis2_conf_get_all_out_transports;
+    config_impl->conf.ops->get_module =
+        axis2_conf_get_module;
+    config_impl->conf.ops->get_all_engaged_modules =
+        axis2_conf_get_all_engaged_modules;
     config_impl->conf.ops->get_in_phases_upto_and_including_post_dispatch =
-            axis2_conf_get_in_phases_upto_and_including_post_dispatch;
-    config_impl->conf.ops->get_outflow =
-            axis2_conf_get_outflow;
-    config_impl->conf.ops->get_in_faultflow =
-            axis2_conf_get_in_faultflow;
-    config_impl->conf.ops->get_out_faultflow =
-            axis2_conf_get_out_faultflow;
-    config_impl->conf.ops->get_faulty_svcs =
-            axis2_conf_get_faulty_svcs;
-    config_impl->conf.ops->get_faulty_modules =
-            axis2_conf_get_faulty_modules;
-    config_impl->conf.ops->get_svcs =
-            axis2_conf_get_svcs;
+        axis2_conf_get_in_phases_upto_and_including_post_dispatch;
+    config_impl->conf.ops->get_out_flow =
+        axis2_conf_get_out_flow;
+    config_impl->conf.ops->get_in_fault_flow =
+        axis2_conf_get_in_fault_flow;
+    config_impl->conf.ops->get_out_fault_flow =
+        axis2_conf_get_out_fault_flow;
+    config_impl->conf.ops->get_all_faulty_svcs =
+        axis2_conf_get_all_faulty_svcs;
+    config_impl->conf.ops->get_all_faulty_modules =
+        axis2_conf_get_all_faulty_modules;
+    config_impl->conf.ops->get_all_svcs =
+        axis2_conf_get_all_svcs;
     config_impl->conf.ops->is_engaged =
-            axis2_conf_is_engaged;
+        axis2_conf_is_engaged;
     config_impl->conf.ops->get_phases_info =
-            axis2_conf_get_phases_info;
+        axis2_conf_get_phases_info;
     config_impl->conf.ops->set_phases_info =
-            axis2_conf_set_phases_info;
+        axis2_conf_set_phases_info;
     config_impl->conf.ops->add_msg_recv =
-            axis2_conf_add_msg_recv;
+        axis2_conf_add_msg_recv;
     config_impl->conf.ops->get_msg_recv =
-            axis2_conf_get_msg_recv;
+        axis2_conf_get_msg_recv;
     config_impl->conf.ops->set_out_phases =
-            axis2_conf_set_out_phases;
+        axis2_conf_set_out_phases;
     config_impl->conf.ops->get_out_phases =
-            axis2_conf_get_out_phases;
-    config_impl->conf.ops->set_in_faultphases =
-            axis2_conf_set_in_faultphases;
-    config_impl->conf.ops->set_out_faultphases =
-            axis2_conf_set_out_faultphases;
-    config_impl->conf.ops->get_modules =
-            axis2_conf_get_modules;
+        axis2_conf_get_out_phases;
+    config_impl->conf.ops->set_in_fault_phases =
+        axis2_conf_set_in_fault_phases;
+    config_impl->conf.ops->set_out_fault_phases =
+        axis2_conf_set_out_fault_phases;
+    config_impl->conf.ops->get_all_modules =
+        axis2_conf_get_all_modules;
     config_impl->conf.ops->add_module =
-            axis2_conf_add_module;
-    config_impl->conf.ops->set_default_dispatchers = 
+        axis2_conf_add_module;
+    config_impl->conf.ops->set_default_dispatchers =
         axis2_conf_set_default_dispatchers;
     config_impl->conf.ops->set_dispatch_phase = axis2_conf_set_dispatch_phase;
-    config_impl->conf.ops->get_repos = axis2_conf_get_repos;
-    config_impl->conf.ops->set_repos = axis2_conf_set_repos;
+    config_impl->conf.ops->get_repo = axis2_conf_get_repo;
+    config_impl->conf.ops->set_repo = axis2_conf_set_repo;
     config_impl->conf.ops->engage_module = axis2_conf_engage_module;
     config_impl->conf.ops->set_dep_engine = axis2_conf_set_dep_engine;
-    
-    config_impl->conf.ops->get_default_module_version = 
-                        axis2_conf_get_default_module_version;
+
+    config_impl->conf.ops->get_default_module_version =
+        axis2_conf_get_default_module_version;
     config_impl->conf.ops->get_default_module = axis2_conf_get_default_module;
-    config_impl->conf.ops->add_default_module_version = 
-                        axis2_conf_add_default_module_version;
-    config_impl->conf.ops->engage_module_with_version = 
-                        axis2_conf_engage_module_with_version;
-    
-   return &(config_impl->conf);   
-}   
+    config_impl->conf.ops->add_default_module_version =
+        axis2_conf_add_default_module_version;
+    config_impl->conf.ops->engage_module_with_version =
+        axis2_conf_engage_module_with_version;
 
-/**********************Start of op impls********************************/
+    return &(config_impl->conf);
+}
 
-axis2_status_t AXIS2_CALL 
+axis2_status_t AXIS2_CALL
 axis2_conf_free (
-    axis2_conf_t *conf, 
+    axis2_conf_t *conf,
     const axis2_env_t *env)
 {
     axis2_conf_impl_t *config_impl = NULL;
     axis2_status_t status = AXIS2_SUCCESS;
-    
-   AXIS2_ENV_CHECK(env, AXIS2_FAILURE);
+
+    AXIS2_ENV_CHECK(env, AXIS2_FAILURE);
     config_impl = AXIS2_INTF_TO_IMPL(conf);
-    
-   if(NULL != conf->param_container)
+
+    if (NULL != conf->param_container)
     {
         AXIS2_PARAM_CONTAINER_FREE(conf->param_container, env);
         conf->param_container = NULL;
     }
-     
-    if(config_impl->svc_grps)
+
+    if (config_impl->svc_grps)
     {
         axis2_hash_index_t *hi = NULL;
         void *val = NULL;
         for (hi = axis2_hash_first (config_impl->svc_grps, env); hi;
-                 hi = axis2_hash_next ( env, hi))
+                hi = axis2_hash_next ( env, hi))
         {
-            axis2_svc_grp_t *svc_grp = NULL;
+            axis2_svc_grp_t * svc_grp = NULL;
             axis2_hash_this (hi, NULL, NULL, &val);
             svc_grp = (axis2_svc_grp_t *) val;
             if (svc_grp)
-               AXIS2_SVC_GRP_FREE (svc_grp, env);
-            
+                AXIS2_SVC_GRP_FREE (svc_grp, env);
+
             val = NULL;
             svc_grp = NULL;
-               
+
         }
         axis2_hash_free(config_impl->svc_grps, env);
         config_impl->svc_grps = NULL;
     }
-    
-    if(config_impl->transports_in)
+
+    if (config_impl->transports_in)
     {
         axis2_hash_index_t *hi = NULL;
         void *val = NULL;
         for (hi = axis2_hash_first (config_impl->transports_in, env); hi;
-                 hi = axis2_hash_next ( env, hi))
+                hi = axis2_hash_next ( env, hi))
         {
-            axis2_transport_in_desc_t *transport_in = NULL;
+            axis2_transport_in_desc_t * transport_in = NULL;
 
             axis2_hash_this (hi, NULL, NULL, &val);
             transport_in = (axis2_transport_in_desc_t *) val;
@@ -719,15 +680,15 @@
         axis2_hash_free(config_impl->transports_in, env);
         config_impl->transports_in = NULL;
     }
-    
-    if(config_impl->transports_out)
+
+    if (config_impl->transports_out)
     {
         axis2_hash_index_t *hi = NULL;
         void *val = NULL;
         for (hi = axis2_hash_first (config_impl->transports_out, env); hi;
-                 hi = axis2_hash_next ( env, hi))
+                hi = axis2_hash_next ( env, hi))
         {
-            axis2_transport_out_desc_t *transport_out = NULL;
+            axis2_transport_out_desc_t * transport_out = NULL;
             axis2_hash_this (hi, NULL, NULL, &val);
             transport_out = (axis2_transport_out_desc_t *) val;
             if (transport_out)
@@ -735,45 +696,22 @@
                 AXIS2_TRANSPORT_OUT_DESC_FREE (transport_out, env);
                 transport_out = NULL;
             }
-            
+
             val = NULL;
-               
+
         }
         axis2_hash_free(config_impl->transports_out, env);
         config_impl->transports_out = NULL;
     }
-    
-    /*if(config_impl->modules)
-    {
-        axis2_hash_index_t *hi = NULL;
-        void *val = NULL;
-        for (hi = axis2_hash_first (config_impl->modules, env); hi;
-                 hi = axis2_hash_next ( env, hi))
-        {
-            axis2_module_desc_t *module_desc = NULL;
 
-            axis2_hash_this (hi, NULL, NULL, &val);
-            module_desc = (axis2_module_desc_t *) val;
-            if (module_desc)
-            {
-                AXIS2_MODULE_DESC_FREE (module_desc, env);
-                module_desc = NULL;
-            }
-            
-            val = NULL;
-        }
-        axis2_hash_free(config_impl->modules, env);
-        config_impl->modules = NULL;
-    }*/
-    
-    if(config_impl->all_modules)
+    if (config_impl->all_modules)
     {
         axis2_hash_index_t *hi = NULL;
         void *val = NULL;
         for (hi = axis2_hash_first (config_impl->all_modules, env); hi;
-                 hi = axis2_hash_next ( env, hi))
+                hi = axis2_hash_next ( env, hi))
         {
-            axis2_module_desc_t *module_desc = NULL;
+            axis2_module_desc_t * module_desc = NULL;
 
             axis2_hash_this (hi, NULL, NULL, &val);
             module_desc = (axis2_module_desc_t *) val;
@@ -782,21 +720,21 @@
                 AXIS2_MODULE_DESC_FREE (module_desc, env);
                 module_desc = NULL;
             }
-            
+
             val = NULL;
         }
         axis2_hash_free(config_impl->all_modules, env);
         config_impl->all_modules = NULL;
     }
-    
-    if(config_impl->name_to_version_map)
+
+    if (config_impl->name_to_version_map)
     {
         axis2_hash_index_t *hi = NULL;
         void *val = NULL;
         for (hi = axis2_hash_first (config_impl->name_to_version_map, env); hi;
-                 hi = axis2_hash_next ( env, hi))
+                hi = axis2_hash_next ( env, hi))
         {
-            axis2_char_t *module_ver = NULL;
+            axis2_char_t * module_ver = NULL;
 
             axis2_hash_this (hi, NULL, NULL, &val);
             module_ver = (axis2_char_t *) val;
@@ -805,87 +743,87 @@
                 AXIS2_FREE(env->allocator, module_ver);
                 module_ver = NULL;
             }
-            
+
             val = NULL;
         }
         axis2_hash_free(config_impl->name_to_version_map, env);
         config_impl->name_to_version_map = NULL;
     }
-    
-    if(config_impl->engaged_modules)
+
+    if (config_impl->engaged_modules)
     {
         int i = 0;
-        for (i = 0; i < AXIS2_ARRAY_LIST_SIZE(config_impl->engaged_modules, 
-            env); i++)
+        for (i = 0; i < AXIS2_ARRAY_LIST_SIZE(config_impl->engaged_modules,
+                env); i++)
         {
             axis2_module_desc_t *module_desc = NULL;
-            module_desc = (axis2_module_desc_t *) 
-                AXIS2_ARRAY_LIST_GET(config_impl->engaged_modules, env, i);
+            module_desc = (axis2_module_desc_t *)
+                    AXIS2_ARRAY_LIST_GET(config_impl->engaged_modules, env, i);
             if (module_desc)
-               AXIS2_MODULE_DESC_FREE (module_desc, env);
-            
+                AXIS2_MODULE_DESC_FREE (module_desc, env);
+
             module_desc = NULL;
-        }            
+        }
         AXIS2_ARRAY_LIST_FREE(config_impl->engaged_modules, env);
         config_impl->engaged_modules = NULL;
     }
-    
-    if(config_impl->out_phases)
+
+    if (config_impl->out_phases)
     {
         int i = 0;
         for (i = 0; i < AXIS2_ARRAY_LIST_SIZE(config_impl->out_phases, env); i++)
         {
             axis2_phase_t *phase = NULL;
-            phase = (axis2_phase_t *) 
-                AXIS2_ARRAY_LIST_GET(config_impl->out_phases, env, i);
+            phase = (axis2_phase_t *)
+                    AXIS2_ARRAY_LIST_GET(config_impl->out_phases, env, i);
             if (phase)
-               AXIS2_PHASE_FREE (phase, env);
-            
+                AXIS2_PHASE_FREE (phase, env);
+
             phase = NULL;
-               
+
         }
         AXIS2_ARRAY_LIST_FREE(config_impl->out_phases, env);
         config_impl->out_phases = NULL;
     }
-    
-    if(config_impl->in_faultphases)
+
+    if (config_impl->in_fault_phases)
     {
         int i = 0;
-        for (i = 0; i < AXIS2_ARRAY_LIST_SIZE(config_impl->in_faultphases, env); i++)
+        for (i = 0; i < AXIS2_ARRAY_LIST_SIZE(config_impl->in_fault_phases, env); i++)
         {
             axis2_phase_t *phase = NULL;
-            phase = (axis2_phase_t *) 
-                AXIS2_ARRAY_LIST_GET(config_impl->in_faultphases, env, i);
+            phase = (axis2_phase_t *)
+                    AXIS2_ARRAY_LIST_GET(config_impl->in_fault_phases, env, i);
             if (phase)
-               AXIS2_PHASE_FREE (phase, env);
-            
+                AXIS2_PHASE_FREE (phase, env);
+
             phase = NULL;
-               
+
         }
-        AXIS2_ARRAY_LIST_FREE(config_impl->in_faultphases, env);
-        config_impl->in_faultphases = NULL;
+        AXIS2_ARRAY_LIST_FREE(config_impl->in_fault_phases, env);
+        config_impl->in_fault_phases = NULL;
     }
-    
-    if(config_impl->out_faultphases)
+
+    if (config_impl->out_fault_phases)
     {
         int i = 0;
-        for (i = 0; i < AXIS2_ARRAY_LIST_SIZE(config_impl->out_faultphases, env); i++)
+        for (i = 0; i < AXIS2_ARRAY_LIST_SIZE(config_impl->out_fault_phases, env); i++)
         {
             axis2_phase_t *phase = NULL;
-            phase = (axis2_phase_t *) 
-                AXIS2_ARRAY_LIST_GET(config_impl->out_faultphases, env, i);
-            
+            phase = (axis2_phase_t *)
+                    AXIS2_ARRAY_LIST_GET(config_impl->out_fault_phases, env, i);
+
             if (phase)
-               AXIS2_PHASE_FREE (phase, env);
-            
+                AXIS2_PHASE_FREE (phase, env);
+
             phase = NULL;
-               
+
         }
-        AXIS2_ARRAY_LIST_FREE(config_impl->out_faultphases, env);
-        config_impl->out_faultphases = NULL;
+        AXIS2_ARRAY_LIST_FREE(config_impl->out_fault_phases, env);
+        config_impl->out_fault_phases = NULL;
     }
-    
-    if(config_impl->in_phases_upto_and_including_post_dispatch)
+
+    if (config_impl->in_phases_upto_and_including_post_dispatch)
     {
         int i = 0;
         for (i = 0; i < AXIS2_ARRAY_LIST_SIZE(config_impl->
@@ -893,32 +831,32 @@
         {
             axis2_phase_t *phase = NULL;
             phase = (axis2_phase_t *) AXIS2_ARRAY_LIST_GET(config_impl->
-                in_phases_upto_and_including_post_dispatch, env, i);
-            
+                    in_phases_upto_and_including_post_dispatch, env, i);
+
             if (phase)
-               AXIS2_PHASE_FREE (phase, env);
+                AXIS2_PHASE_FREE (phase, env);
             phase = NULL;
-               
+
         }
         AXIS2_ARRAY_LIST_FREE(config_impl->
-            in_phases_upto_and_including_post_dispatch, env);
+                in_phases_upto_and_including_post_dispatch, env);
         config_impl->in_phases_upto_and_including_post_dispatch = NULL;
     }
-    
-    if(config_impl->all_svcs)
+
+    if (config_impl->all_svcs)
     {
         axis2_hash_free(config_impl->all_svcs, env);
         config_impl->all_svcs = NULL;
     }
-    
-    if(config_impl->msg_recvs)
+
+    if (config_impl->msg_recvs)
     {
         axis2_hash_index_t *hi = NULL;
         void *val = NULL;
         for (hi = axis2_hash_first (config_impl->msg_recvs, env); hi;
-                 hi = axis2_hash_next ( env, hi))
+                hi = axis2_hash_next ( env, hi))
         {
-            axis2_msg_recv_t *msg_recv = NULL;
+            axis2_msg_recv_t * msg_recv = NULL;
             axis2_hash_this (hi, NULL, NULL, &val);
             msg_recv = (axis2_msg_recv_t *) val;
             if (msg_recv)
@@ -926,88 +864,88 @@
                 AXIS2_MSG_RECV_FREE(msg_recv, env);
                 msg_recv = NULL;
             }
-            
+
             val = NULL;
         }
         axis2_hash_free(config_impl->msg_recvs, env);
         config_impl->msg_recvs = NULL;
     }
-    
-    if(config_impl->faulty_svcs)
+
+    if (config_impl->faulty_svcs)
     {
         axis2_hash_free(config_impl->faulty_svcs, env);
         config_impl->faulty_svcs = NULL;
     }
-    
-    if(config_impl->faulty_modules)
+
+    if (config_impl->faulty_modules)
     {
         axis2_hash_index_t *hi = NULL;
         void *val = NULL;
         for (hi = axis2_hash_first (config_impl->faulty_modules, env); hi;
-                 hi = axis2_hash_next ( env, hi))
+                hi = axis2_hash_next ( env, hi))
         {
-            axis2_module_desc_t *module_desc = NULL;
+            axis2_module_desc_t * module_desc = NULL;
             axis2_hash_this (hi, NULL, NULL, &val);
             module_desc = (axis2_module_desc_t *) val;
             if (module_desc)
-               AXIS2_MODULE_DESC_FREE (module_desc, env);
-            
+                AXIS2_MODULE_DESC_FREE (module_desc, env);
+
             val = NULL;
             module_desc = NULL;
-               
+
         }
         axis2_hash_free(config_impl->faulty_modules, env);
         config_impl->faulty_modules = NULL;
     }
-    
-   if(NULL != config_impl->dep_engine)
+
+    if (NULL != config_impl->dep_engine)
     {
         AXIS2_DEP_ENGINE_FREE(config_impl->dep_engine, env);
-        config_impl->dep_engine= NULL;
+        config_impl->dep_engine = NULL;
     }
-    
-    if(config_impl->handlers)
+
+    if (config_impl->handlers)
     {
         int i = 0;
         for (i = 0; i < AXIS2_ARRAY_LIST_SIZE(config_impl->handlers, env); i++)
         {
             axis2_handler_desc_t *handler_desc = NULL;
-            handler_desc = (axis2_handler_desc_t *) 
-                AXIS2_ARRAY_LIST_GET(config_impl->handlers, env, i);
+            handler_desc = (axis2_handler_desc_t *)
+                    AXIS2_ARRAY_LIST_GET(config_impl->handlers, env, i);
             if (handler_desc)
-               AXIS2_HANDLER_FREE (handler_desc, env);
-            
+                AXIS2_HANDLER_FREE (handler_desc, env);
+
             handler_desc = NULL;
-               
+
         }
         AXIS2_ARRAY_LIST_FREE(config_impl->handlers, env);
         config_impl->handlers = NULL;
     }
-    
-    if(config_impl->axis2_repos)
+
+    if (config_impl->axis2_repo)
     {
-        AXIS2_FREE(env->allocator, config_impl->axis2_repos);
-        config_impl->axis2_repos = NULL;
+        AXIS2_FREE(env->allocator, config_impl->axis2_repo);
+        config_impl->axis2_repo = NULL;
     }
 
-    if(NULL != conf->ops)
+    if (NULL != conf->ops)
     {
-      AXIS2_FREE(env->allocator, conf->ops);
+        AXIS2_FREE(env->allocator, conf->ops);
         conf->ops = NULL;
     }
-    
-    if(config_impl)
+
+    if (config_impl)
     {
-       AXIS2_FREE(env->allocator, config_impl);
+        AXIS2_FREE(env->allocator, config_impl);
         config_impl = NULL;
     }
-    
-   return status;
+
+    return status;
 }
 
-axis2_status_t AXIS2_CALL 
+axis2_status_t AXIS2_CALL
 axis2_conf_add_svc_grp(
-    axis2_conf_t *conf, 
+    axis2_conf_t *conf,
     const axis2_env_t *env,
     axis2_svc_grp_t *svc_grp)
 {
@@ -1017,122 +955,106 @@
     axis2_char_t *svc_name = NULL;
     axis2_char_t *svc_grp_name = NULL;
     int k = 0;
-        
+
     AXIS2_ENV_CHECK(env, AXIS2_FAILURE);
     AXIS2_PARAM_CHECK(env->error, svc_grp, AXIS2_FAILURE);
-    
+
     config_impl = AXIS2_INTF_TO_IMPL(conf);
     svcs = AXIS2_SVC_GRP_GET_SVCS(svc_grp, env);
-    if(!config_impl->all_svcs)
+    if (!config_impl->all_svcs)
     {
         config_impl->all_svcs = axis2_hash_make(env);
-        if(!config_impl->all_svcs)
+        if (!config_impl->all_svcs)
             return AXIS2_FAILURE;
     }
     k = axis2_hash_count(svcs);
     index_i = axis2_hash_first (svcs, env);
-    while(index_i)
+    while (index_i)
     {
         void *value = NULL;
         axis2_svc_t *desc = NULL;
         axis2_char_t *svc_name2 = NULL;
         axis2_qname_t *svc_qname = NULL;
-        
+
         axis2_hash_this (index_i, NULL, NULL, &value);
         desc = (axis2_svc_t *) value;
         svc_qname = AXIS2_SVC_GET_QNAME(desc, env);
         svc_name = AXIS2_QNAME_GET_LOCALPART(svc_qname, env);
-        
-        svc_name2 = axis2_hash_get(config_impl->all_svcs, svc_name, 
+
+        svc_name2 = axis2_hash_get(config_impl->all_svcs, svc_name,
                 AXIS2_HASH_KEY_STRING);
         /* no two service names deployed in the engine can be same */
-        if(NULL != svc_name2)
+        if (NULL != svc_name2)
         {
-            AXIS2_ERROR_SET(env->error, 
-                AXIS2_ERROR_TWO_SVCS_CANNOT_HAVE_SAME_NAME, AXIS2_FAILURE);
-            return AXIS2_FAILURE;            
+            AXIS2_ERROR_SET(env->error,
+                    AXIS2_ERROR_TWO_SVCS_CANNOT_HAVE_SAME_NAME, AXIS2_FAILURE);
+            return AXIS2_FAILURE;
         }
         index_i = axis2_hash_next(env, index_i);
     }
-   
+
     svcs = AXIS2_SVC_GRP_GET_SVCS(svc_grp, env);
     index_i = axis2_hash_first (svcs, env);
-    while(NULL != index_i)
+    while (NULL != index_i)
     {
         void *value = NULL;
         axis2_svc_t *desc = NULL;
-        
+
         axis2_hash_this (index_i, NULL, NULL, &value);
         desc = (axis2_svc_t *) value;
         svc_name = AXIS2_QNAME_GET_LOCALPART(AXIS2_SVC_GET_QNAME(desc, env), env);
         axis2_hash_set(config_impl->all_svcs, svc_name, AXIS2_HASH_KEY_STRING,
-            desc);
-        /* notifyObservers(AxisEvent.SERVICE_DEPLOY ,description); */
+                desc);
         index_i = axis2_hash_next(env, index_i);
     }
-    
-    /*size = AXIS2_ARRAY_LIST_SIZE(config_impl->engaged_modules, env);
-    while(i < size)
-    {
-        axis2_qname_t *module_desc = NULL;
-        
-        module_desc = (axis2_qname_t *) AXIS2_ARRAY_LIST_GET(config_impl->
-            engaged_modules, env, i);
-        status = AXIS2_SVC_GRP_ENGAGE_MODULE_TO_GRP(svc_grp, env, module_desc);
-        if(AXIS2_SUCCESS != status)
-        {
-            return status;
-        }
-        i++;    
-    }*/
 
     svc_grp_name = AXIS2_SVC_GRP_GET_NAME(svc_grp, env);
-    if(!config_impl->svc_grps)
+    if (!config_impl->svc_grps)
     {
         config_impl->svc_grps = axis2_hash_make(env);
-        if(!config_impl->svc_grps)
+        if (!config_impl->svc_grps)
             return AXIS2_FAILURE;
     }
-    axis2_hash_set (config_impl->svc_grps, svc_grp_name, 
-        AXIS2_HASH_KEY_STRING, svc_grp);
-   
-   return AXIS2_SUCCESS;
+    axis2_hash_set (config_impl->svc_grps, svc_grp_name,
+            AXIS2_HASH_KEY_STRING, svc_grp);
+
+    return AXIS2_SUCCESS;
 }
 
-axis2_svc_grp_t *AXIS2_CALL 
+axis2_svc_grp_t *AXIS2_CALL
 axis2_conf_get_svc_grp(
-    const axis2_conf_t *conf, 
+    const axis2_conf_t *conf,
     const axis2_env_t *env,
-    axis2_char_t *svc_grp_name)
+    const axis2_char_t *svc_grp_name)
 {
     axis2_conf_impl_t *config_impl = NULL;
-   AXIS2_ENV_CHECK(env, NULL);
-   AXIS2_PARAM_CHECK(env->error, svc_grp_name, NULL);
-    
+    AXIS2_ENV_CHECK(env, NULL);
+    AXIS2_PARAM_CHECK(env->error, svc_grp_name, NULL);
+
     config_impl = AXIS2_INTF_TO_IMPL(conf);
-    if(!config_impl->svc_grps)
+    if (!config_impl->svc_grps)
     {
-        AXIS2_ERROR_SET(env->error, AXIS2_ERROR_INVALID_STATE_CONF, 
-            AXIS2_FAILURE);
+        AXIS2_ERROR_SET(env->error, AXIS2_ERROR_INVALID_STATE_CONF,
+                AXIS2_FAILURE);
         return NULL;
     }
-   return (axis2_svc_grp_t *) (axis2_hash_get (config_impl->svc_grps, 
+    return (axis2_svc_grp_t *) (axis2_hash_get (config_impl->svc_grps,
             svc_grp_name, AXIS2_HASH_KEY_STRING));
 }
 
 axis2_hash_t *AXIS2_CALL
-axis2_conf_get_svc_grps(
-    const axis2_conf_t *conf, 
-    const axis2_env_t *env) 
+axis2_conf_get_all_svc_grps(
+    const axis2_conf_t *conf,
+    const axis2_env_t *env)
 {
     AXIS2_ENV_CHECK(env, NULL);
     return AXIS2_INTF_TO_IMPL(conf)->svc_grps;
 }
 
-axis2_status_t AXIS2_CALL 
+axis2_status_t AXIS2_CALL
 axis2_conf_add_svc(
-    axis2_conf_t *conf, 
-    const axis2_env_t *env, 
+    axis2_conf_t *conf,
+    const axis2_env_t *env,
     axis2_svc_t *svc)
 {
     axis2_svc_grp_t *svc_grp = NULL;
@@ -1141,170 +1063,165 @@
     axis2_status_t status = AXIS2_FAILURE;
     AXIS2_ENV_CHECK(env, AXIS2_FAILURE);
     AXIS2_PARAM_CHECK(env->error, svc, AXIS2_FAILURE);
-   
-   svc_grp = axis2_svc_grp_create(env);
-   
-   if(NULL == svc_grp)
+
+    svc_grp = axis2_svc_grp_create(env);
+
+    if (NULL == svc_grp)
     {
         AXIS2_ERROR_SET(env->error, AXIS2_ERROR_NO_MEMORY, AXIS2_FAILURE)
         return AXIS2_FAILURE;
     }
-      
-   
-   svc_grp_qname = AXIS2_SVC_GET_QNAME(svc, env);
-    if(NULL == svc_grp_qname)
+
+    svc_grp_qname = AXIS2_SVC_GET_QNAME(svc, env);
+    if (NULL == svc_grp_qname)
     {
-        AXIS2_ERROR_SET(env->error, AXIS2_ERROR_INVALID_STATE_SVC, 
-            AXIS2_FAILURE);
+        AXIS2_ERROR_SET(env->error, AXIS2_ERROR_INVALID_STATE_SVC,
+                AXIS2_FAILURE);
         return AXIS2_FAILURE;
     }
-   
-   svc_grp_name = AXIS2_QNAME_GET_LOCALPART(svc_grp_qname, env);
-   if(!svc_grp_name)
+
+    svc_grp_name = AXIS2_QNAME_GET_LOCALPART(svc_grp_qname, env);
+    if (!svc_grp_name)
     {
         return AXIS2_FAILURE;
     }
-   status = AXIS2_SVC_GRP_SET_NAME(svc_grp, env, svc_grp_name);
-    if(AXIS2_FAILURE == status)
+    status = AXIS2_SVC_GRP_SET_NAME(svc_grp, env, svc_grp_name);
+    if (AXIS2_FAILURE == status)
     {
         return status;
     }
-   status = AXIS2_SVC_GRP_SET_PARENT(svc_grp, env, conf);
-    if(AXIS2_FAILURE == status)
+    status = AXIS2_SVC_GRP_SET_PARENT(svc_grp, env, conf);
+    if (AXIS2_FAILURE == status)
     {
         return status;
     }
-   status = AXIS2_SVC_GRP_ADD_SVC(svc_grp, env, svc);
-    if(AXIS2_FAILURE == status)
+    status = AXIS2_SVC_GRP_ADD_SVC(svc_grp, env, svc);
+    if (AXIS2_FAILURE == status)
     {
         return status;
     }
     status = axis2_conf_add_svc_grp(conf, env, svc_grp);
-   
-   return status;
+
+    return status;
 }
 
-axis2_svc_t *AXIS2_CALL 
+axis2_svc_t *AXIS2_CALL
 axis2_conf_get_svc (
-    const axis2_conf_t *conf, 
+    const axis2_conf_t *conf,
     const axis2_env_t *env,
-    axis2_char_t* svc_name)
-{   
-   AXIS2_ENV_CHECK(env, NULL);
+    const axis2_char_t* svc_name)
+{
+    AXIS2_ENV_CHECK(env, NULL);
     AXIS2_PARAM_CHECK(env->error, svc_name, NULL);
-   
+
     return axis2_hash_get(AXIS2_INTF_TO_IMPL(conf)->all_svcs, svc_name,
-        AXIS2_HASH_KEY_STRING);
+            AXIS2_HASH_KEY_STRING);
 }
 
-axis2_status_t AXIS2_CALL 
+axis2_status_t AXIS2_CALL
 axis2_conf_remove_svc (
-    axis2_conf_t *conf, 
+    axis2_conf_t *conf,
     const axis2_env_t *env,
     const axis2_char_t *svc_name)
 {
     axis2_conf_impl_t *config_impl = NULL;
-        
+
     AXIS2_ENV_CHECK(env, AXIS2_FAILURE);
     AXIS2_PARAM_CHECK(env->error, svc_name, AXIS2_FAILURE);
-    
+
     axis2_hash_set(config_impl->all_svcs, svc_name, AXIS2_HASH_KEY_STRING,
-        NULL);
+            NULL);
     return AXIS2_SUCCESS;
 }
 
 axis2_status_t AXIS2_CALL
 axis2_conf_add_param (
-    axis2_conf_t *conf, 
+    axis2_conf_t *conf,
     const axis2_env_t *env,
     axis2_param_t *param)
 {
     axis2_status_t status = AXIS2_FAILURE;
-    
+
     AXIS2_ENV_CHECK(env, AXIS2_FAILURE);
     AXIS2_PARAM_CHECK(env->error, param, AXIS2_FAILURE);
-    
-   if(AXIS2_TRUE == axis2_conf_is_param_locked(conf, env, 
-        AXIS2_PARAM_GET_NAME(param, env)))
+
+    if (AXIS2_TRUE == axis2_conf_is_param_locked(conf, env,
+            AXIS2_PARAM_GET_NAME(param, env)))
     {
-        AXIS2_ERROR_SET(env->error, 
-            AXIS2_ERROR_PARAMETER_LOCKED_CANNOT_OVERRIDE, AXIS2_FAILURE);
+        AXIS2_ERROR_SET(env->error,
+                AXIS2_ERROR_PARAMETER_LOCKED_CANNOT_OVERRIDE, AXIS2_FAILURE);
         return AXIS2_FAILURE;
     }
-   else
+    else
     {
-        status = AXIS2_PARAM_CONTAINER_ADD_PARAM(conf->param_container, env, 
-            param);
+        status = AXIS2_PARAM_CONTAINER_ADD_PARAM(conf->param_container, env,
+                param);
     }
     return status;
 }
 
 axis2_param_t *AXIS2_CALL
 axis2_conf_get_param (
-    const axis2_conf_t *conf, 
+    const axis2_conf_t *conf,
     const axis2_env_t *env,
     const axis2_char_t *name)
 {
     AXIS2_ENV_CHECK(env, NULL);
     AXIS2_PARAM_CHECK(env->error, name, NULL);
-    
-   if(NULL == conf->param_container)
-   {
-      AXIS2_ERROR_SET(env->error, 
-            AXIS2_ERROR_INVALID_STATE_PARAM_CONTAINER, AXIS2_FAILURE);
-        return NULL;
-   }
-      
-   return AXIS2_PARAM_CONTAINER_GET_PARAM(conf->param_container, env,
-        name);
-   
+
+    if (NULL == conf->param_container)
+    {
+        AXIS2_ERROR_SET(env->error,
+                AXIS2_ERROR_INVALID_STATE_PARAM_CONTAINER, AXIS2_FAILURE);
+        return NULL;
+    }
+
+    return AXIS2_PARAM_CONTAINER_GET_PARAM(conf->param_container, env,
+            name);
+
 }
 
 axis2_array_list_t *AXIS2_CALL
-axis2_conf_get_params (
-    const axis2_conf_t *conf, 
+axis2_conf_get_all_params (
+    const axis2_conf_t *conf,
     const axis2_env_t *env)
 {
-   AXIS2_ENV_CHECK(env, NULL);
-   return AXIS2_PARAM_CONTAINER_GET_PARAMS(conf->param_container, env);
-   
+    AXIS2_ENV_CHECK(env, NULL);
+    return AXIS2_PARAM_CONTAINER_GET_PARAMS(conf->param_container, env);
+
 }
 
 axis2_bool_t AXIS2_CALL
 axis2_conf_is_param_locked (
-    axis2_conf_t *conf, 
+    const axis2_conf_t *conf,
     const axis2_env_t *env,
-    axis2_char_t *param_name)
+    const axis2_char_t *param_name)
 {
     axis2_param_t *param = NULL;
-        
+
     AXIS2_ENV_CHECK(env, AXIS2_FALSE);
     AXIS2_PARAM_CHECK(env->error, param_name, AXIS2_FALSE);
-    
+
     param = axis2_conf_get_param(conf, env, param_name);
-    return (NULL != param  && AXIS2_TRUE == AXIS2_PARAM_IS_LOCKED(param, env));
+    return (NULL != param && AXIS2_TRUE == AXIS2_PARAM_IS_LOCKED(param, env));
 }
 
 axis2_transport_in_desc_t *AXIS2_CALL
 axis2_conf_get_transport_in(
     const axis2_conf_t *conf,
     const axis2_env_t *env,
-    axis2_qname_t *qname)
+    const axis2_qname_t *qname)
 {
     axis2_conf_impl_t *config_impl = NULL;
     AXIS2_ENV_CHECK(env, NULL);
     AXIS2_PARAM_CHECK(env->error, qname, NULL);
-    
+
     config_impl = AXIS2_INTF_TO_IMPL(conf);
-    
+
     return (axis2_transport_in_desc_t *) axis2_hash_get(config_impl->
             transports_in, AXIS2_QNAME_TO_STRING(qname, env), AXIS2_HASH_KEY_STRING);
 }
 
-/**
- * Function add_transport_in
- * @param transport
- */
 axis2_status_t AXIS2_CALL
 axis2_conf_add_transport_in(
     axis2_conf_t *conf,
@@ -1313,50 +1230,46 @@
 {
     axis2_conf_impl_t *config_impl = NULL;
     axis2_qname_t *qname = NULL;
-    
+
     AXIS2_ENV_CHECK(env, AXIS2_FAILURE);
     AXIS2_PARAM_CHECK(env->error, transport, AXIS2_FAILURE);
-    
+
     config_impl = AXIS2_INTF_TO_IMPL(conf);
-    
-    if(!config_impl->transports_in)
+
+    if (!config_impl->transports_in)
     {
         config_impl->transports_in = (axis2_hash_t *) axis2_hash_make(env);
-        if(!config_impl->transports_in)
+        if (!config_impl->transports_in)
             return AXIS2_FAILURE;
     }
     qname = AXIS2_TRANSPORT_IN_DESC_GET_QNAME(transport, env);
     if (!qname)
         return AXIS2_FAILURE;
-        
+
     axis2_hash_set(config_impl->transports_in,
-        AXIS2_QNAME_TO_STRING(qname, env), AXIS2_HASH_KEY_STRING, 
+            AXIS2_QNAME_TO_STRING(qname, env), AXIS2_HASH_KEY_STRING,
             transport);
-    
+
     return AXIS2_SUCCESS;
-    
+
 }
 
 axis2_transport_out_desc_t *AXIS2_CALL
 axis2_conf_get_transport_out(
     const axis2_conf_t *conf,
     const axis2_env_t *env,
-    axis2_qname_t *qname)
+    const axis2_qname_t *qname)
 {
     axis2_conf_impl_t *config_impl = NULL;
     AXIS2_ENV_CHECK(env, NULL);
     AXIS2_PARAM_CHECK(env->error, qname, NULL);
-    
+
     config_impl = AXIS2_INTF_TO_IMPL(conf);
-    
+
     return (axis2_transport_out_desc_t *) axis2_hash_get(config_impl->
             transports_out, AXIS2_QNAME_TO_STRING(qname, env), AXIS2_HASH_KEY_STRING);
 }
 
-/**
- * Function add_transport_out
- * @param transport
- */
 axis2_status_t AXIS2_CALL
 axis2_conf_add_transport_out(
     axis2_conf_t *conf,
@@ -1365,95 +1278,85 @@
 {
     axis2_conf_impl_t *config_impl = NULL;
     axis2_qname_t *qname = NULL;
-    
+
     AXIS2_ENV_CHECK(env, AXIS2_FAILURE);
     AXIS2_PARAM_CHECK(env->error, transport, AXIS2_FAILURE);
-    
+
     config_impl = AXIS2_INTF_TO_IMPL(conf);
-    
-    if(!config_impl->transports_out)
+
+    if (!config_impl->transports_out)
     {
         config_impl->transports_out = axis2_hash_make(env);
-        if(!config_impl->transports_out)
+        if (!config_impl->transports_out)
             return AXIS2_FAILURE;
     }
-    
+
     qname = AXIS2_TRANSPORT_OUT_DESC_GET_QNAME(transport, env);
     if (!qname)
         return AXIS2_FAILURE;
     axis2_hash_set(config_impl->transports_out,
-        AXIS2_QNAME_TO_STRING(qname, env),
+            AXIS2_QNAME_TO_STRING(qname, env),
             AXIS2_HASH_KEY_STRING, transport);
     return AXIS2_SUCCESS;
-    
+
 }
 
 axis2_hash_t *AXIS2_CALL
-axis2_conf_get_transports_in(
+axis2_conf_get_all_in_transports(
     const axis2_conf_t *conf,
-    const axis2_env_t *env) 
+    const axis2_env_t *env)
 {
     AXIS2_ENV_CHECK(env, NULL);
     return AXIS2_INTF_TO_IMPL(conf)->transports_in;
 }
 
-/**
- * Function get_module
- * @param name
- * @return axis2_module_desc_t*
- */
 axis2_module_desc_t *AXIS2_CALL
 axis2_conf_get_module(
     const axis2_conf_t *conf,
     const axis2_env_t *env,
-    const axis2_qname_t *qname) 
+    const axis2_qname_t *qname)
 {
     axis2_conf_impl_t *conf_impl = NULL;
     axis2_char_t *name = NULL;
     axis2_module_desc_t *ret = NULL;
     axis2_char_t *module_name = NULL;
     axis2_qname_t *mod_qname = NULL;
-    axis2_char_t *def_mod_ver = NULL;
-    
+    const axis2_char_t *def_mod_ver = NULL;
+
     AXIS2_ENV_CHECK(env, NULL);
     AXIS2_PARAM_CHECK(env->error, qname, NULL);
     conf_impl = AXIS2_INTF_TO_IMPL(conf);
 
     name = AXIS2_QNAME_TO_STRING(qname, env);
-    ret =  (axis2_module_desc_t *) axis2_hash_get(conf_impl->all_modules, 
-        name, AXIS2_HASH_KEY_STRING);
-    if(NULL != ret)
+    ret = (axis2_module_desc_t *) axis2_hash_get(conf_impl->all_modules,
+            name, AXIS2_HASH_KEY_STRING);
+    if (NULL != ret)
     {
         return ret;
     }
     module_name = AXIS2_QNAME_GET_LOCALPART(qname, env);
-    if(NULL == module_name)
+    if (NULL == module_name)
     {
         return NULL;
     }
     def_mod_ver = AXIS2_CONF_GET_DEFAULT_MODULE_VERSION(conf, env, module_name);
     mod_qname = axis2_core_utils_get_module_qname(env, name, def_mod_ver);
-    if(NULL == mod_qname)
+    if (NULL == mod_qname)
     {
         return NULL;
     }
     name = AXIS2_QNAME_TO_STRING(mod_qname, env);
-    ret =  (axis2_module_desc_t *) axis2_hash_get(conf_impl->all_modules, 
-        name, AXIS2_HASH_KEY_STRING);
+    ret = (axis2_module_desc_t *) axis2_hash_get(conf_impl->all_modules,
+            name, AXIS2_HASH_KEY_STRING);
     AXIS2_QNAME_FREE(mod_qname, env);
     mod_qname = NULL;
     return ret;
 }
 
-
-/**
- * Function get_engadged_modules
- * @return  axis2_array_list_t*
- */
 axis2_array_list_t *AXIS2_CALL
-axis2_conf_get_engaged_modules(
+axis2_conf_get_all_engaged_modules(
     const axis2_conf_t *conf,
-    const axis2_env_t *env) 
+    const axis2_env_t *env)
 {
     AXIS2_ENV_CHECK(env, NULL);
     return AXIS2_INTF_TO_IMPL(conf)->engaged_modules;
@@ -1462,107 +1365,72 @@
 axis2_array_list_t *AXIS2_CALL
 axis2_conf_get_in_phases_upto_and_including_post_dispatch(
     const axis2_conf_t *conf,
-    const axis2_env_t *env) 
+    const axis2_env_t *env)
 {
     AXIS2_ENV_CHECK(env, NULL);
     return AXIS2_INTF_TO_IMPL(conf)->
-        in_phases_upto_and_including_post_dispatch;
+            in_phases_upto_and_including_post_dispatch;
 }
 
 axis2_array_list_t *AXIS2_CALL
-axis2_conf_get_outflow(
+axis2_conf_get_out_flow(
     const axis2_conf_t *conf,
-    const axis2_env_t *env) 
+    const axis2_env_t *env)
 {
     AXIS2_ENV_CHECK(env, NULL);
     return AXIS2_INTF_TO_IMPL(conf)->out_phases;
 }
 
-
-/**
- * Function get_in_fault_flow
- * @return axis2_array_list_t*
- */
 axis2_array_list_t *AXIS2_CALL
-axis2_conf_get_in_faultflow(
+axis2_conf_get_in_fault_flow(
     const axis2_conf_t *conf,
-    const axis2_env_t *env) 
+    const axis2_env_t *env)
 {
     AXIS2_ENV_CHECK(env, NULL);
-    return AXIS2_INTF_TO_IMPL(conf)->in_faultphases;
+    return AXIS2_INTF_TO_IMPL(conf)->in_fault_phases;
 }
 
-/**
- * @return ArrayList
- */
 axis2_array_list_t *AXIS2_CALL
-axis2_conf_get_out_faultflow(
+axis2_conf_get_out_fault_flow(
     const axis2_conf_t *conf,
-    const axis2_env_t *env) 
+    const axis2_env_t *env)
 {
     AXIS2_ENV_CHECK(env, NULL);
-    return AXIS2_INTF_TO_IMPL(conf)->out_faultphases;
+    return AXIS2_INTF_TO_IMPL(conf)->out_fault_phases;
 }
 
 axis2_hash_t *AXIS2_CALL
-axis2_conf_get_transports_out(
+axis2_conf_get_all_out_transports(
     const axis2_conf_t *conf,
-    const axis2_env_t *env) 
+    const axis2_env_t *env)
 {
     AXIS2_ENV_CHECK(env, NULL);
     return AXIS2_INTF_TO_IMPL(conf)->transports_out;
-}   
-
-/*static axis2_status_t 
-split_svc_name (const axis2_env_t *env, 
-                axis2_char_t *svc_name, 
-                axis2_char_t **svc_name_st)
-{
-    axis2_char_t *svc_name_l = NULL;
-    axis2_char_t *grp_name = NULL;
-   
-    svc_name_l = strpbrk(svc_name, SERVICE_NAME_SPLIT_CHAR);
-    if(NULL == svc_name_l)
-    {
-        *(svc_name_st + 1) = AXIS2_STRDUP(svc_name, env);
-        *(svc_name_st + 2) = AXIS2_STRDUP(svc_name, env);
-        return AXIS2_SUCCESS;
-    }
-    svc_name_l[0] = AXIS2_EOLN;
-    grp_name = AXIS2_MALLOC(env->allocator, strlen(svc_name));
-    sscanf(svc_name, "%s", grp_name);
-    svc_name_l = svc_name_l + 1;
-    *(svc_name_st + 1) = AXIS2_STRDUP(svc_name_l, env);
-    *(svc_name_st + 2) = grp_name;
-   
-    return AXIS2_SUCCESS;   
 }
-*/
 
 axis2_hash_t *AXIS2_CALL
-axis2_conf_get_faulty_svcs(
+axis2_conf_get_all_faulty_svcs(
     const axis2_conf_t *conf,
-    const axis2_env_t *env) 
+    const axis2_env_t *env)
 {
     AXIS2_ENV_CHECK(env, NULL);
-    
+
     return AXIS2_INTF_TO_IMPL(conf)->faulty_svcs;
 }
 
-axis2_hash_t *AXIS2_CALL 
-axis2_conf_get_faulty_modules(
+axis2_hash_t *AXIS2_CALL
+axis2_conf_get_all_faulty_modules(
     const axis2_conf_t *conf,
-    const axis2_env_t *env) 
+    const axis2_env_t *env)
 {
     AXIS2_ENV_CHECK(env, NULL);
     return AXIS2_INTF_TO_IMPL(conf)->faulty_modules;
 }
-    
-/*to get all the services in the system */
+
 axis2_hash_t *AXIS2_CALL
-axis2_conf_get_svcs(
+axis2_conf_get_all_svcs(
     const axis2_conf_t *conf,
-    const axis2_env_t *env) 
+    const axis2_env_t *env)
 {
     axis2_conf_impl_t *config_impl = NULL;
     axis2_hash_t *sgs = NULL;
@@ -1572,31 +1440,31 @@
     void *value2 = NULL;
     axis2_svc_grp_t *axis_svc_grp = NULL;
     axis2_hash_t *svcs = NULL;
-    axis2_svc_t *svc =NULL;
+    axis2_svc_t *svc = NULL;
     axis2_char_t *svc_name = NULL;
-    
+
     AXIS2_ENV_CHECK(env, NULL);
-    
+
     config_impl = AXIS2_INTF_TO_IMPL(conf);
-    sgs = axis2_conf_get_svc_grps(conf, env);
+    sgs = axis2_conf_get_all_svc_grps(conf, env);
     index_i = axis2_hash_first(sgs, env);
-    while(NULL != index_i)
+    while (NULL != index_i)
     {
         axis2_hash_this(index_i, NULL, NULL, &value);
         axis_svc_grp = (axis2_svc_grp_t *) value;
         svcs = AXIS2_SVC_GRP_GET_SVCS(axis_svc_grp, env);
         index_j = axis2_hash_first(svcs, env);
-        while(NULL != index_j)
+        while (NULL != index_j)
         {
             axis2_hash_this(index_j, NULL, NULL, &value2);
             svc = (axis2_svc_t *) value2;
             svc_name = AXIS2_QNAME_GET_LOCALPART(AXIS2_SVC_GET_QNAME(svc, env), env);
             axis2_hash_set(config_impl->all_svcs, svc_name,
-                AXIS2_HASH_KEY_STRING, svc);    
-                            
+                    AXIS2_HASH_KEY_STRING, svc);
+
             index_j = axis2_hash_next(env, index_j);
         }
-        
+
         index_i = axis2_hash_next(env, index_i);
     }
     return config_impl->all_svcs;
@@ -1606,7 +1474,7 @@
 axis2_conf_is_engaged(
     axis2_conf_t *conf,
     const axis2_env_t *env,
-    const axis2_qname_t *module_name) 
+    const axis2_qname_t *module_name)
 {
     axis2_conf_impl_t *config_impl = NULL;
     axis2_qname_t *def_mod_qname = NULL;
@@ -1616,25 +1484,25 @@
     AXIS2_ENV_CHECK(env, AXIS2_FALSE);
     AXIS2_PARAM_CHECK(env->error, module_name, AXIS2_FALSE);
     config_impl = AXIS2_INTF_TO_IMPL(conf);
-   
-    def_mod = AXIS2_CONF_GET_DEFAULT_MODULE(conf, env, 
-                        AXIS2_QNAME_GET_LOCALPART(module_name, env));
-    if(NULL != def_mod)
+
+    def_mod = AXIS2_CONF_GET_DEFAULT_MODULE(conf, env,
+            AXIS2_QNAME_GET_LOCALPART(module_name, env));
+    if (NULL != def_mod)
     {
         def_mod_qname = AXIS2_MODULE_DESC_GET_NAME(def_mod, env);
     }
     size = AXIS2_ARRAY_LIST_SIZE(config_impl->engaged_modules, env);
-    for(i = 0; i < size; i++)
+    for (i = 0; i < size; i++)
     {
         axis2_qname_t *qname = NULL;
-        
-        
+
+
         qname = (axis2_qname_t *) AXIS2_ARRAY_LIST_GET(config_impl->
-            engaged_modules, env, i);
-        
-        if(AXIS2_TRUE == AXIS2_QNAME_EQUALS(module_name, env, qname) ||
-            (NULL != def_mod_qname && AXIS2_TRUE == AXIS2_QNAME_EQUALS(
-            def_mod_qname, env, qname)))
+                engaged_modules, env, i);
+
+        if (AXIS2_TRUE == AXIS2_QNAME_EQUALS(module_name, env, qname) ||
+                (NULL != def_mod_qname && AXIS2_TRUE == AXIS2_QNAME_EQUALS(
+                            def_mod_qname, env, qname)))
         {
             return AXIS2_TRUE;
         }
@@ -1645,10 +1513,10 @@
 axis2_phases_info_t *AXIS2_CALL
 axis2_conf_get_phases_info(
     const axis2_conf_t *conf,
-    const axis2_env_t *env) 
+    const axis2_env_t *env)
 {
     AXIS2_ENV_CHECK(env, NULL);
-    
+
     return (AXIS2_INTF_TO_IMPL(conf))->phases_info;
 }
 
@@ -1656,21 +1524,20 @@
 axis2_conf_set_phases_info(
     axis2_conf_t *conf,
     const axis2_env_t *env,
-    axis2_phases_info_t *phases_info) 
+    axis2_phases_info_t *phases_info)
 {
     axis2_conf_impl_t *config_impl = NULL;
-    
+
     AXIS2_ENV_CHECK(env, AXIS2_FAILURE);
     AXIS2_PARAM_CHECK(env->error, phases_info, AXIS2_FAILURE);
     config_impl = AXIS2_INTF_TO_IMPL(conf);
-    
-    if(config_impl->phases_info)
+
+    if (config_impl->phases_info)
     {
         AXIS2_PHASES_INFO_FREE(phases_info, env);
         config_impl->phases_info = NULL;
     }
     config_impl->phases_info = phases_info;
-    /*config_impl->out_faultphases = AXIS2_PHASES_INFO_GET_OP_OUT_FAULTPHASES(phases_info, env); */
     return AXIS2_SUCCESS;
 }
 
@@ -1678,21 +1545,21 @@
 axis2_conf_add_msg_recv(
     axis2_conf_t *conf,
     const axis2_env_t *env,
-    axis2_char_t *key,
-    axis2_msg_recv_t *msg_recv) 
+    const axis2_char_t *key,
+    axis2_msg_recv_t *msg_recv)
 {
     axis2_conf_impl_t *config_impl = NULL;
-    
+
     AXIS2_ENV_CHECK(env, AXIS2_FAILURE);
     AXIS2_PARAM_CHECK(env->error, key, AXIS2_FAILURE);
     AXIS2_PARAM_CHECK(env->error, msg_recv, AXIS2_FAILURE);
-    
+
     config_impl = AXIS2_INTF_TO_IMPL(conf);
-    
-    if(!config_impl->msg_recvs)
+
+    if (!config_impl->msg_recvs)
     {
         config_impl->msg_recvs = axis2_hash_make(env);
-        if(!config_impl->msg_recvs)
+        if (!config_impl->msg_recvs)
             return AXIS2_FAILURE;
     }
     axis2_hash_set(config_impl->msg_recvs, key, AXIS2_HASH_KEY_STRING, msg_recv);
@@ -1703,27 +1570,27 @@
 axis2_conf_get_msg_recv(
     const axis2_conf_t *conf,
     const axis2_env_t *env,
-    axis2_char_t *key) 
+    axis2_char_t *key)
 {
     AXIS2_ENV_CHECK(env, NULL);
-    
+
     return (axis2_msg_recv_t *) axis2_hash_get(AXIS2_INTF_TO_IMPL(
-        conf)->msg_recvs, key, AXIS2_HASH_KEY_STRING);
+                conf)->msg_recvs, key, AXIS2_HASH_KEY_STRING);
 }
 
 axis2_status_t AXIS2_CALL
 axis2_conf_set_out_phases(
     axis2_conf_t *conf,
     const axis2_env_t *env,
-    axis2_array_list_t *out_phases) 
+    axis2_array_list_t *out_phases)
 {
     axis2_conf_impl_t *config_impl = NULL;
-    
+
     AXIS2_ENV_CHECK(env, AXIS2_FAILURE);
     AXIS2_PARAM_CHECK(env->error, out_phases, AXIS2_FAILURE);
-    
+
     config_impl = AXIS2_INTF_TO_IMPL(conf);
-    if(config_impl->out_phases)
+    if (config_impl->out_phases)
     {
         AXIS2_ARRAY_LIST_FREE(config_impl->out_phases, env);
         config_impl->out_phases = NULL;
@@ -1740,102 +1607,81 @@
     AXIS2_ENV_CHECK(env, NULL);
     return AXIS2_INTF_TO_IMPL(conf)->out_phases;
 }
-    
-/**
- * @param list
- */
+
 axis2_status_t AXIS2_CALL
-axis2_conf_set_in_faultphases(
+axis2_conf_set_in_fault_phases(
     axis2_conf_t *conf,
     const axis2_env_t *env,
-    axis2_array_list_t *list) 
+    axis2_array_list_t *list)
 {
     axis2_conf_impl_t *config_impl = NULL;
-    
+
     AXIS2_ENV_CHECK(env, AXIS2_FAILURE);
     AXIS2_PARAM_CHECK(env->error, list, AXIS2_FAILURE);
-    
+
     config_impl = AXIS2_INTF_TO_IMPL(conf);
-    if(config_impl->in_faultphases)
+    if (config_impl->in_fault_phases)
     {
-        AXIS2_ARRAY_LIST_FREE(config_impl->in_faultphases, env);
-        config_impl->in_faultphases = NULL;
+        AXIS2_ARRAY_LIST_FREE(config_impl->in_fault_phases, env);
+        config_impl->in_fault_phases = NULL;
     }
-    config_impl->in_faultphases = list;
+    config_impl->in_fault_phases = list;
     return AXIS2_SUCCESS;
 }
 
-/**
- * @param list
- */
 axis2_status_t AXIS2_CALL
-axis2_conf_set_out_faultphases(
+axis2_conf_set_out_fault_phases(
     axis2_conf_t *conf,
     const axis2_env_t *env,
-    axis2_array_list_t *list) 
+    axis2_array_list_t *list)
 {
     axis2_conf_impl_t *config_impl = NULL;
-    
+
     AXIS2_ENV_CHECK(env, AXIS2_FAILURE);
     AXIS2_PARAM_CHECK(env->error, list, AXIS2_FAILURE);
-    
+
     config_impl = AXIS2_INTF_TO_IMPL(conf);
-    if(config_impl->out_faultphases)
+    if (config_impl->out_fault_phases)
     {
-        AXIS2_ARRAY_LIST_FREE(config_impl->out_faultphases, env);
-        config_impl->out_faultphases = NULL;
+        AXIS2_ARRAY_LIST_FREE(config_impl->out_fault_phases, env);
+        config_impl->out_fault_phases = NULL;
     }
-    config_impl->out_faultphases = list;
+    config_impl->out_fault_phases = list;
     return AXIS2_SUCCESS;
 }
-    
-/**
- * @return axis2_hash_t*
- */
+
 axis2_hash_t *AXIS2_CALL
-axis2_conf_get_modules(
+axis2_conf_get_all_modules(
     const axis2_conf_t *conf,
-    const axis2_env_t *env) 
+    const axis2_env_t *env)
 {
     AXIS2_ENV_CHECK(env, NULL);
-    
+
     return AXIS2_INTF_TO_IMPL(conf)->all_modules;
-}  
+}
 
-/**
- * Method add_module
- * @param module
- */
 axis2_status_t AXIS2_CALL
 axis2_conf_add_module(
     axis2_conf_t *conf,
     const axis2_env_t *env,
-    axis2_module_desc_t *module) 
+    axis2_module_desc_t *module)
 {
     axis2_conf_impl_t *config_impl = NULL;
     axis2_qname_t *module_qname = NULL;
-    
+
     axis2_status_t status = AXIS2_FAILURE;
     AXIS2_ENV_CHECK(env, AXIS2_FAILURE);
     AXIS2_PARAM_CHECK(env->error, module, AXIS2_FAILURE);
-    
+
     config_impl = AXIS2_INTF_TO_IMPL(conf);
-    
+
     status = AXIS2_MODULE_DESC_SET_PARENT(module, env, conf);
-    if(AXIS2_FAILURE == status)
+    if (AXIS2_FAILURE == status)
     {
         return status;
     }
-    /* Do we need to do this ? */
-    /*if(!config_impl->modules)
-    {
-        config_impl->modules = axis2_hash_make(env);
-        if(!config_impl->modules)
-        {
-            return AXIS2_FAILURE;
-        }
-    }*/
-    if(!config_impl->all_modules)
+
+    if (!config_impl->all_modules)
     {
         return AXIS2_FAILURE;
     }
@@ -1845,10 +1691,10 @@
         axis2_char_t *module_name = NULL;
 
         module_name = AXIS2_QNAME_TO_STRING(module_qname, env);
-        axis2_hash_set(config_impl->all_modules, module_name, 
-            AXIS2_HASH_KEY_STRING, module);
+        axis2_hash_set(config_impl->all_modules, module_name,
+                AXIS2_HASH_KEY_STRING, module);
     }
-    /* TODO we do not support module_name-SNAPSHOT.so case */
+
     return AXIS2_SUCCESS;
 }
 
@@ -1867,17 +1713,17 @@
     axis2_handler_t *handler = NULL;
     axis2_phase_t *post_dispatch = NULL;
     axis2_disp_checker_t *disp_checker = NULL;
-    
+
     AXIS2_ENV_CHECK(env, AXIS2_FAILURE);
     config_impl = AXIS2_INTF_TO_IMPL(conf);
 
     dispatch = axis2_phase_create(env, AXIS2_PHASE_DISPATCH);
-    if(!dispatch)
+    if (!dispatch)
     {
         return AXIS2_FAILURE;
     }
     add_dispatch = axis2_addr_disp_create(env);
-    if(!add_dispatch)
+    if (!add_dispatch)
     {
         AXIS2_PHASE_FREE(dispatch, env);
         return AXIS2_FAILURE;
@@ -1888,9 +1734,9 @@
     AXIS2_PHASE_ADD_HANDLER_AT(dispatch, env, 0, handler);
     AXIS2_ARRAY_LIST_ADD(config_impl->handlers, env, AXIS2_HANDLER_GET_HANDLER_DESC(handler, env));
     handler = NULL;
-    
+
     uri_dispatch = axis2_req_uri_disp_create(env);
-    if(!uri_dispatch)
+    if (!uri_dispatch)
     {
         return AXIS2_FAILURE;
     }
@@ -1900,9 +1746,9 @@
     AXIS2_PHASE_ADD_HANDLER_AT(dispatch, env, 1, handler);
     AXIS2_ARRAY_LIST_ADD(config_impl->handlers, env, AXIS2_HANDLER_GET_HANDLER_DESC(handler, env));
     handler = NULL;
-    
+
     soap_action_based_dispatch = axiom_soap_action_disp_create(env);
-    if(!soap_action_based_dispatch)
+    if (!soap_action_based_dispatch)
     {
         return AXIS2_FAILURE;
     }
@@ -1912,34 +1758,34 @@
     AXIS2_PHASE_ADD_HANDLER_AT(dispatch, env, 2, handler);
     AXIS2_ARRAY_LIST_ADD(config_impl->handlers, env, AXIS2_HANDLER_GET_HANDLER_DESC(handler, env));
     handler = NULL;
-    
+
     soap_msg_body_based_dispatch = axiom_soap_body_disp_create(env);
-    if(!soap_msg_body_based_dispatch)
+    if (!soap_msg_body_based_dispatch)
     {
         return AXIS2_FAILURE;
     }
-    
+
     handler = AXIS2_DISP_GET_BASE(soap_msg_body_based_dispatch, env);
     AXIS2_DISP_FREE(soap_msg_body_based_dispatch, env);
     AXIS2_PHASE_ADD_HANDLER_AT(dispatch, env, 3, handler);
     AXIS2_ARRAY_LIST_ADD(config_impl->handlers, env, AXIS2_HANDLER_GET_HANDLER_DESC(handler, env));
     handler = NULL;
-    
+
     status = AXIS2_ARRAY_LIST_ADD(config_impl->
             in_phases_upto_and_including_post_dispatch, env, dispatch);
-    if(AXIS2_SUCCESS != status)
+    if (AXIS2_SUCCESS != status)
     {
         AXIS2_PHASE_FREE(dispatch, env);
-        return status;   
+        return status;
     }
-    
+
     post_dispatch = axis2_phase_create(env, AXIS2_PHASE_POST_DISPATCH);
-    if(NULL == post_dispatch)
+    if (NULL == post_dispatch)
     {
         AXIS2_PHASE_FREE(dispatch, env);
         return AXIS2_FAILURE;
     }
-    
+
     disp_checker = axis2_disp_checker_create(env, NULL);
     handler = AXIS2_DISP_CHECKER_GET_BASE(disp_checker, env);
     AXIS2_DISP_CHECKER_FREE(disp_checker, env);
@@ -1950,14 +1796,14 @@
     handler = axis2_ctx_handler_create(env, NULL);
     AXIS2_PHASE_ADD_HANDLER_AT(post_dispatch, env, 1, handler);
     AXIS2_ARRAY_LIST_ADD(config_impl->handlers, env, AXIS2_HANDLER_GET_HANDLER_DESC(handler, env));
-    
+
     status = AXIS2_ARRAY_LIST_ADD(config_impl->
             in_phases_upto_and_including_post_dispatch, env, post_dispatch);
-    if(AXIS2_SUCCESS != status)
+    if (AXIS2_SUCCESS != status)
     {
         AXIS2_PHASE_FREE(dispatch, env);
         AXIS2_PHASE_FREE(post_dispatch, env);
-        return status;   
+        return status;
     }
     return AXIS2_SUCCESS;
 }
@@ -1973,45 +1819,38 @@
     axis2_handler_t *handler = NULL;
     axis2_phase_t *post_dispatch = NULL;
     axis2_disp_checker_t *disp_checker = NULL;
-    
+
     AXIS2_ENV_CHECK(env, AXIS2_FAILURE);
     AXIS2_PARAM_CHECK(env->error, dispatch, AXIS2_FAILURE);
     config_impl = AXIS2_INTF_TO_IMPL(conf);
-    
+
     status = AXIS2_ARRAY_LIST_ADD(config_impl->
             in_phases_upto_and_including_post_dispatch, env, dispatch);
-    if(AXIS2_FAILURE == status)
+    if (AXIS2_FAILURE == status)
     {
-        return AXIS2_FAILURE;   
+        return AXIS2_FAILURE;
     }
-    
+
     post_dispatch = axis2_phase_create(env, AXIS2_PHASE_POST_DISPATCH);
-    if(NULL == post_dispatch)
+    if (NULL == post_dispatch)
     {
         AXIS2_PHASE_FREE(dispatch, env);
         return AXIS2_FAILURE;
     }
-    
+
     disp_checker = axis2_disp_checker_create(env, NULL);
-    
-    /* TODO uncomment this when instant dispatcher is available */
-    /*
-    InstanceDispatcher instanceDispatcher = new InstanceDispatcher();
-    instanceDispatcher.getHandlerDesc().setParent(this);
-    */
-    
+
     handler = AXIS2_DISP_CHECKER_GET_BASE(disp_checker, env);
     AXIS2_PHASE_ADD_HANDLER_AT(post_dispatch, env, 0, handler);
- 
-    /*postDispatch.addHandler(instanceDispatcher,1); */
+
     status = AXIS2_ARRAY_LIST_ADD(config_impl->
             in_phases_upto_and_including_post_dispatch, env, post_dispatch);
-    if(AXIS2_FAILURE == status)
+    if (AXIS2_FAILURE == status)
     {
         AXIS2_PHASE_FREE(dispatch, env);
         AXIS2_PHASE_FREE(post_dispatch, env);
         AXIS2_DISP_CHECKER_FREE(disp_checker, env);
-        return AXIS2_FAILURE;   
+        return AXIS2_FAILURE;
     }
     return AXIS2_SUCCESS;
 }
@@ -2020,7 +1859,7 @@
 axis2_conf_engage_module(
     axis2_conf_t *conf,
     const axis2_env_t *env,
-    const axis2_qname_t *module_ref) 
+    const axis2_qname_t *module_ref)
 {
     axis2_conf_impl_t *config_impl = NULL;
     axis2_module_desc_t *module_desc = NULL;
@@ -2028,33 +1867,33 @@
     axis2_bool_t to_be_engaged = AXIS2_TRUE;
     axis2_dep_engine_t *dep_engine = NULL;
     axis2_status_t status = AXIS2_FAILURE;
-    
+
     AXIS2_ENV_CHECK(env, AXIS2_FAILURE);
     AXIS2_PARAM_CHECK(env->error, module_ref, AXIS2_FAILURE);
     config_impl = AXIS2_INTF_TO_IMPL(conf);
-    
+
     module_desc = axis2_conf_get_module(conf, env, module_ref);
-    if(NULL == module_desc)
+    if (NULL == module_desc)
     {
         axis2_file_t *file = NULL;
         axis2_char_t *file_name = NULL;
         axis2_arch_reader_t *arch_reader = NULL;
-        axis2_char_t *repos_path = NULL;
+        const axis2_char_t *repos_path = NULL;
         axis2_arch_file_data_t *file_data = NULL;
         axis2_char_t *temp_path1 = NULL;
         axis2_char_t *temp_path2 = NULL;
         axis2_char_t *temp_path3 = NULL;
         axis2_char_t *path = NULL;
-        
+
         arch_reader = axis2_arch_reader_create(env);
-        if(!arch_reader)
+        if (!arch_reader)
         {
             return AXIS2_FAILURE;
         }
         file_name = AXIS2_QNAME_GET_LOCALPART(module_ref, env);
         file = (axis2_file_t *) AXIS2_ARCH_READER_CREATE_MODULE_ARCH(
-            arch_reader, env, file_name) ;
-        repos_path = AXIS2_CONF_GET_REPOS(conf, env);
+                    arch_reader, env, file_name) ;
+        repos_path = AXIS2_CONF_GET_REPO(conf, env);
         temp_path1 = AXIS2_STRACAT(repos_path, AXIS2_PATH_SEP_STR, env);
         temp_path2 = AXIS2_STRACAT(temp_path1, AXIS2_MODULE_FOLDER, env);
         temp_path3 = AXIS2_STRACAT(temp_path2, AXIS2_PATH_SEP_STR, env);
@@ -2071,66 +1910,64 @@
         module_desc = AXIS2_DEP_ENGINE_BUILD_MODULE(dep_engine, env, file, conf);
         is_new_module = AXIS2_TRUE;
     }
-    if(NULL != module_desc)
+    if (NULL != module_desc)
     {
         int size = 0;
         int i = 0;
         axis2_qname_t *module_qname = NULL;
-        
+
         size = AXIS2_ARRAY_LIST_SIZE(config_impl->engaged_modules, env);
         module_qname = AXIS2_MODULE_DESC_GET_NAME(module_desc, env);
-        for(i = 0; i < size; i++)
+        for (i = 0; i < size; i++)
         {
             axis2_qname_t *qname = NULL;
-            
+
             qname = (axis2_qname_t *) AXIS2_ARRAY_LIST_GET(config_impl->
-                engaged_modules, env, i);
-            if(AXIS2_TRUE == AXIS2_QNAME_EQUALS(module_qname, env, qname))
+                    engaged_modules, env, i);
+            if (AXIS2_TRUE == AXIS2_QNAME_EQUALS(module_qname, env, qname))
             {
                 to_be_engaged = AXIS2_FALSE;
-                /* Instead of throwing the error, we can just log this problem */
-                /*log.info("Attempt to engage an already engaged module "+ qName);*/
             }
         }
-    } 
-    else 
+    }
+    else
     {
         AXIS2_ERROR_SET(env->error, AXIS2_ERROR_INVALID_MODULE, AXIS2_FAILURE);
         return AXIS2_FAILURE;
     }
-    if (to_be_engaged) 
+    if (to_be_engaged)
     {
         axis2_phase_resolver_t *phase_resolver = NULL;
         axis2_qname_t *module_qref_l = NULL;
         axis2_qname_t *module_qname = NULL;
 
         phase_resolver = axis2_phase_resolver_create_with_config(env, conf);
-        if(!phase_resolver)
+        if (!phase_resolver)
         {
             return AXIS2_FAILURE;
         }
 
-        status = AXIS2_PHASE_RESOLVER_ENGAGE_MODULE_GLOBALLY(phase_resolver, env, 
-            module_desc);
+        status = AXIS2_PHASE_RESOLVER_ENGAGE_MODULE_GLOBALLY(phase_resolver, env,
+                module_desc);
         AXIS2_PHASE_RESOLVER_FREE(phase_resolver, env);
-        if(AXIS2_SUCCESS != status)
+        if (AXIS2_SUCCESS != status)
         {
             return AXIS2_FAILURE;
         }
         module_qname = AXIS2_MODULE_DESC_GET_NAME(module_desc, env);
         module_qref_l = AXIS2_QNAME_CLONE(module_qname, env);
-        status = AXIS2_ARRAY_LIST_ADD(config_impl->engaged_modules, env, 
-            module_qref_l);
+        status = AXIS2_ARRAY_LIST_ADD(config_impl->engaged_modules, env,
+                module_qref_l);
     }
-    if (is_new_module) 
+    if (is_new_module)
     {
         status = axis2_conf_add_module(conf, env, module_desc);
     }
     return status;
 }
 
-axis2_char_t *AXIS2_CALL
-axis2_conf_get_repos(
+const axis2_char_t *AXIS2_CALL
+axis2_conf_get_repo(
     const axis2_conf_t *conf,
     const axis2_env_t *env)
 {
@@ -2139,12 +1976,11 @@
     AXIS2_ENV_CHECK(env, AXIS2_FAILURE);
     conf_impl = AXIS2_INTF_TO_IMPL(conf);
 
-    return conf_impl->axis2_repos;
-
+    return conf_impl->axis2_repo;
 }
 
 axis2_status_t AXIS2_CALL
-axis2_conf_set_repos(
+axis2_conf_set_repo(
     axis2_conf_t *conf,
     const axis2_env_t *env,
     axis2_char_t *repos_path)
@@ -2154,12 +1990,12 @@
     AXIS2_ENV_CHECK(env, AXIS2_FAILURE);
     conf_impl = AXIS2_INTF_TO_IMPL(conf);
 
-    if(conf_impl->axis2_repos)
+    if (conf_impl->axis2_repo)
     {
-        AXIS2_FREE(env->allocator, conf_impl->axis2_repos);
-        conf_impl->axis2_repos = NULL;
+        AXIS2_FREE(env->allocator, conf_impl->axis2_repo);
+        conf_impl->axis2_repo = NULL;
     }
-    conf_impl->axis2_repos = AXIS2_STRDUP(repos_path, env);
+    conf_impl->axis2_repo = AXIS2_STRDUP(repos_path, env);
     return AXIS2_SUCCESS;
 }
 
@@ -2175,18 +2011,18 @@
     return AXIS2_SUCCESS;
 }
 
-axis2_char_t *AXIS2_CALL
+const axis2_char_t *AXIS2_CALL
 axis2_conf_get_default_module_version(
-    const axis2_conf_t *conf, 
-    const axis2_env_t *env, 
+    const axis2_conf_t *conf,
+    const axis2_env_t *env,
     const axis2_char_t *module_name)
 {
     axis2_hash_t *def_ver_map = NULL;
     AXIS2_ENV_CHECK(env, NULL);
     AXIS2_PARAM_CHECK(env->error, module_name, NULL);
-    
+
     def_ver_map = AXIS2_INTF_TO_IMPL(conf)->name_to_version_map;
-    if(NULL == def_ver_map)
+    if (NULL == def_ver_map)
     {
         return NULL;
     }
@@ -2196,22 +2032,22 @@
 axis2_module_desc_t *AXIS2_CALL
 axis2_conf_get_default_module(
     const axis2_conf_t *conf,
-    const axis2_env_t *env, 
+    const axis2_env_t *env,
     const axis2_char_t *module_name)
 {
     axis2_module_desc_t *ret_mod = NULL;
     axis2_char_t *mod_name = NULL;
-    axis2_char_t *mod_ver = NULL;
+    const axis2_char_t *mod_ver = NULL;
     axis2_hash_t *all_modules = NULL;
     axis2_qname_t *mod_qname = NULL;
-    
+
     AXIS2_ENV_CHECK(env, NULL);
     AXIS2_PARAM_CHECK(env->error, module_name, NULL);
-    
+
     all_modules = AXIS2_INTF_TO_IMPL(conf)->all_modules;
     mod_ver = AXIS2_CONF_GET_DEFAULT_MODULE_VERSION(conf, env, module_name);
-    
-    if(NULL == mod_ver)
+
+    if (NULL == mod_ver)
     {
         mod_name = AXIS2_STRDUP(module_name, env);
     }
@@ -2225,36 +2061,28 @@
     mod_qname = axis2_qname_create(env, mod_name, NULL, NULL);
     AXIS2_FREE(env->allocator, mod_name);
     mod_name = NULL;
-    
-    if(NULL == mod_qname)
+
+    if (NULL == mod_qname)
     {
         return NULL;
     }
-    ret_mod = (axis2_module_desc_t *)axis2_hash_get(all_modules, 
-                    AXIS2_QNAME_TO_STRING(mod_qname, env), 
-                    AXIS2_HASH_KEY_STRING);
-                    
-    return ret_mod;                    
+    ret_mod = (axis2_module_desc_t *)axis2_hash_get(all_modules,
+            AXIS2_QNAME_TO_STRING(mod_qname, env),
+            AXIS2_HASH_KEY_STRING);
+
+    return ret_mod;
 }
 
 
-/**
- * Adds a dafault module version , which can be done either programatically 
- * or by using axis2.xml . The default module version is important if user asks
- * to engage a module without given version ID, in which case, we will engage 
- * the default version.
- *
- * @param module_name
- * @param module_version
- */
 axis2_status_t AXIS2_CALL
 axis2_conf_add_default_module_version(
-    axis2_conf_t *conf, const axis2_env_t *env,
-    axis2_char_t *module_name, 
-    axis2_char_t *module_version)
+    axis2_conf_t *conf, 
+    const axis2_env_t *env,
+    const axis2_char_t *module_name,
+    const axis2_char_t *module_version)
 {
     axis2_hash_t *name_to_ver_map = NULL;
-    
+
     AXIS2_ENV_CHECK(env, AXIS2_FAILURE);
     AXIS2_PARAM_CHECK(env->error, module_name, AXIS2_FAILURE);
     AXIS2_PARAM_CHECK(env->error, module_version, AXIS2_FAILURE);
@@ -2263,44 +2091,38 @@
      * it again
      */
     name_to_ver_map = AXIS2_INTF_TO_IMPL(conf)->name_to_version_map;
-    
-    if (NULL == axis2_hash_get(name_to_ver_map, module_name, 
-                        AXIS2_HASH_KEY_STRING))
+
+    if (NULL == axis2_hash_get(name_to_ver_map, module_name,
+            AXIS2_HASH_KEY_STRING))
     {
         axis2_char_t *new_entry = AXIS2_STRDUP(module_version, env);
-        if(NULL == new_entry)
+        if (NULL == new_entry)
         {
             return AXIS2_FAILURE;
         }
-        axis2_hash_set(name_to_ver_map,module_name, AXIS2_HASH_KEY_STRING, 
-            new_entry);
+        axis2_hash_set(name_to_ver_map, module_name, AXIS2_HASH_KEY_STRING,
+                new_entry);
         return AXIS2_SUCCESS;
     }
     return AXIS2_FAILURE;
 }
 
-/**
- * Engages a module using given name and its version ID.
- *
- * @param module_name
- * @param version_id
- */
 axis2_status_t AXIS2_CALL
 axis2_conf_engage_module_with_version(
-    axis2_conf_t *conf, 
+    axis2_conf_t *conf,
     const axis2_env_t *env,
     const axis2_char_t *module_name,
     const axis2_char_t *version_id)
 {
     axis2_qname_t *module_qname = NULL;
     axis2_status_t status = AXIS2_FAILURE;
-    
+
     AXIS2_ENV_CHECK(env, AXIS2_FAILURE);
     AXIS2_PARAM_CHECK(env->error, module_name, AXIS2_FAILURE);
-    
-    module_qname = axis2_core_utils_get_module_qname(env, module_name, 
-                                                            version_id);
-    if(NULL == module_qname)
+
+    module_qname = axis2_core_utils_get_module_qname(env, module_name,
+            version_id);
+    if (NULL == module_qname)
     {
         return AXIS2_FAILURE;
     }



---------------------------------------------------------------------
To unsubscribe, e-mail: axis-cvs-unsubscribe@ws.apache.org
For additional commands, e-mail: axis-cvs-help@ws.apache.org