You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by sf...@apache.org on 2010/12/04 12:22:30 UTC

svn commit: r1042146 - in /httpd/httpd/trunk: include/ modules/metadata/ modules/ssl/ server/

Author: sf
Date: Sat Dec  4 11:22:30 2010
New Revision: 1042146

URL: http://svn.apache.org/viewvc?rev=1042146&view=rev
Log:
Rename ap_expr's typedef names:
ap_expr            ->  ap_expr_t
ap_expr_parse_ctx  ->  ap_expr_parse_ctx_t
ap_expr_eval_ctx   ->  ap_expr_eval_ctx_t
ap_expr_lookup_fn  ->  ap_expr_lookup_fn_t
ap_expr_node_op    ->  ap_expr_node_op_e

Modified:
    httpd/httpd/trunk/include/ap_expr.h
    httpd/httpd/trunk/include/ap_mmn.h
    httpd/httpd/trunk/modules/metadata/mod_setenvif.c
    httpd/httpd/trunk/modules/ssl/ssl_engine_vars.c
    httpd/httpd/trunk/server/util_expr_eval.c
    httpd/httpd/trunk/server/util_expr_parse.c
    httpd/httpd/trunk/server/util_expr_parse.h
    httpd/httpd/trunk/server/util_expr_parse.y
    httpd/httpd/trunk/server/util_expr_private.h
    httpd/httpd/trunk/server/util_expr_scan.c
    httpd/httpd/trunk/server/util_expr_scan.l

Modified: httpd/httpd/trunk/include/ap_expr.h
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/include/ap_expr.h?rev=1042146&r1=1042145&r2=1042146&view=diff
==============================================================================
--- httpd/httpd/trunk/include/ap_expr.h (original)
+++ httpd/httpd/trunk/include/ap_expr.h Sat Dec  4 11:22:30 2010
@@ -31,12 +31,12 @@ extern "C" {
 #endif
 
 /** A node in the expression parse tree */
-typedef struct ap_expr_node ap_expr;
+typedef struct ap_expr_node ap_expr_t;
 
 /** Struct describing a parsed expression */
 typedef struct {
     /** The root of the actual expression parse tree */
-    ap_expr *root_node;
+    ap_expr_t *root_node;
     /** The filename where the expression has been defined (for logging).
      *  May be NULL
      */
@@ -115,7 +115,7 @@ typedef struct {
      * interested in this information.
      */
     const char **vary_this;
-} ap_expr_eval_ctx;
+} ap_expr_eval_ctx_t;
 
 
 /**
@@ -143,7 +143,7 @@ typedef struct {
  * @param arg The (right) operand
  * @return 0 or 1
  */
-typedef int ap_expr_op_unary_t(ap_expr_eval_ctx *ctx, const void *data,
+typedef int ap_expr_op_unary_t(ap_expr_eval_ctx_t *ctx, const void *data,
                                const char *arg);
 
 /** Binary operator, takes two string arguments and returns a bool value.
@@ -154,7 +154,7 @@ typedef int ap_expr_op_unary_t(ap_expr_e
  * @param arg2 The right operand
  * @return 0 or 1
  */
-typedef int ap_expr_op_binary_t(ap_expr_eval_ctx *ctx, const void *data,
+typedef int ap_expr_op_binary_t(ap_expr_eval_ctx_t *ctx, const void *data,
                                 const char *arg1, const char *arg2);
 
 /** String valued function, takes a string argument and returns a string
@@ -163,7 +163,8 @@ typedef int ap_expr_op_binary_t(ap_expr_
  * @param arg The argument
  * @return The functions result string, may be NULL for 'empty string'
  */
-typedef const char *(ap_expr_string_func_t)(ap_expr_eval_ctx *ctx, const void *data,
+typedef const char *(ap_expr_string_func_t)(ap_expr_eval_ctx_t *ctx,
+                                            const void *data,
                                             const char *arg);
 
 /** List valued function, takes a string argument and returns a list of strings
@@ -173,7 +174,7 @@ typedef const char *(ap_expr_string_func
  * @param arg The argument
  * @return The functions result list of strings, may be NULL for 'empty array'
  */
-typedef apr_array_header_t *(ap_expr_list_func_t)(ap_expr_eval_ctx *ctx, const void *data,
+typedef apr_array_header_t *(ap_expr_list_func_t)(ap_expr_eval_ctx_t *ctx, const void *data,
                                                   const char *arg);
 
 /** Variable lookup function, takes no argument and returns a string
@@ -181,7 +182,7 @@ typedef apr_array_header_t *(ap_expr_lis
  * @param data An opaque context provided by the lookup hook function
  * @return The expanded variable
  */
-typedef const char *(ap_expr_var_func_t)(ap_expr_eval_ctx *ctx, const void *data);
+typedef const char *(ap_expr_var_func_t)(ap_expr_eval_ctx_t *ctx, const void *data);
 
 /** parameter struct passed to the lookup hook functions */
 typedef struct {
@@ -220,7 +221,7 @@ typedef struct {
  *          !OK on failure,
  *          DECLINED if the requested name is not handled by this function
  */
-typedef int (ap_expr_lookup_fn)(ap_expr_lookup_parms *parms);
+typedef int (ap_expr_lookup_fn_t)(ap_expr_lookup_parms *parms);
 
 /** Default lookup function which just calls ap_run_expr_lookup().
  *  ap_run_expr_lookup cannot be used directly because it has the wrong
@@ -243,7 +244,7 @@ AP_DECLARE_HOOK(int, expr_lookup, (ap_ex
  */
 AP_DECLARE(const char *) ap_expr_parse(apr_pool_t *pool, apr_pool_t *ptemp,
                                        ap_expr_info_t *info, const char *expr,
-                                       ap_expr_lookup_fn *lookup_fn);
+                                       ap_expr_lookup_fn_t *lookup_fn);
 
 /**
  * High level interface to ap_expr_parse that also creates ap_expr_info_t and
@@ -256,7 +257,7 @@ AP_DECLARE(const char *) ap_expr_parse(a
 AP_DECLARE(ap_expr_info_t *) ap_expr_parse_cmd(const cmd_parms *cmd,
                                                const char *expr,
                                                const char **err,
-                                               ap_expr_lookup_fn *lookup_fn);
+                                               ap_expr_lookup_fn_t *lookup_fn);
 
 
  /**

Modified: httpd/httpd/trunk/include/ap_mmn.h
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/include/ap_mmn.h?rev=1042146&r1=1042145&r2=1042146&view=diff
==============================================================================
--- httpd/httpd/trunk/include/ap_mmn.h (original)
+++ httpd/httpd/trunk/include/ap_mmn.h Sat Dec  4 11:22:30 2010
@@ -286,14 +286,15 @@
  * 20101113.0 (2.3.9-dev)  Add source address to mod_proxy.h
  * 20101113.1 (2.3.9-dev)  Add ap_set_flag_slot_char()
  * 20101113.2 (2.3.9-dev)  Add ap_expr_exec_re()
+ * 20101204.0 (2.3.10-dev) Add _t to ap_expr's typedef names
  */
 
 #define MODULE_MAGIC_COOKIE 0x41503234UL /* "AP24" */
 
 #ifndef MODULE_MAGIC_NUMBER_MAJOR
-#define MODULE_MAGIC_NUMBER_MAJOR 20101113
+#define MODULE_MAGIC_NUMBER_MAJOR 20101204
 #endif
-#define MODULE_MAGIC_NUMBER_MINOR 2                     /* 0...n */
+#define MODULE_MAGIC_NUMBER_MINOR 0                     /* 0...n */
 
 /**
  * Determine if the server's current MODULE_MAGIC_NUMBER is at least a

Modified: httpd/httpd/trunk/modules/metadata/mod_setenvif.c
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/metadata/mod_setenvif.c?rev=1042146&r1=1042145&r2=1042146&view=diff
==============================================================================
--- httpd/httpd/trunk/modules/metadata/mod_setenvif.c (original)
+++ httpd/httpd/trunk/modules/metadata/mod_setenvif.c Sat Dec  4 11:22:30 2010
@@ -644,7 +644,7 @@ static int match_headers(request_rec *r)
          * wasn't present or is undefined.  Represent that as an empty string
          * so that REs like "^$" will work and allow envariable setting
          * based on missing or empty field. This is also necessary to make
-         * ap_pregsub work after evaluating an ap_expr which does set the
+         * ap_pregsub work after evaluating an ap_expr_t which does set the
          * regexp backref data.
          */
         if (val == NULL) {

Modified: httpd/httpd/trunk/modules/ssl/ssl_engine_vars.c
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/ssl/ssl_engine_vars.c?rev=1042146&r1=1042145&r2=1042146&view=diff
==============================================================================
--- httpd/httpd/trunk/modules/ssl/ssl_engine_vars.c (original)
+++ httpd/httpd/trunk/modules/ssl/ssl_engine_vars.c Sat Dec  4 11:22:30 2010
@@ -63,14 +63,14 @@ static const char var_interface[] = "mod
 static char var_library_interface[] = SSL_LIBRARY_TEXT;
 static char *var_library = NULL;
 
-static apr_array_header_t *expr_peer_ext_list_fn(ap_expr_eval_ctx *ctx,
+static apr_array_header_t *expr_peer_ext_list_fn(ap_expr_eval_ctx_t *ctx,
                                                  const void *dummy,
                                                  const char *arg)
 {
     return ssl_ext_list(ctx->p, ctx->c, 1, arg);
 }
 
-static const char *expr_var_fn(ap_expr_eval_ctx *ctx, const void *data)
+static const char *expr_var_fn(ap_expr_eval_ctx_t *ctx, const void *data)
 {
     char *var = (char *)data;
     return ssl_var_lookup_ssl(ctx->p, ctx->c, var);

Modified: httpd/httpd/trunk/server/util_expr_eval.c
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/server/util_expr_eval.c?rev=1042146&r1=1042145&r2=1042146&view=diff
==============================================================================
--- httpd/httpd/trunk/server/util_expr_eval.c (original)
+++ httpd/httpd/trunk/server/util_expr_eval.c Sat Dec  4 11:22:30 2010
@@ -37,19 +37,22 @@ APR_HOOK_STRUCT(
 AP_IMPLEMENT_HOOK_RUN_FIRST(int, expr_lookup, (ap_expr_lookup_parms *parms),
                             (parms), DECLINED)
 
-static const char *ap_expr_eval_string_func(ap_expr_eval_ctx *ctx, const ap_expr *info,
-                                            const ap_expr *args);
-static const char *ap_expr_eval_re_backref(ap_expr_eval_ctx *ctx, int n);
-static const char *ap_expr_eval_var(ap_expr_eval_ctx *ctx,
+static const char *ap_expr_eval_string_func(ap_expr_eval_ctx_t *ctx,
+                                            const ap_expr_t *info,
+                                            const ap_expr_t *args);
+static const char *ap_expr_eval_re_backref(ap_expr_eval_ctx_t *ctx, int n);
+static const char *ap_expr_eval_var(ap_expr_eval_ctx_t *ctx,
                                     const ap_expr_var_func_t *func,
                                     const void *data);
 
 /* define AP_EXPR_DEBUG to log the parse tree when parsing an expression */
 #ifdef AP_EXPR_DEBUG
-static void expr_dump_tree(const ap_expr *e, const server_rec *s, int loglevel, int indent);
+static void expr_dump_tree(const ap_expr_t *e, const server_rec *s,
+                           int loglevel, int indent);
 #endif
 
-static const char *ap_expr_eval_word(ap_expr_eval_ctx *ctx, const ap_expr *node)
+static const char *ap_expr_eval_word(ap_expr_eval_ctx_t *ctx,
+                                     const ap_expr_t *node)
 {
     const char *result = "";
     switch (node->node_op) {
@@ -72,8 +75,8 @@ static const char *ap_expr_eval_word(ap_
             break;
         }
         case op_StringFuncCall: {
-            const ap_expr *info = node->node_arg1;
-            const ap_expr *args = node->node_arg2;
+            const ap_expr_t *info = node->node_arg1;
+            const ap_expr_t *args = node->node_arg2;
             result = ap_expr_eval_string_func(ctx, info, args);
             break;
         }
@@ -91,7 +94,7 @@ static const char *ap_expr_eval_word(ap_
     return result;
 }
 
-static const char *ap_expr_eval_var(ap_expr_eval_ctx *ctx, 
+static const char *ap_expr_eval_var(ap_expr_eval_ctx_t *ctx, 
                                     const ap_expr_var_func_t *func,
                                     const void *data)
 {
@@ -100,7 +103,7 @@ static const char *ap_expr_eval_var(ap_e
     return (*func)(ctx, data);
 }
 
-static const char *ap_expr_eval_re_backref(ap_expr_eval_ctx *ctx, int n)
+static const char *ap_expr_eval_re_backref(ap_expr_eval_ctx_t *ctx, int n)
 {
     int len;
 
@@ -115,8 +118,9 @@ static const char *ap_expr_eval_re_backr
     return apr_pstrndup(ctx->p, *ctx->re_source + ctx->re_pmatch[n].rm_so, len);
 }
 
-static const char *ap_expr_eval_string_func(ap_expr_eval_ctx *ctx, const ap_expr *info,
-                                            const ap_expr *arg)
+static const char *ap_expr_eval_string_func(ap_expr_eval_ctx_t *ctx,
+                                            const ap_expr_t *info,
+                                            const ap_expr_t *arg)
 {
     ap_expr_string_func_t *func = (ap_expr_string_func_t *)info->node_arg1;
     const void *data = info->node_arg2;
@@ -140,76 +144,76 @@ static int intstrcmp(const char *s1, con
         return 1;
 }
 
-static int ap_expr_eval_comp(ap_expr_eval_ctx *ctx, const ap_expr *node)
+static int ap_expr_eval_comp(ap_expr_eval_ctx_t *ctx, const ap_expr_t *node)
 {
     switch (node->node_op) {
         case op_EQ: {
-            const ap_expr *e1 = node->node_arg1;
-            const ap_expr *e2 = node->node_arg2;
+            const ap_expr_t *e1 = node->node_arg1;
+            const ap_expr_t *e2 = node->node_arg2;
             return (intstrcmp(ap_expr_eval_word(ctx, e1), ap_expr_eval_word(ctx, e2)) == 0);
         }
         case op_NE: {
-            const ap_expr *e1 = node->node_arg1;
-            const ap_expr *e2 = node->node_arg2;
+            const ap_expr_t *e1 = node->node_arg1;
+            const ap_expr_t *e2 = node->node_arg2;
             return (intstrcmp(ap_expr_eval_word(ctx, e1), ap_expr_eval_word(ctx, e2)) != 0);
         }
         case op_LT: {
-            const ap_expr *e1 = node->node_arg1;
-            const ap_expr *e2 = node->node_arg2;
+            const ap_expr_t *e1 = node->node_arg1;
+            const ap_expr_t *e2 = node->node_arg2;
             return (intstrcmp(ap_expr_eval_word(ctx, e1), ap_expr_eval_word(ctx, e2)) <  0);
         }
         case op_LE: {
-            const ap_expr *e1 = node->node_arg1;
-            const ap_expr *e2 = node->node_arg2;
+            const ap_expr_t *e1 = node->node_arg1;
+            const ap_expr_t *e2 = node->node_arg2;
             return (intstrcmp(ap_expr_eval_word(ctx, e1), ap_expr_eval_word(ctx, e2)) <= 0);
         }
         case op_GT: {
-            const ap_expr *e1 = node->node_arg1;
-            const ap_expr *e2 = node->node_arg2;
+            const ap_expr_t *e1 = node->node_arg1;
+            const ap_expr_t *e2 = node->node_arg2;
             return (intstrcmp(ap_expr_eval_word(ctx, e1), ap_expr_eval_word(ctx, e2)) >  0);
         }
         case op_GE: {
-            const ap_expr *e1 = node->node_arg1;
-            const ap_expr *e2 = node->node_arg2;
+            const ap_expr_t *e1 = node->node_arg1;
+            const ap_expr_t *e2 = node->node_arg2;
             return (intstrcmp(ap_expr_eval_word(ctx, e1), ap_expr_eval_word(ctx, e2)) >= 0);
         }
         case op_STR_EQ: {
-            const ap_expr *e1 = node->node_arg1;
-            const ap_expr *e2 = node->node_arg2;
+            const ap_expr_t *e1 = node->node_arg1;
+            const ap_expr_t *e2 = node->node_arg2;
             return (strcmp(ap_expr_eval_word(ctx, e1), ap_expr_eval_word(ctx, e2)) == 0);
         }
         case op_STR_NE: {
-            const ap_expr *e1 = node->node_arg1;
-            const ap_expr *e2 = node->node_arg2;
+            const ap_expr_t *e1 = node->node_arg1;
+            const ap_expr_t *e2 = node->node_arg2;
             return (strcmp(ap_expr_eval_word(ctx, e1), ap_expr_eval_word(ctx, e2)) != 0);
         }
         case op_STR_LT: {
-            const ap_expr *e1 = node->node_arg1;
-            const ap_expr *e2 = node->node_arg2;
+            const ap_expr_t *e1 = node->node_arg1;
+            const ap_expr_t *e2 = node->node_arg2;
             return (strcmp(ap_expr_eval_word(ctx, e1), ap_expr_eval_word(ctx, e2)) <  0);
         }
         case op_STR_LE: {
-            const ap_expr *e1 = node->node_arg1;
-            const ap_expr *e2 = node->node_arg2;
+            const ap_expr_t *e1 = node->node_arg1;
+            const ap_expr_t *e2 = node->node_arg2;
             return (strcmp(ap_expr_eval_word(ctx, e1), ap_expr_eval_word(ctx, e2)) <= 0);
         }
         case op_STR_GT: {
-            const ap_expr *e1 = node->node_arg1;
-            const ap_expr *e2 = node->node_arg2;
+            const ap_expr_t *e1 = node->node_arg1;
+            const ap_expr_t *e2 = node->node_arg2;
             return (strcmp(ap_expr_eval_word(ctx, e1), ap_expr_eval_word(ctx, e2)) >  0);
         }
         case op_STR_GE: {
-            const ap_expr *e1 = node->node_arg1;
-            const ap_expr *e2 = node->node_arg2;
+            const ap_expr_t *e1 = node->node_arg1;
+            const ap_expr_t *e2 = node->node_arg2;
             return (strcmp(ap_expr_eval_word(ctx, e1), ap_expr_eval_word(ctx, e2)) >= 0);
         }
         case op_IN: {
-            const ap_expr *e1 = node->node_arg1;
-            const ap_expr *e2 = node->node_arg2;
+            const ap_expr_t *e1 = node->node_arg1;
+            const ap_expr_t *e2 = node->node_arg2;
             const char *needle = ap_expr_eval_word(ctx, e1);
             if (e2->node_op == op_ListElement) {
                 do {
-                    const ap_expr *val = e2->node_arg1;
+                    const ap_expr_t *val = e2->node_arg1;
                     AP_DEBUG_ASSERT(e2->node_op == op_ListElement);
                     if (strcmp(needle, ap_expr_eval_word(ctx, val)) == 0) {
                         return 1;
@@ -219,8 +223,8 @@ static int ap_expr_eval_comp(ap_expr_eva
                 } while (e2 != NULL);
             }
             else if (e2->node_op == op_ListFuncCall) {
-                const ap_expr *info = e2->node_arg1;
-                const ap_expr *arg = e2->node_arg2;
+                const ap_expr_t *info = e2->node_arg1;
+                const ap_expr_t *arg = e2->node_arg2;
                 ap_expr_list_func_t *func = (ap_expr_list_func_t *)info->node_arg1;
                 apr_array_header_t *haystack;
                 int i = 0;
@@ -238,8 +242,8 @@ static int ap_expr_eval_comp(ap_expr_eva
         }
         case op_REG:
         case op_NRE: {
-            const ap_expr *e1;
-            const ap_expr *e2;
+            const ap_expr_t *e1;
+            const ap_expr_t *e2;
             const char *word;
             const ap_regex_t *regex;
             int result;
@@ -298,10 +302,10 @@ static int strcmplex(const char *str1, c
     return 0;
 }
 
-static int ssl_expr_eval_comp(ap_expr_eval_ctx *ctx, const ap_expr *node)
+static int ssl_expr_eval_comp(ap_expr_eval_ctx_t *ctx, const ap_expr_t *node)
 {
-    const ap_expr *e1 = node->node_arg1;
-    const ap_expr *e2 = node->node_arg2;
+    const ap_expr_t *e1 = node->node_arg1;
+    const ap_expr_t *e2 = node->node_arg2;
     switch (node->node_op) {
     case op_EQ:
     case op_STR_EQ:
@@ -333,9 +337,9 @@ AP_DECLARE_NONSTD(int) ap_expr_lookup_de
 
 AP_DECLARE(const char *) ap_expr_parse(apr_pool_t *pool, apr_pool_t *ptemp,
                                        ap_expr_info_t *info, const char *expr,
-                                       ap_expr_lookup_fn *lookup_fn)
+                                       ap_expr_lookup_fn_t *lookup_fn)
 {
-    ap_expr_parse_ctx ctx;
+    ap_expr_parse_ctx_t ctx;
     int rc;
 
     ctx.pool     = pool;
@@ -391,7 +395,7 @@ AP_DECLARE(const char *) ap_expr_parse(a
 AP_DECLARE(ap_expr_info_t*) ap_expr_parse_cmd(const cmd_parms *cmd,
                                               const char *expr,
                                               const char **err,
-                                              ap_expr_lookup_fn *lookup_fn)
+                                              ap_expr_lookup_fn_t *lookup_fn)
 {
     ap_expr_info_t *info = apr_pcalloc(cmd->pool, sizeof(ap_expr_info_t));
     info->filename = cmd->directive->filename;
@@ -404,21 +408,21 @@ AP_DECLARE(ap_expr_info_t*) ap_expr_pars
     return info;
 }
 
-ap_expr *ap_expr_make(ap_expr_node_op op, const void *a1, const void *a2,
-                      ap_expr_parse_ctx *ctx)
+ap_expr_t *ap_expr_make(ap_expr_node_op_e op, const void *a1, const void *a2,
+                      ap_expr_parse_ctx_t *ctx)
 {
-    ap_expr *node = apr_palloc(ctx->pool, sizeof(ap_expr));
+    ap_expr_t *node = apr_palloc(ctx->pool, sizeof(ap_expr_t));
     node->node_op   = op;
     node->node_arg1 = a1;
     node->node_arg2 = a2;
     return node;
 }
 
-static ap_expr *ap_expr_info_make(int type, const char *name,
-                                  ap_expr_parse_ctx *ctx,
-                                  const ap_expr *arg)
+static ap_expr_t *ap_expr_info_make(int type, const char *name,
+                                  ap_expr_parse_ctx_t *ctx,
+                                  const ap_expr_t *arg)
 {
-    ap_expr *info = apr_palloc(ctx->pool, sizeof(ap_expr));
+    ap_expr_t *info = apr_palloc(ctx->pool, sizeof(ap_expr_t));
     ap_expr_lookup_parms parms;
     parms.type  = type;
     parms.flags = 0;
@@ -434,10 +438,10 @@ static ap_expr *ap_expr_info_make(int ty
     return info;
 }
 
-ap_expr *ap_expr_str_func_make(const char *name, const ap_expr *arg,
-                               ap_expr_parse_ctx *ctx)
+ap_expr_t *ap_expr_str_func_make(const char *name, const ap_expr_t *arg,
+                               ap_expr_parse_ctx_t *ctx)
 {
-    ap_expr *info = ap_expr_info_make(AP_EXPR_FUNC_STRING, name, ctx, arg);
+    ap_expr_t *info = ap_expr_info_make(AP_EXPR_FUNC_STRING, name, ctx, arg);
     if (!info)
         return NULL;
 
@@ -445,10 +449,10 @@ ap_expr *ap_expr_str_func_make(const cha
     return ap_expr_make(op_StringFuncCall, info, arg, ctx);
 }
 
-ap_expr *ap_expr_list_func_make(const char *name, const ap_expr *arg,
-                                ap_expr_parse_ctx *ctx)
+ap_expr_t *ap_expr_list_func_make(const char *name, const ap_expr_t *arg,
+                                ap_expr_parse_ctx_t *ctx)
 {
-    ap_expr *info = ap_expr_info_make(AP_EXPR_FUNC_LIST, name, ctx, arg);
+    ap_expr_t *info = ap_expr_info_make(AP_EXPR_FUNC_LIST, name, ctx, arg);
     if (!info)
         return NULL;
 
@@ -456,10 +460,10 @@ ap_expr *ap_expr_list_func_make(const ch
     return ap_expr_make(op_ListFuncCall, info, arg, ctx);
 }
 
-ap_expr *ap_expr_unary_op_make(const char *name, const ap_expr *arg,
-                               ap_expr_parse_ctx *ctx)
+ap_expr_t *ap_expr_unary_op_make(const char *name, const ap_expr_t *arg,
+                               ap_expr_parse_ctx_t *ctx)
 {
-    ap_expr *info = ap_expr_info_make(AP_EXPR_FUNC_OP_UNARY, name, ctx, arg);
+    ap_expr_t *info = ap_expr_info_make(AP_EXPR_FUNC_OP_UNARY, name, ctx, arg);
     if (!info)
         return NULL;
 
@@ -467,11 +471,12 @@ ap_expr *ap_expr_unary_op_make(const cha
     return ap_expr_make(op_UnaryOpCall, info, arg, ctx);
 }
 
-ap_expr *ap_expr_binary_op_make(const char *name, const ap_expr *arg1,
-                                const ap_expr *arg2, ap_expr_parse_ctx *ctx)
+ap_expr_t *ap_expr_binary_op_make(const char *name, const ap_expr_t *arg1,
+                                const ap_expr_t *arg2, ap_expr_parse_ctx_t *ctx)
 {
-    ap_expr *args;
-    ap_expr *info = ap_expr_info_make(AP_EXPR_FUNC_OP_BINARY, name, ctx, arg2);
+    ap_expr_t *args;
+    ap_expr_t *info = ap_expr_info_make(AP_EXPR_FUNC_OP_BINARY, name, ctx,
+                                        arg2);
     if (!info)
         return NULL;
 
@@ -481,9 +486,9 @@ ap_expr *ap_expr_binary_op_make(const ch
 }
 
 
-ap_expr *ap_expr_var_make(const char *name, ap_expr_parse_ctx *ctx)
+ap_expr_t *ap_expr_var_make(const char *name, ap_expr_parse_ctx_t *ctx)
 {
-    ap_expr *node = ap_expr_info_make(AP_EXPR_FUNC_VAR, name, ctx, NULL);
+    ap_expr_t *node = ap_expr_info_make(AP_EXPR_FUNC_VAR, name, ctx, NULL);
     if (!node)
         return NULL;
 
@@ -511,14 +516,15 @@ ap_expr *ap_expr_var_make(const char *na
     ap_log_error(MARK,"%*s%s: '%s' '%s'", indent, " ", op, (char *)s1, (char *)s2)
 #define DUMP_P(op, p1)                                                      \
     ap_log_error(MARK,"%*s%s: %pp", indent, " ", op, p1);
-#define DUMP_IP(op, p1)                                                      \
+#define DUMP_IP(op, p1)                                                     \
     ap_log_error(MARK,"%*s%s: %d", indent, " ", op, *(int *)p1);
 #define DUMP_S(op, s1)                                                      \
     ap_log_error(MARK,"%*s%s: '%s'", indent, " ", op, (char *)s1)
 
 #define CASE_OP(op)                  case op: name = #op ; break;
 
-static void expr_dump_tree(const ap_expr *e, const server_rec *s, int loglevel, int indent)
+static void expr_dump_tree(const ap_expr_t *e, const server_rec *s,
+                           int loglevel, int indent)
 {
     switch (e->node_op) {
     /* no arg */
@@ -660,8 +666,8 @@ static void expr_dump_tree(const ap_expr
 }
 #endif /* AP_EXPR_DEBUG */
 
-static int ap_expr_eval_unary_op(ap_expr_eval_ctx *ctx, const ap_expr *info,
-                                 const ap_expr *arg)
+static int ap_expr_eval_unary_op(ap_expr_eval_ctx_t *ctx, const ap_expr_t *info,
+                                 const ap_expr_t *arg)
 {
     const ap_expr_op_unary_t *op_func = info->node_arg1;
     const void *data = info->node_arg2;
@@ -672,13 +678,14 @@ static int ap_expr_eval_unary_op(ap_expr
     return (*op_func)(ctx, data, ap_expr_eval_word(ctx, arg));
 }
 
-static int ap_expr_eval_binary_op(ap_expr_eval_ctx *ctx, const ap_expr *info,
-                                  const ap_expr *args)
+static int ap_expr_eval_binary_op(ap_expr_eval_ctx_t *ctx,
+                                  const ap_expr_t *info,
+                                  const ap_expr_t *args)
 {
     const ap_expr_op_binary_t *op_func = info->node_arg1;
     const void *data = info->node_arg2;
-    const ap_expr *a1 = args->node_arg1;
-    const ap_expr *a2 = args->node_arg2;
+    const ap_expr_t *a1 = args->node_arg1;
+    const ap_expr_t *a2 = args->node_arg2;
 
     AP_DEBUG_ASSERT(info->node_op == op_BinaryOpInfo);
     AP_DEBUG_ASSERT(args->node_op == op_BinaryOpArgs);
@@ -689,7 +696,7 @@ static int ap_expr_eval_binary_op(ap_exp
 }
 
 
-static int ap_expr_eval(ap_expr_eval_ctx *ctx, const ap_expr *node)
+static int ap_expr_eval(ap_expr_eval_ctx_t *ctx, const ap_expr_t *node)
 {
     switch (node->node_op) {
         case op_True: {
@@ -699,31 +706,31 @@ static int ap_expr_eval(ap_expr_eval_ctx
             return 0;
         }
         case op_Not: {
-            const ap_expr *e = node->node_arg1;
+            const ap_expr_t *e = node->node_arg1;
             return (!ap_expr_eval(ctx, e));
         }
         case op_Or: {
-            const ap_expr *e1 = node->node_arg1;
-            const ap_expr *e2 = node->node_arg2;
+            const ap_expr_t *e1 = node->node_arg1;
+            const ap_expr_t *e2 = node->node_arg2;
             return (ap_expr_eval(ctx, e1) || ap_expr_eval(ctx, e2));
         }
         case op_And: {
-            const ap_expr *e1 = node->node_arg1;
-            const ap_expr *e2 = node->node_arg2;
+            const ap_expr_t *e1 = node->node_arg1;
+            const ap_expr_t *e2 = node->node_arg2;
             return (ap_expr_eval(ctx, e1) && ap_expr_eval(ctx, e2));
         }
         case op_UnaryOpCall: {
-            const ap_expr *info = node->node_arg1;
-            const ap_expr *args = node->node_arg2;
+            const ap_expr_t *info = node->node_arg1;
+            const ap_expr_t *args = node->node_arg2;
             return ap_expr_eval_unary_op(ctx, info, args);
         }
         case op_BinaryOpCall: {
-            const ap_expr *info = node->node_arg1;
-            const ap_expr *args = node->node_arg2;
+            const ap_expr_t *info = node->node_arg1;
+            const ap_expr_t *args = node->node_arg2;
             return ap_expr_eval_binary_op(ctx, info, args);
         }
         case op_Comp: {
-            const ap_expr *e = node->node_arg1;
+            const ap_expr_t *e = node->node_arg1;
             if (ctx->info->flags & AP_EXPR_FLAGS_SSL_EXPR_COMPAT)
                 return ssl_expr_eval_comp(ctx, e);
             else
@@ -736,7 +743,8 @@ static int ap_expr_eval(ap_expr_eval_ctx
     }
 }
 
-AP_DECLARE(int) ap_expr_exec(request_rec *r, const ap_expr_info_t *info, const char **err)
+AP_DECLARE(int) ap_expr_exec(request_rec *r, const ap_expr_info_t *info,
+                             const char **err)
 {
     return ap_expr_exec_re(r, info, 0, NULL, NULL, err);
 }
@@ -745,7 +753,7 @@ AP_DECLARE(int) ap_expr_exec_re(request_
                                 apr_size_t nmatch, ap_regmatch_t *pmatch,
                                 const char **source, const char **err)
 {
-    ap_expr_eval_ctx ctx;
+    ap_expr_eval_ctx_t ctx;
     int rc;
     int dont_vary = (info->flags & AP_EXPR_FLAGS_DONT_VARY);
     const char *tmp_source = NULL, *vary_this = NULL;
@@ -792,7 +800,7 @@ AP_DECLARE(int) ap_expr_exec_re(request_
     }
 }
 
-static void add_vary(ap_expr_eval_ctx *ctx, const char *name)
+static void add_vary(ap_expr_eval_ctx_t *ctx, const char *name)
 {
     if (!ctx->vary_this)
         return;
@@ -806,7 +814,7 @@ static void add_vary(ap_expr_eval_ctx *c
     }
 }
 
-static const char *req_table_func(ap_expr_eval_ctx *ctx, const void *data,
+static const char *req_table_func(ap_expr_eval_ctx_t *ctx, const void *data,
                                   const char *arg)
 {
     const char *name = (const char *)data;
@@ -827,7 +835,7 @@ static const char *req_table_func(ap_exp
     return apr_table_get(t, arg);
 }
 
-static const char *env_func(ap_expr_eval_ctx *ctx, const void *data,
+static const char *env_func(ap_expr_eval_ctx_t *ctx, const void *data,
                             const char *arg)
 {
     const char *res;
@@ -841,13 +849,13 @@ static const char *env_func(ap_expr_eval
     return getenv(arg);
 }
 
-static const char *osenv_func(ap_expr_eval_ctx *ctx, const void *data,
+static const char *osenv_func(ap_expr_eval_ctx_t *ctx, const void *data,
                               const char *arg)
 {
     return getenv(arg);
 }
 
-static const char *tolower_func(ap_expr_eval_ctx *ctx, const void *data,
+static const char *tolower_func(ap_expr_eval_ctx_t *ctx, const void *data,
                                 const char *arg)
 {
     char *result = apr_pstrdup(ctx->p, arg);
@@ -855,7 +863,7 @@ static const char *tolower_func(ap_expr_
     return result;
 }
 
-static const char *toupper_func(ap_expr_eval_ctx *ctx, const void *data,
+static const char *toupper_func(ap_expr_eval_ctx_t *ctx, const void *data,
                                 const char *arg)
 {
     char *p;
@@ -868,14 +876,15 @@ static const char *toupper_func(ap_expr_
     return result;
 }
 
-static const char *escape_func(ap_expr_eval_ctx *ctx, const void *data,
+static const char *escape_func(ap_expr_eval_ctx_t *ctx, const void *data,
                                const char *arg)
 {
     return ap_escape_uri(ctx->p, arg);
 }
 
 #define MAX_FILE_SIZE 10*1024*1024
-static const char *file_func(ap_expr_eval_ctx *ctx, const void *data, char *arg)
+static const char *file_func(ap_expr_eval_ctx_t *ctx, const void *data,
+                             char *arg)
 {
     apr_file_t *fp;
     char *buf;
@@ -919,7 +928,7 @@ static const char *file_func(ap_expr_eva
 }
 
 
-static const char *unescape_func(ap_expr_eval_ctx *ctx, const void *data,
+static const char *unescape_func(ap_expr_eval_ctx_t *ctx, const void *data,
                                  const char *arg)
 {
     char *result = apr_pstrdup(ctx->p, arg);
@@ -930,7 +939,7 @@ static const char *unescape_func(ap_expr
 
 }
 
-static int op_nz(ap_expr_eval_ctx *ctx, const void *data, const char *arg)
+static int op_nz(ap_expr_eval_ctx_t *ctx, const void *data, const char *arg)
 {
     const char *name = (const char *)data;
     if (name[0] == 'z')
@@ -950,7 +959,7 @@ static const char *conn_var_names[] = {
     NULL
 };
 
-static const char *conn_var_fn(ap_expr_eval_ctx *ctx, const void *data)
+static const char *conn_var_fn(ap_expr_eval_ctx_t *ctx, const void *data)
 {
     int index = ((const char **)data - conn_var_names);
     conn_rec *c = ctx->c;
@@ -1011,7 +1020,7 @@ static const char *request_var_names[] =
     NULL
 };
 
-static const char *request_var_fn(ap_expr_eval_ctx *ctx, const void *data)
+static const char *request_var_fn(ap_expr_eval_ctx_t *ctx, const void *data)
 {
     int index = ((const char **)data - request_var_names);
     request_rec *r = ctx->r;
@@ -1089,7 +1098,7 @@ static const char *req_header_header_nam
     "Accept"
 };
 
-static const char *req_header_var_fn(ap_expr_eval_ctx *ctx, const void *data)
+static const char *req_header_var_fn(ap_expr_eval_ctx_t *ctx, const void *data)
 {
     const char **varname = (const char **)data;
     int index = (varname - req_header_var_names);
@@ -1118,7 +1127,7 @@ static const char *misc_var_names[] = {
     NULL
 };
 
-static const char *misc_var_fn(ap_expr_eval_ctx *ctx, const void *data)
+static const char *misc_var_fn(ap_expr_eval_ctx_t *ctx, const void *data)
 {
     apr_time_exp_t tm;
     int index = ((const char **)data - misc_var_names);
@@ -1186,7 +1195,7 @@ static int subnet_parse_arg(ap_expr_look
     return OK;
 }
 
-static int op_ipmatch(ap_expr_eval_ctx *ctx, const void *data, const char *arg1,
+static int op_ipmatch(ap_expr_eval_ctx_t *ctx, const void *data, const char *arg1,
                 const char *arg2)
 {
     apr_ipsubnet_t *subnet = (apr_ipsubnet_t *)data;
@@ -1201,7 +1210,7 @@ static int op_ipmatch(ap_expr_eval_ctx *
     return apr_ipsubnet_test(subnet, saddr);
 }
 
-static int op_R(ap_expr_eval_ctx *ctx, const void *data, const char *arg1)
+static int op_R(ap_expr_eval_ctx_t *ctx, const void *data, const char *arg1)
 {
     apr_ipsubnet_t *subnet = (apr_ipsubnet_t *)data;
 
@@ -1213,20 +1222,20 @@ static int op_R(ap_expr_eval_ctx *ctx, c
     return apr_ipsubnet_test(subnet, ctx->c->remote_addr);
 }
 
-static int op_fnmatch(ap_expr_eval_ctx *ctx, const void *data, const char *arg1,
-                       const char *arg2)
+static int op_fnmatch(ap_expr_eval_ctx_t *ctx, const void *data,
+                      const char *arg1, const char *arg2)
 {
     return (APR_SUCCESS == apr_fnmatch(arg2, arg1, APR_FNM_PATHNAME));
 }
 
-static int op_strmatch(ap_expr_eval_ctx *ctx, const void *data, const char *arg1,
-                       const char *arg2)
+static int op_strmatch(ap_expr_eval_ctx_t *ctx, const void *data,
+                       const char *arg1, const char *arg2)
 {
     return (APR_SUCCESS == apr_fnmatch(arg2, arg1, 0));
 }
 
-static int op_strcmatch(ap_expr_eval_ctx *ctx, const void *data, const char *arg1,
-                       const char *arg2)
+static int op_strcmatch(ap_expr_eval_ctx_t *ctx, const void *data,
+                        const char *arg1, const char *arg2)
 {
     return (APR_SUCCESS == apr_fnmatch(arg2, arg1, APR_FNM_CASE_BLIND));
 }
@@ -1234,7 +1243,7 @@ static int op_strcmatch(ap_expr_eval_ctx
 struct expr_provider_single {
     const void *func;
     const char *name;
-    ap_expr_lookup_fn *arg_parsing_func;
+    ap_expr_lookup_fn_t *arg_parsing_func;
 };
 
 struct expr_provider_multi {

Modified: httpd/httpd/trunk/server/util_expr_parse.c
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/server/util_expr_parse.c?rev=1042146&r1=1042145&r2=1042146&view=diff
==============================================================================
--- httpd/httpd/trunk/server/util_expr_parse.c (original)
+++ httpd/httpd/trunk/server/util_expr_parse.c Sat Dec  4 11:22:30 2010
@@ -155,9 +155,9 @@ typedef union YYSTYPE
 /* Line 214 of yacc.c  */
 #line 35 "util_expr_parse.y"
 
-    char    *cpVal;
-    ap_expr *exVal;
-    int      num;
+    char      *cpVal;
+    ap_expr_t *exVal;
+    int        num;
 
 
 
@@ -777,14 +777,14 @@ do {									  \
 #if (defined __STDC__ || defined __C99__FUNC__ \
      || defined __cplusplus || defined _MSC_VER)
 static void
-yy_symbol_value_print (FILE *yyoutput, int yytype, YYSTYPE const * const yyvaluep, ap_expr_parse_ctx *ctx)
+yy_symbol_value_print (FILE *yyoutput, int yytype, YYSTYPE const * const yyvaluep, ap_expr_parse_ctx_t *ctx)
 #else
 static void
 yy_symbol_value_print (yyoutput, yytype, yyvaluep, ctx)
     FILE *yyoutput;
     int yytype;
     YYSTYPE const * const yyvaluep;
-    ap_expr_parse_ctx *ctx;
+    ap_expr_parse_ctx_t *ctx;
 #endif
 {
   if (!yyvaluep)
@@ -811,14 +811,14 @@ yy_symbol_value_print (yyoutput, yytype,
 #if (defined __STDC__ || defined __C99__FUNC__ \
      || defined __cplusplus || defined _MSC_VER)
 static void
-yy_symbol_print (FILE *yyoutput, int yytype, YYSTYPE const * const yyvaluep, ap_expr_parse_ctx *ctx)
+yy_symbol_print (FILE *yyoutput, int yytype, YYSTYPE const * const yyvaluep, ap_expr_parse_ctx_t *ctx)
 #else
 static void
 yy_symbol_print (yyoutput, yytype, yyvaluep, ctx)
     FILE *yyoutput;
     int yytype;
     YYSTYPE const * const yyvaluep;
-    ap_expr_parse_ctx *ctx;
+    ap_expr_parse_ctx_t *ctx;
 #endif
 {
   if (yytype < YYNTOKENS)
@@ -869,13 +869,13 @@ do {								\
 #if (defined __STDC__ || defined __C99__FUNC__ \
      || defined __cplusplus || defined _MSC_VER)
 static void
-yy_reduce_print (YYSTYPE *yyvsp, int yyrule, ap_expr_parse_ctx *ctx)
+yy_reduce_print (YYSTYPE *yyvsp, int yyrule, ap_expr_parse_ctx_t *ctx)
 #else
 static void
 yy_reduce_print (yyvsp, yyrule, ctx)
     YYSTYPE *yyvsp;
     int yyrule;
-    ap_expr_parse_ctx *ctx;
+    ap_expr_parse_ctx_t *ctx;
 #endif
 {
   int yynrhs = yyr2[yyrule];
@@ -1148,14 +1148,14 @@ yysyntax_error (char *yyresult, int yyst
 #if (defined __STDC__ || defined __C99__FUNC__ \
      || defined __cplusplus || defined _MSC_VER)
 static void
-yydestruct (const char *yymsg, int yytype, YYSTYPE *yyvaluep, ap_expr_parse_ctx *ctx)
+yydestruct (const char *yymsg, int yytype, YYSTYPE *yyvaluep, ap_expr_parse_ctx_t *ctx)
 #else
 static void
 yydestruct (yymsg, yytype, yyvaluep, ctx)
     const char *yymsg;
     int yytype;
     YYSTYPE *yyvaluep;
-    ap_expr_parse_ctx *ctx;
+    ap_expr_parse_ctx_t *ctx;
 #endif
 {
   YYUSE (yyvaluep);
@@ -1182,7 +1182,7 @@ int yyparse ();
 #endif
 #else /* ! YYPARSE_PARAM */
 #if defined __STDC__ || defined __cplusplus
-int yyparse (ap_expr_parse_ctx *ctx);
+int yyparse (ap_expr_parse_ctx_t *ctx);
 #else
 int yyparse ();
 #endif
@@ -1210,11 +1210,11 @@ yyparse (YYPARSE_PARAM)
 #if (defined __STDC__ || defined __C99__FUNC__ \
      || defined __cplusplus || defined _MSC_VER)
 int
-yyparse (ap_expr_parse_ctx *ctx)
+yyparse (ap_expr_parse_ctx_t *ctx)
 #else
 int
 yyparse (ctx)
-    ap_expr_parse_ctx *ctx;
+    ap_expr_parse_ctx_t *ctx;
 #endif
 #endif
 {
@@ -2047,7 +2047,7 @@ yyreturn:
 #line 204 "util_expr_parse.y"
 
 
-void yyerror(ap_expr_parse_ctx *ctx, char *s)
+void yyerror(ap_expr_parse_ctx_t *ctx, char *s)
 {
     /* s is allocated on the stack */
     ctx->error = apr_pstrdup(ctx->ptemp, s);

Modified: httpd/httpd/trunk/server/util_expr_parse.h
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/server/util_expr_parse.h?rev=1042146&r1=1042145&r2=1042146&view=diff
==============================================================================
--- httpd/httpd/trunk/server/util_expr_parse.h (original)
+++ httpd/httpd/trunk/server/util_expr_parse.h Sat Dec  4 11:22:30 2010
@@ -85,9 +85,9 @@ typedef union YYSTYPE
 /* Line 1676 of yacc.c  */
 #line 35 "util_expr_parse.y"
 
-    char    *cpVal;
-    ap_expr *exVal;
-    int      num;
+    char      *cpVal;
+    ap_expr_t *exVal;
+    int        num;
 
 
 

Modified: httpd/httpd/trunk/server/util_expr_parse.y
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/server/util_expr_parse.y?rev=1042146&r1=1042145&r2=1042146&view=diff
==============================================================================
--- httpd/httpd/trunk/server/util_expr_parse.y (original)
+++ httpd/httpd/trunk/server/util_expr_parse.y Sat Dec  4 11:22:30 2010
@@ -26,16 +26,16 @@
 %error-verbose
 %defines
 %lex-param   { void *yyscanner }
-%parse-param { ap_expr_parse_ctx *ctx }
+%parse-param { ap_expr_parse_ctx_t *ctx }
 
 %{
 #include "util_expr_private.h"
 %}
 
 %union {
-    char    *cpVal;
-    ap_expr *exVal;
-    int      num;
+    char      *cpVal;
+    ap_expr_t *exVal;
+    int        num;
 }
 
 %token  T_TRUE
@@ -203,7 +203,7 @@ strfunccall : T_ID '(' word ')' { $$ = a
 
 %%
 
-void yyerror(ap_expr_parse_ctx *ctx, char *s)
+void yyerror(ap_expr_parse_ctx_t *ctx, char *s)
 {
     /* s is allocated on the stack */
     ctx->error = apr_pstrdup(ctx->ptemp, s);

Modified: httpd/httpd/trunk/server/util_expr_private.h
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/server/util_expr_private.h?rev=1042146&r1=1042145&r2=1042146&view=diff
==============================================================================
--- httpd/httpd/trunk/server/util_expr_private.h (original)
+++ httpd/httpd/trunk/server/util_expr_private.h Sat Dec  4 11:22:30 2010
@@ -64,11 +64,11 @@ typedef enum {
     op_BinaryOpCall, op_BinaryOpInfo, op_BinaryOpArgs,
     op_StringFuncCall, op_StringFuncInfo,
     op_ListFuncCall, op_ListFuncInfo
-} ap_expr_node_op;
+} ap_expr_node_op_e;
 
 /** The basic parse tree node */
 struct ap_expr_node {
-    ap_expr_node_op node_op;
+    ap_expr_node_op_e node_op;
     const void *node_arg1;
     const void *node_arg2;
 };
@@ -89,7 +89,7 @@ typedef struct {
     apr_pool_t        *ptemp;
 
     /* The created parse tree */
-    ap_expr           *expr;
+    ap_expr_t         *expr;
 
     const char        *error;
     const char        *error2;
@@ -99,33 +99,34 @@ typedef struct {
      * The function to use to lookup provider functions for variables
      * and funtctions
      */
-    ap_expr_lookup_fn   *lookup_fn;
-} ap_expr_parse_ctx;
+    ap_expr_lookup_fn_t *lookup_fn;
+} ap_expr_parse_ctx_t;
 
 /* flex/bison functions */
-int  ap_expr_yyparse(ap_expr_parse_ctx *context);
-void ap_expr_yyerror(ap_expr_parse_ctx *context, char *err);
+int  ap_expr_yyparse(ap_expr_parse_ctx_t *context);
+void ap_expr_yyerror(ap_expr_parse_ctx_t *context, char *err);
 int  ap_expr_yylex_init(void **scanner);
 int  ap_expr_yylex_destroy(void *scanner);
-void ap_expr_yyset_extra(ap_expr_parse_ctx *context, void *scanner);
+void ap_expr_yyset_extra(ap_expr_parse_ctx_t *context, void *scanner);
 
 /* create a parse tree node */
-ap_expr *ap_expr_make(ap_expr_node_op op, const void *arg1, const void *arg2,
-                        ap_expr_parse_ctx *ctx);
+ap_expr_t *ap_expr_make(ap_expr_node_op_e op, const void *arg1,
+                        const void *arg2, ap_expr_parse_ctx_t *ctx);
 /* create parse tree node for the string-returning function 'name' */
-ap_expr *ap_expr_str_func_make(const char *name, const ap_expr *arg,
-                               ap_expr_parse_ctx *ctx);
+ap_expr_t *ap_expr_str_func_make(const char *name, const ap_expr_t *arg,
+                               ap_expr_parse_ctx_t *ctx);
 /* create parse tree node for the list-returning function 'name' */
-ap_expr *ap_expr_list_func_make(const char *name, const ap_expr *arg,
-                                ap_expr_parse_ctx *ctx);
+ap_expr_t *ap_expr_list_func_make(const char *name, const ap_expr_t *arg,
+                                ap_expr_parse_ctx_t *ctx);
 /* create parse tree node for the variable 'name' */
-ap_expr *ap_expr_var_make(const char *name, ap_expr_parse_ctx *ctx);
+ap_expr_t *ap_expr_var_make(const char *name, ap_expr_parse_ctx_t *ctx);
 /* create parse tree node for the unary operator 'name' */
-ap_expr *ap_expr_unary_op_make(const char *name, const ap_expr *arg,
-                               ap_expr_parse_ctx *ctx);
+ap_expr_t *ap_expr_unary_op_make(const char *name, const ap_expr_t *arg,
+                               ap_expr_parse_ctx_t *ctx);
 /* create parse tree node for the binary operator 'name' */
-ap_expr *ap_expr_binary_op_make(const char *name, const ap_expr *arg1,
-                                const ap_expr *arg2, ap_expr_parse_ctx *ctx);
+ap_expr_t *ap_expr_binary_op_make(const char *name, const ap_expr_t *arg1,
+                                  const ap_expr_t *arg2,
+                                  ap_expr_parse_ctx_t *ctx);
 
 
 #endif /* __AP_EXPR_PRIVATE_H__ */

Modified: httpd/httpd/trunk/server/util_expr_scan.c
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/server/util_expr_scan.c?rev=1042146&r1=1042145&r2=1042146&view=diff
==============================================================================
--- httpd/httpd/trunk/server/util_expr_scan.c (original)
+++ httpd/httpd/trunk/server/util_expr_scan.c Sat Dec  4 11:22:30 2010
@@ -597,7 +597,7 @@ static yyconst flex_int16_t yy_chk[319] 
     }                                                       \
 }
 
-#define YY_EXTRA_TYPE ap_expr_parse_ctx*
+#define YY_EXTRA_TYPE ap_expr_parse_ctx_t*
 
 #define PERROR(msg) yyextra->error2 = msg ; return T_ERROR;
 

Modified: httpd/httpd/trunk/server/util_expr_scan.l
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/server/util_expr_scan.l?rev=1042146&r1=1042145&r2=1042146&view=diff
==============================================================================
--- httpd/httpd/trunk/server/util_expr_scan.l (original)
+++ httpd/httpd/trunk/server/util_expr_scan.l Sat Dec  4 11:22:30 2010
@@ -58,7 +58,7 @@
     }                                                       \
 }
 
-#define YY_EXTRA_TYPE ap_expr_parse_ctx*
+#define YY_EXTRA_TYPE ap_expr_parse_ctx_t*
 
 #define PERROR(msg) yyextra->error2 = msg ; return T_ERROR;