Merge r1883184 from trunk:

Optimize 'ap_list_provider_names'.

Save some memory and potentialy avoid some useless copy .

Using 'goto' is not realy in line with httpd coding style, but the function is small and still readable IMHO.

Submitted by: jailletc36
Reviewed by: jailletc36, jorton, ylavic

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/branches/2.4.x@1887380 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/server/provider.c b/server/provider.c
index cf307e7c72f8d744c75c215a6345c39291b4261a..f54fb5e36bd2281ce96caaa457b2269c01fb6ce1 100644 (file)
--- a/server/provider.c
+++ b/server/provider.c
@@ -55,7 +55,6 @@ AP_DECLARE(apr_status_t) ap_register_provider(apr_pool_t *pool,
         provider_group_hash = apr_hash_make(pool);
         apr_hash_set(global_providers, provider_group, APR_HASH_KEY_STRING,
                      provider_group_hash);
-
     }
 
     provider_version_hash = apr_hash_get(provider_group_hash, provider_name,
@@ -65,7 +64,6 @@ AP_DECLARE(apr_status_t) ap_register_provider(apr_pool_t *pool,
         provider_version_hash = apr_hash_make(pool);
         apr_hash_set(provider_group_hash, provider_name, APR_HASH_KEY_STRING,
                      provider_version_hash);
-
     }
 
     /* just set it. no biggy if it was there before. */
@@ -80,7 +78,6 @@ AP_DECLARE(apr_status_t) ap_register_provider(apr_pool_t *pool,
         provider_group_hash = apr_hash_make(pool);
         apr_hash_set(global_providers_names, provider_group, APR_HASH_KEY_STRING,
                      provider_group_hash);
-
     }
 
     provider_version_hash = apr_hash_get(provider_group_hash, provider_version,
@@ -90,7 +87,6 @@ AP_DECLARE(apr_status_t) ap_register_provider(apr_pool_t *pool,
         provider_version_hash = apr_hash_make(pool);
         apr_hash_set(provider_group_hash, provider_version, APR_HASH_KEY_STRING,
                      provider_version_hash);
-
     }
 
     /* just set it. no biggy if it was there before. */
@@ -132,35 +128,40 @@ AP_DECLARE(apr_array_header_t *) ap_list_provider_names(apr_pool_t *pool,
                                               const char *provider_group,
                                               const char *provider_version)
 {
-    apr_array_header_t *ret = apr_array_make(pool, 10, sizeof(ap_list_provider_names_t));
+    apr_array_header_t *ret = NULL;
     ap_list_provider_names_t *entry;
     apr_hash_t *provider_group_hash, *h;
     apr_hash_index_t *hi;
     char *val;
 
     if (global_providers_names == NULL) {
-        return ret;
+        goto out;
     }
 
     provider_group_hash = apr_hash_get(global_providers_names, provider_group,
                                        APR_HASH_KEY_STRING);
 
     if (provider_group_hash == NULL) {
-        return ret;
+        goto out;
     }
 
-    h = apr_hash_get(provider_group_hash, provider_version,
-                                      APR_HASH_KEY_STRING);
+    h = apr_hash_get(provider_group_hash, provider_version, APR_HASH_KEY_STRING);
 
     if (h == NULL) {
-        return ret;
+        goto out;
     }
 
+    ret = apr_array_make(pool, apr_hash_count(h), sizeof(ap_list_provider_names_t));
     for (hi = apr_hash_first(pool, h); hi; hi = apr_hash_next(hi)) {
         apr_hash_this(hi, NULL, NULL, (void *)&val);
         entry = apr_array_push(ret);
         entry->provider_name = apr_pstrdup(pool, val);
     }
+    
+out:
+    if (ret == NULL) {
+        ret = apr_array_make(pool, 1, sizeof(ap_list_provider_names_t));
+    }
     return ret;
 }