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/01/31 01:05:05 UTC

svn commit: r149188 - httpd/apreq/branches/multi-env-unstable/STATUS

Author: joes
Date: Sun Jan 30 16:05:03 2005
New Revision: 149188

URL: http://svn.apache.org/viewcvs?view=rev&rev=149188
Log:

Progress update.

Modified:
    httpd/apreq/branches/multi-env-unstable/STATUS

Modified: httpd/apreq/branches/multi-env-unstable/STATUS
URL: http://svn.apache.org/viewcvs/httpd/apreq/branches/multi-env-unstable/STATUS?view=diff&r1=149187&r2=149188
==============================================================================
--- httpd/apreq/branches/multi-env-unstable/STATUS (original)
+++ httpd/apreq/branches/multi-env-unstable/STATUS Sun Jan 30 16:05:03 2005
@@ -17,59 +17,11 @@
        place.
 
 
-     -  joes: We should also remove apreq_jar_t and apreq_request_t.
-        I want apreq_env_handle_t to ultimately look something like this:
-
-        typedef struct apreq_env_module_t {
-            const char *name;
-            apr_uint32_t magic_number;
-
-            apr_status_t (*jar)(apreq_env_handle_t *, const apr_table_t **);
-            apr_status_t (*args)(apreq_env_handle_t *, const apr_table_t **);
-            apr_status_t (*body)(apreq_env_handle_t *, const apr_table_t **);
-
-            apreq_cookie_t *(*jar_get)(apreq_env_handle_t *, const char *);
-            apreq_param_t *(*args_get)(apreq_env_handle_t *, const char *);
-            apreq_param_t *(*body_get)(apreq_env_handle_t *, const char *);
-
-            apr_status_t (*parser_get)(apreq_env_handle_t *, const apreq_parser_t **);
-            apr_status_t (*parser_set)(apreq_env_handle_t *, apreq_parser_t *);
-            apr_status_t (*hook_add)(apreq_env_handle_t *, apreq_hook_t *);
-
-            apr_status_t (*heap_limit_set)(apreq_env_handle_t *, apr_size_t);
-            apr_status_t (*heap_limit_get)(apreq_env_handle_t *, apr_size_t *);
-
-            apr_status_t (*data_limit_set)(apreq_env_handle_t *, apr_off_t);
-            apr_status_t (*data_limit_get)(apreq_env_handle_t *, apr_off_t *);
-
-            apr_status_t (*temp_dir_set)(apreq_env_handle_t *, const char *);
-            apr_status_t (*temp_dir_get)(apreq_env_handle_t *, const char **);
-
-            const char *(*header_in)(apreq_env_handle_t *,const char *);
-            apr_status_t (*header_out)(apreq_env_handle_t *, const char *,char *);
-        } apreq_env_module_t;
-
-
-        Things to definitely remove (being either unneeded, boneheaded, 
-        or now obsolete):
-
-            apreq_jar_t (*jar)(apreq_env_handle_t *, apreq_jar_t *);
-            apreq_request_t (*request)(apreq_env_handle_t *, apreq_request_t *);
-
-            const char *(*query_string)(apreq_env_handle_t *);
-            apr_status_t (*read)(apreq_env_handle_t *,apr_read_type_e,apr_off_t);
-
-            apr_pool_t *(*pool)(apreq_env_handle_t *);
-            apr_bucket_alloc_t *(*bucket_alloc)(apreq_env_handle_t *);
-
-            void (*log)(const char *,int,int,apr_status_t,apreq_env_handle_t *,
-                 const char *,va_list);
-
-        Logs shouldn't be needed anymore. Folks can use (soon to be improved) 
-        hooks for tracing parser activity.  Better error status codes 
-        should allow users to diagnose problems through the API itself, 
-        not by reading the logfile.
-
+     - joes: We will also remove apreq_jar_t and apreq_request_t.
+       Quick status update: I've implemented this, and have src/ 
+       and env/ compiling again here locally.  I'm working on the
+       tests now, and the (many, many) apidoc fixes, so it may be a 
+       few more days before I'm ready to commit these changes.
 
 
 RELEASE SHOWSTOPPERS: