You are viewing a plain text version of this content. The canonical link for it is here.
Posted to general@commons.apache.org by gs...@apache.org on 2003/11/05 04:11:10 UTC

svn commit: rev 90 - commons/serf/branches/gen2

Author: gstein
Date: Tue Nov  4 19:11:10 2003
New Revision: 90

Modified:
   commons/serf/branches/gen2/context.c
   commons/serf/branches/gen2/serf.h
Log:
Problems noted by Garrett Rooney <ro...@electricjellyfish.net>.

* serf.h:
   (serf_request_get_alloc): return a pointer to a serf_bucket_alloc_t, as
     you can't return the structure since it's only forward declared, and
     we only have a pointer anyway.

* context.c:
   (serf_request_deliver): remove return, this function returns void.
   (serf_request_get_alloc): adjust for change to prototype.


Modified: commons/serf/branches/gen2/context.c
==============================================================================
--- commons/serf/branches/gen2/context.c	(original)
+++ commons/serf/branches/gen2/context.c	Tue Nov  4 19:11:10 2003
@@ -502,7 +502,6 @@
     serf_response_handler_t handler,
     void *handler_baton)
 {
-    return APR_ENOTIMPL;
 }
 
 
@@ -516,7 +515,7 @@
     return request->respool;
 }
 
-SERF_DECLARE(serf_bucket_alloc_t) serf_request_get_alloc(
+SERF_DECLARE(serf_bucket_alloc_t *) serf_request_get_alloc(
     serf_request_t *request)
 {
     return request->allocator;

Modified: commons/serf/branches/gen2/serf.h
==============================================================================
--- commons/serf/branches/gen2/serf.h	(original)
+++ commons/serf/branches/gen2/serf.h	Tue Nov  4 19:11:10 2003
@@ -297,7 +297,7 @@
 /**
  * Return the bucket allocator associated with @a request.
  */
-SERF_DECLARE(serf_bucket_alloc_t) serf_request_get_alloc(
+SERF_DECLARE(serf_bucket_alloc_t *) serf_request_get_alloc(
     serf_request_t *request);