You are viewing a plain text version of this content. The canonical link for it is here.
Posted to apreq-cvs@httpd.apache.org by jo...@apache.org on 2005/03/15 16:14:47 UTC

svn commit: r157551 - httpd/apreq/branches/multi-env-unstable/module/apache2/handle.c

Author: joes
Date: Tue Mar 15 07:14:45 2005
New Revision: 157551

URL: http://svn.apache.org/viewcvs?view=rev&rev=157551
Log:
Move the typical case (APR_SUCCESS)
to the top of the switch statement.



Modified:
    httpd/apreq/branches/multi-env-unstable/module/apache2/handle.c

Modified: httpd/apreq/branches/multi-env-unstable/module/apache2/handle.c
URL: http://svn.apache.org/viewcvs/httpd/apreq/branches/multi-env-unstable/module/apache2/handle.c?view=diff&r1=157550&r2=157551
==============================================================================
--- httpd/apreq/branches/multi-env-unstable/module/apache2/handle.c (original)
+++ httpd/apreq/branches/multi-env-unstable/module/apache2/handle.c Tue Mar 15 07:14:45 2005
@@ -188,6 +188,14 @@
 
     switch (ctx->body_status) {
 
+    case APR_SUCCESS:
+
+        val = apr_table_get(ctx->body, name);
+        if (val != NULL)
+            return apreq_value_to_param(val);
+        return NULL;
+
+
     case APR_EINIT:
 
         apreq_filter_init_context(f);
@@ -225,13 +233,6 @@
         ctx->parser->hook = h->next;
         return NULL;
 
-
-    case APR_SUCCESS:
-
-        val = apr_table_get(ctx->body, name);
-        if (val != NULL)
-            return apreq_value_to_param(val);
-        return NULL;
 
     default: