mturk       2004/12/07 05:46:56

  Modified:    jk/native/apache-1.3 mod_jk.c
               jk/native/apache-2.0 mod_jk.c
               jk/native/common jk_map.c
  Log:
  Remove the need for supplying the pointer to old value for jk_map_put
  
  Revision  Changes    Path
  1.57      +5 -8      jakarta-tomcat-connectors/jk/native/apache-1.3/mod_jk.c
  
  Index: mod_jk.c
  ===================================================================
  RCS file: /home/cvs/jakarta-tomcat-connectors/jk/native/apache-1.3/mod_jk.c,v
  retrieving revision 1.56
  retrieving revision 1.57
  diff -u -r1.56 -r1.57
  --- mod_jk.c  4 Dec 2004 12:13:59 -0000       1.56
  +++ mod_jk.c  7 Dec 2004 13:46:55 -0000       1.57
  @@ -671,7 +671,6 @@
       jk_server_conf_t *conf =
           (jk_server_conf_t *) ap_get_module_config(s->module_config,
                                                     &jk_module);
  -    char *old;
       const char *c, *w;
   
       if (worker != NULL && cmd->path == NULL ) {
  @@ -695,7 +694,7 @@
       /*
        * Add the new worker to the alias map.
        */
  -    jk_map_put(conf->uri_to_context, c, w, (void **)&old);
  +    jk_map_put(conf->uri_to_context, c, w, NULL);
       return NULL;
   }
   
  @@ -714,7 +713,7 @@
       jk_server_conf_t *conf =
           (jk_server_conf_t *) ap_get_module_config(s->module_config,
                                                     &jk_module);
  -    char *old , *uri;
  +    char *uri;
       const char *c, *w;
   
       if (worker != NULL && cmd->path == NULL ) {
  @@ -737,7 +736,7 @@
       /*
        * Add the new worker to the alias map.
        */
  -    jk_map_put(conf->uri_to_context, uri, w, (void **)&old);
  +    jk_map_put(conf->uri_to_context, uri, w, NULL);
       return NULL;
   }
   
  @@ -759,8 +758,7 @@
       /*
        * Add the new automount to the auto map.
        */
  -    char *old;
  -    jk_map_put(conf->automount, worker, virtualhost, (void **)&old);
  +    jk_map_put(conf->automount, worker, virtualhost, NULL);
       return NULL;
   }
   
  @@ -1764,12 +1762,11 @@
       int sz = jk_map_size(src);
       int i;
       for (i = 0; i < sz; i++) {
  -        void *old;
           char *name = jk_map_name_at(src, i);
           if (jk_map_get(src, name, NULL) == NULL) {
               if (!jk_map_put (dst, name,
                    ap_pstrdup(p, jk_map_get_string(src, name, NULL)),
  -                            &old)) {
  +                            NULL)) {
                   jk_error_exit(APLOG_MARK, APLOG_EMERG, s, p, "Memory error");
               }
           }
  
  
  
  1.107     +5 -8      jakarta-tomcat-connectors/jk/native/apache-2.0/mod_jk.c
  
  Index: mod_jk.c
  ===================================================================
  RCS file: /home/cvs/jakarta-tomcat-connectors/jk/native/apache-2.0/mod_jk.c,v
  retrieving revision 1.106
  retrieving revision 1.107
  diff -u -r1.106 -r1.107
  --- mod_jk.c  4 Dec 2004 12:11:58 -0000       1.106
  +++ mod_jk.c  7 Dec 2004 13:46:55 -0000       1.107
  @@ -715,7 +715,6 @@
       jk_server_conf_t *conf =
           (jk_server_conf_t *) ap_get_module_config(s->module_config,
                                                     &jk_module);
  -    char *old;
       const char *c, *w;
   
       if (worker != NULL && cmd->path == NULL ) {

  @@ -739,7 +738,7 @@
       /*
        * Add the new worker to the alias map.
        */
  -    jk_map_put(conf->uri_to_context, c, w, (void **)&old);
  +    jk_map_put(conf->uri_to_context, c, w, NULL);
       return NULL;
   }
   
  @@ -758,7 +757,7 @@
       jk_server_conf_t *conf =
           (jk_server_conf_t *) ap_get_module_config(s->module_config,
                                                     &jk_module);
  -    char *old , *uri;
  +    char *uri;
       const char *c, *w;
   
       if (worker != NULL && cmd->path == NULL ) {

  @@ -783,7 +782,7 @@
       /*
        * Add the new worker to the alias map.
        */
  -    jk_map_put(conf->uri_to_context, uri, w, (void **)&old);
  +    jk_map_put(conf->uri_to_context, uri, w, NULL);
       return NULL;
   }
   
  @@ -807,8 +806,7 @@
       /*
        * Add the new automount to the auto map.
        */
  -    char *old;
  -    jk_map_put(conf->automount, worker, virtualhost, (void **)&old);
  +    jk_map_put(conf->automount, worker, virtualhost, NULL);
       return NULL;
   }
   
  @@ -1978,12 +1976,11 @@
       int sz = jk_map_size(src);
       int i;
       for (i = 0; i < sz; i++) {
  -        void *old;
           char *name = jk_map_name_at(src, i);
           if (jk_map_get(src, name, NULL) == NULL) {
               if (!jk_map_put(dst, name,
                               apr_pstrdup(p, jk_map_get_string(src, name, 
NULL)),
  -                            &old)) {
  +                            NULL)) {
                   jk_error_exit(APLOG_MARK, APLOG_EMERG, s, p, "Memory error");
               }
           }
  
  
  
  1.23      +4 -4      jakarta-tomcat-connectors/jk/native/common/jk_map.c
  
  Index: jk_map.c
  ===================================================================
  RCS file: /home/cvs/jakarta-tomcat-connectors/jk/native/common/jk_map.c,v
  retrieving revision 1.22
  retrieving revision 1.23
  diff -u -r1.22 -r1.23
  --- jk_map.c  29 Nov 2004 16:04:54 -0000      1.22
  +++ jk_map.c  7 Dec 2004 13:46:56 -0000       1.23
  @@ -303,7 +303,8 @@
           }
   
           if (i < m->size) {
  -            *old = (void *)m->values[i];        /* DIRTY */
  +            if (old)
  +                *old = (void *)m->values[i];        /* DIRTY */
               m->values[i] = value;
               rc = JK_TRUE;
           }
  @@ -362,8 +363,7 @@
                       v = jk_pool_strdup(&m->p, v);
                   }
                   if (v) {
  -                    void *old = NULL;
  -                    jk_map_put(m, prp, v, &old);
  +                    jk_map_put(m, prp, v, NULL);
                   }
                   else {
                       rc = JK_FALSE;
  
  
  

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to