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

[jo...@apache.org]

Author: joes
Date: Thu Jan 27 13:40:04 2005
New Revision: 128428

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

Update my long-range goals for apreq_env_handle_t:
  replace bucket_alloc and pool functions with simple
  data strutures in apreq_parser_t (adding args to
  parser constructor)

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&rev=128428&p1=httpd/apreq/branches/multi-env-unstable/STATUS&r1=128427&p2=httpd/apreq/branches/multi-env-unstable/STATUS&r2=128428
==============================================================================
--- httpd/apreq/branches/multi-env-unstable/STATUS	(original)
+++ httpd/apreq/branches/multi-env-unstable/STATUS	Thu Jan 27 13:40:04 2005
@@ -23,8 +23,6 @@
         typedef struct apreq_env_module_t {
             const char *name;
             apr_uint32_t magic_number;
-            apr_pool_t *(*pool)(apreq_env_handle_t *);
-            apr_bucket_alloc_t *(*bucket_alloc)(apreq_env_handle_t *);
 
             apr_status_t (*jar)(apreq_env_handle_t *, const apr_table_t **);
             apr_status_t (*args)(apreq_env_handle_t *, const apr_table_t **);
@@ -48,12 +46,18 @@
             void (*log)(const char *,int,int,apr_status_t,apreq_env_handle_t *,
                  const char *,va_list);
 
+            apr_pool_t *(*pool)(apreq_env_handle_t *);
+            apr_bucket_alloc_t *(*bucket_alloc)(apreq_env_handle_t *);
+
             const char *(*temp_dir)(apreq_env_handle_t *, const char *);
             apr_off_t (*max_body)(apreq_env_handle_t *,apr_off_t);
             apr_ssize_t (*max_brigade)(apreq_env_handle_t *, apr_ssize_t);
 
         If possible the log calls inside libapreq2 should be replaced by 
         error codes that either libapreq2 apps or modules can log (or not).
+        The pool and bucket_alloc functions should become simple pointers 
+        in apreq_parser_t; env() doesn't need to provide them.
+
         The rest are controls (temp_dir, max_body, max_brigade) that
         users will need to configure, and libapreq2 parsers will need
         to look at.  However these APIs are missing status codes, for