X-Git-Url: https://git.m6w6.name/?p=m6w6%2Fext-http;a=blobdiff_plain;f=http_response_object.c;h=a9f60c478aeeb5ec4bdb124abda922a9549a3b5e;hp=918647dc721db4c14269f865728491afcefc6892;hb=121eac188fc0c1cabe3ea571a72867d3fd6b1b3f;hpb=1a4ecd0664889f053891e861096f2ae1bc23e6e6

diff --git a/http_response_object.c b/http_response_object.c
index 918647d..a9f60c4 100644
--- a/http_response_object.c
+++ b/http_response_object.c
@@ -33,7 +33,7 @@
 #define HTTP_RESPONSE_ME(method, visibility)	PHP_ME(HttpResponse, method, HTTP_ARGS(HttpResponse, method), visibility|ZEND_ACC_STATIC)
 #define HTTP_RESPONSE_ALIAS(method, func)		HTTP_STATIC_ME_ALIAS(method, func, HTTP_ARGS(HttpResponse, method))
 
-HTTP_BEGIN_ARGS(setHeader, 2)
+HTTP_BEGIN_ARGS(setHeader, 1)
 	HTTP_ARG_VAL(name, 0)
 	HTTP_ARG_VAL(value, 0)
 	HTTP_ARG_VAL(replace, 0)
@@ -130,9 +130,7 @@ HTTP_END_ARGS;
 
 HTTP_EMPTY_ARGS(getRequestHeaders);
 HTTP_EMPTY_ARGS(getRequestBody);
-
-#define http_grab_response_headers _http_grab_response_headers
-static void _http_grab_response_headers(void *data, void *arg TSRMLS_DC);
+HTTP_EMPTY_ARGS(getRequestBodyStream);
 
 #define OBJ_PROP_CE http_response_object_ce
 zend_class_entry *http_response_object_ce;
@@ -186,6 +184,7 @@ zend_function_entry http_response_object_fe[] = {
 	HTTP_RESPONSE_ALIAS(status, http_send_status)
 	HTTP_RESPONSE_ALIAS(getRequestHeaders, http_get_request_headers)
 	HTTP_RESPONSE_ALIAS(getRequestBody, http_get_request_body)
+	HTTP_RESPONSE_ALIAS(getRequestBodyStream, http_get_request_body_stream)
 
 	EMPTY_FUNCTION_ENTRY
 };
@@ -207,7 +206,7 @@ PHP_MINIT_FUNCTION(http_response_object)
 	DCL_STATIC_PROP_N(PROTECTED, cacheControl);
 	DCL_STATIC_PROP_N(PROTECTED, contentType);
 	DCL_STATIC_PROP_N(PROTECTED, contentDisposition);
-	DCL_STATIC_PROP(PROTECTED, long, bufferSize, HTTP_SENDBUF_SIZE);
+	DCL_STATIC_PROP(PROTECTED, long, bufferSize, 0);
 	DCL_STATIC_PROP(PROTECTED, double, throttleDelay, 0.0);
 
 #ifndef WONKY
@@ -222,36 +221,18 @@ PHP_MINIT_FUNCTION(http_response_object)
 	return SUCCESS;
 }
 
-static void _http_grab_response_headers(void *data, void *arg TSRMLS_DC)
-{
-	phpstr_appendl(PHPSTR(arg), ((sapi_header_struct *)data)->header);
-	phpstr_appends(PHPSTR(arg), HTTP_CRLF);
-}
-
 /* ### USERLAND ### */
 
-/* {{{ proto static bool HttpResponse::setHeader(string name, mixed value[, bool replace = true])
- *
- * Send an HTTP header.
- * 
- * Expects a string parameter containing the name of the header and a mixed
- * parameter containing the value of the header, which will be converted to
- * a string.  Additionally accepts an optional boolean parameter, which
- * specifies whether an existing header should be replaced.  If the second
- * parameter is unset no header with this name will be sent. 
- * 
- * Returns TRUE on success, or FALSE on failure.
- * 
- * Throws HttpHeaderException if http.only_exceptions is TRUE.
- */
+/* {{{ proto static bool HttpResponse::setHeader(string name[, mixed value[, bool replace = true]])
+	Send an HTTP header. */
 PHP_METHOD(HttpResponse, setHeader)
 {
 	zend_bool replace = 1;
 	char *name;
 	int name_len = 0;
-	zval *value = NULL, *orig = NULL;
+	zval *value = NULL;
 
-	if (SUCCESS != zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "sz/!|b", &name, &name_len, &value, &replace)) {
+	if (SUCCESS != zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "s|z/!b", &name, &name_len, &value, &replace)) {
 		RETURN_FALSE;
 	}
 	if (SG(headers_sent)) {
@@ -262,76 +243,29 @@ PHP_METHOD(HttpResponse, setHeader)
 		http_error(HE_WARNING, HTTP_E_HEADER, "Cannot send anonymous headers");
 		RETURN_FALSE;
 	}
-
-	/* delete header if value == null */
-	if (!value || Z_TYPE_P(value) == IS_NULL) {
-		RETURN_SUCCESS(http_send_header_ex(name, name_len, "", 0, replace, NULL));
-	}
-	/* send multiple header if replace is false and value is an array */
-	if (!replace && Z_TYPE_P(value) == IS_ARRAY) {
-		zval **data;
-		HashPosition pos;
-		
-		FOREACH_VAL(pos, value, data) {
-			zval *orig = *data;
-			
-			convert_to_string_ex(data);
-			if (SUCCESS != http_send_header_ex(name, name_len, Z_STRVAL_PP(data), Z_STRLEN_PP(data), 0, NULL)) {
-				if (orig != *data) {
-					zval_ptr_dtor(data);
-				}
-				RETURN_FALSE;
-			}
-			if (orig != *data) {
-				zval_ptr_dtor(data);
-			}
-		}
-		RETURN_TRUE;
-	}
-	/* send standard header */
-	orig = value;
-	convert_to_string_ex(&value);
-	RETVAL_SUCCESS(http_send_header_ex(name, name_len, Z_STRVAL_P(value), Z_STRLEN_P(value), replace, NULL));
-	if (orig != value) {
-		zval_ptr_dtor(&value);
-	}
+	http_send_header_zval_ex(name, name_len, &value, replace);
+	RETURN_TRUE;
 }
 /* }}} */
 
 /* {{{ proto static mixed HttpResponse::getHeader([string name])
- *
- * Get header(s) about to be sent.
- * 
- * Accepts a string as optional parameter which specifies the name of the
- * header to read.  If the parameter is empty or omitted, an associative array
- * with all headers will be returned.
- * 
- * NOTE: In Apache2 this only works for PHP-5.1.3 and greater.
- * 
- * Returns either a string containing the value of the header matching name,
- * FALSE on failure, or an associative array with all headers. 
- */
+	Get header(s) about to be sent. */
 PHP_METHOD(HttpResponse, getHeader)
 {
 	char *name = NULL;
 	int name_len = 0;
-	phpstr headers;
 	
 	if (SUCCESS != zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "|s", &name, &name_len)) {
 		RETURN_FALSE;
 	}
 	
-	phpstr_init(&headers);
-	zend_llist_apply_with_argument(&SG(sapi_headers).headers, http_grab_response_headers, &headers TSRMLS_CC);
-	phpstr_fix(&headers);
-	
 	if (name && name_len) {
 		zval **header;
 		HashTable headers_ht;
 		
 		zend_hash_init(&headers_ht, sizeof(zval *), NULL, ZVAL_PTR_DTOR, 0);
-		if (	(SUCCESS == http_parse_headers_ex(PHPSTR_VAL(&headers), &headers_ht, 1)) &&
-				(SUCCESS == zend_hash_find(&headers_ht, name, name_len + 1, (void **) &header))) {
+		if (	(SUCCESS == http_get_response_headers(&headers_ht)) &&
+				(SUCCESS == zend_hash_find(&headers_ht, name, name_len + 1, (void *) &header))) {
 			RETVAL_ZVAL(*header, 1, 0);
 		} else {
 			RETVAL_NULL();
@@ -339,27 +273,13 @@ PHP_METHOD(HttpResponse, getHeader)
 		zend_hash_destroy(&headers_ht);
 	} else {
 		array_init(return_value);
-		http_parse_headers_ex(PHPSTR_VAL(&headers), Z_ARRVAL_P(return_value), 1);
+		http_get_response_headers(Z_ARRVAL_P(return_value));
 	}
-	
-	phpstr_dtor(&headers);
 }
 /* }}} */
 
 /* {{{ proto static bool HttpResponse::setCache(bool cache)
- *
- * Whether it should be attempted to cache the entity.
- * This will result in necessary caching headers and checks of clients
- * "If-Modified-Since" and "If-None-Match" headers.  If one of those headers
- * matches a "304 Not Modified" status code will be issued.
- *
- * NOTE: If you're using sessions, be sure that you set session.cache_limiter
- * to something more appropriate than "no-cache"!
- * 
- * Expects a boolean as parameter specifying whether caching should be attempted.
- * 
- * Returns TRUE on success, or FALSE on failure.
- */
+	Whether it should be attempted to cache the entity. */
 PHP_METHOD(HttpResponse, setCache)
 {
 	zend_bool do_cache = 0;
@@ -373,16 +293,12 @@ PHP_METHOD(HttpResponse, setCache)
 /* }}} */
 
 /* {{{ proto static bool HttpResponse::getCache()
- *
- * Get current caching setting.
- * 
- * Returns TRUE if caching should be attempted, else FALSE.
- */
+	Get current caching setting. */
 PHP_METHOD(HttpResponse, getCache)
 {
 	NO_ARGS;
 
-	IF_RETVAL_USED {
+	if (return_value_used) {
 		zval *cache_p, *cache = convert_to_type_ex(IS_BOOL, GET_STATIC_PROP(cache), &cache_p);
 		
 		RETVAL_ZVAL(cache, 1, 0);
@@ -395,13 +311,7 @@ PHP_METHOD(HttpResponse, getCache)
 /* }}}*/
 
 /* {{{ proto static bool HttpResponse::setGzip(bool gzip)
- *
- * Enable on-thy-fly gzipping of the sent entity.
- * 
- * Expects a boolean as parameter indicating if GZip compression should be enabled.
- * 
- * Returns TRUE on success, or FALSE on failure.
- */
+	Enable on-thy-fly gzipping of the sent entity. */
 PHP_METHOD(HttpResponse, setGzip)
 {
 	zend_bool do_gzip = 0;
@@ -415,16 +325,12 @@ PHP_METHOD(HttpResponse, setGzip)
 /* }}} */
 
 /* {{{ proto static bool HttpResponse::getGzip()
- *
- * Get current gzipping setting.
- * 
- * Returns TRUE if GZip compression is enabled, else FALSE.
- */
+	Get current gzipping setting. */
 PHP_METHOD(HttpResponse, getGzip)
 {
 	NO_ARGS;
 
-	IF_RETVAL_USED {
+	if (return_value_used) {
 		zval *gzip_p, *gzip = convert_to_type_ex(IS_BOOL, GET_STATIC_PROP(gzip), &gzip_p);
 		
 		RETVAL_ZVAL(gzip, 1, 0);
@@ -437,20 +343,7 @@ PHP_METHOD(HttpResponse, getGzip)
 /* }}} */
 
 /* {{{ proto static bool HttpResponse::setCacheControl(string control[, int max_age = 0[, bool must_revalidate = true]])
- *
- * Set a custom cache-control header, usually being "private" or "public";
- * The max_age parameter controls how long the cache entry is valid on the client side.
- * 
- * Expects a string parameter containing the primary cache control setting.
- * Additionally accepts an int parameter specifying the max-age setting.
- * Accepts an optional third bool parameter indicating whether the cache
- * must be revalidated every request.
- * 
- * Returns TRUE on success, or FALSE if control does not match one of
- * "public" , "private" or "no-cache".
- * 
- * Throws HttpInvalidParamException if http.only_exceptions is TRUE.
- */
+	Set a custom cache-control header, usually being "private" or "public"; The max_age parameter controls how long the cache entry is valid on the client side. */
 PHP_METHOD(HttpResponse, setCacheControl)
 {
 	char *ccontrol, *cctl;
@@ -474,16 +367,12 @@ PHP_METHOD(HttpResponse, setCacheControl)
 /* }}} */
 
 /* {{{ proto static string HttpResponse::getCacheControl()
- *
- * Get current Cache-Control header setting.
- * 
- * Returns the current cache control setting as a string like sent in a header.
- */
+	Get current Cache-Control header setting. */
 PHP_METHOD(HttpResponse, getCacheControl)
 {
 	NO_ARGS;
 
-	IF_RETVAL_USED {
+	if (return_value_used) {
 		zval *ccontrol_p, *ccontrol = convert_to_type_ex(IS_STRING, GET_STATIC_PROP(cacheControl), &ccontrol_p);
 		
 		RETVAL_ZVAL(ccontrol, 1, 0);
@@ -496,16 +385,7 @@ PHP_METHOD(HttpResponse, getCacheControl)
 /* }}} */
 
 /* {{{ proto static bool HttpResponse::setContentType(string content_type)
- *
- * Set the content-type of the sent entity.
- * 
- * Expects a string as parameter specifying the content type of the sent entity.
- * 
- * Returns TRUE on success, or FALSE if the content type does not seem to
- * contain a primary and secondary content type part.
- * 
- * Throws HttpInvalidParamException if http.only_exceptions is TRUE.
- */
+	Set the content-type of the sent entity. */
 PHP_METHOD(HttpResponse, setContentType)
 {
 	char *ctype;
@@ -521,16 +401,12 @@ PHP_METHOD(HttpResponse, setContentType)
 /* }}} */
 
 /* {{{ proto static string HttpResponse::getContentType()
- *
- * Get current Content-Type header setting.
- * 
- * Returns the currently set content type as string.
- */
+	Get current Content-Type header setting. */
 PHP_METHOD(HttpResponse, getContentType)
 {
 	NO_ARGS;
 
-	IF_RETVAL_USED {
+	if (return_value_used) {
 		zval *ctype_p, *ctype = convert_to_type_ex(IS_STRING, GET_STATIC_PROP(contentType), &ctype_p);
 		
 		RETVAL_ZVAL(ctype, 1, 0);
@@ -543,19 +419,7 @@ PHP_METHOD(HttpResponse, getContentType)
 /* }}} */
 
 /* {{{ proto static string HttpResponse::guessContentType(string magic_file[, int magic_mode = MAGIC_MIME])
- *
- * Attempts to guess the content type of supplied payload through libmagic.
- * If the attempt is successful, the guessed content type will automatically
- * be set as response content type.  
- * 
- * Expects a string parameter specifying the magic.mime database to use.
- * Additionally accepts an optional int parameter, being flags for libmagic.
- * 
- * Returns the guessed content type on success, or FALSE on failure.
- * 
- * Throws HttpRuntimeException, HttpInvalidParamException 
- * if http.only_exceptions is TRUE.
- */
+	Attempts to guess the content type of supplied payload through libmagic. */
 PHP_METHOD(HttpResponse, guessContentType)
 {
 #ifdef HTTP_HAVE_MAGIC
@@ -571,8 +435,8 @@ PHP_METHOD(HttpResponse, guessContentType)
 			{
 				zval *data = GET_STATIC_PROP(data);
 				ct = http_guess_content_type(magic_file, magic_mode, Z_STRVAL_P(data), Z_STRLEN_P(data), SEND_DATA);
+				break;
 			}
-			break;
 			
 			case SEND_RSRC:
 			{
@@ -581,12 +445,12 @@ PHP_METHOD(HttpResponse, guessContentType)
 				z->type = IS_RESOURCE;
 				php_stream_from_zval(s, &z);
 				ct = http_guess_content_type(magic_file, magic_mode, s, 0, SEND_RSRC);
+				break;
 			}
-			break;
 			
 			default:
 				ct = http_guess_content_type(magic_file, magic_mode, Z_STRVAL_P(GET_STATIC_PROP(file)), 0, -1);
-			break;
+				break;
 		}
 		if (ct) {
 			UPD_STATIC_PROP(string, contentType, ct);
@@ -602,18 +466,7 @@ PHP_METHOD(HttpResponse, guessContentType)
 /* }}} */
 
 /* {{{ proto static bool HttpResponse::setContentDisposition(string filename[, bool inline = false])
- *
- * Set the Content-Disposition.  The Content-Disposition header is very useful
- * if the data actually sent came from a file or something similar, that should
- * be "saved" by the client/user (i.e. by browsers "Save as..." popup window).
- *
- * Expects a string parameter specifying the file name the "Save as..." dialog
- * should display.  Optionally accepts a bool parameter, which, if set to true
- * and the user agent knows how to handle the content type, will probably not
- * cause the popup window to be shown.
- * 
- * Returns TRUE on success or FALSE on failure.
- */
+	Set the Content-Disposition. */
 PHP_METHOD(HttpResponse, setContentDisposition)
 {
 	char *file, *cd;
@@ -632,16 +485,12 @@ PHP_METHOD(HttpResponse, setContentDisposition)
 /* }}} */
 
 /* {{{ proto static string HttpResponse::getContentDisposition()
- *
- * Get current Content-Disposition setting.
- * 
- * Returns the current content disposition as string like sent in a header.
- */
+	Get current Content-Disposition setting. */
 PHP_METHOD(HttpResponse, getContentDisposition)
 {
 	NO_ARGS;
 
-	IF_RETVAL_USED {
+	if (return_value_used) {
 		zval *cd_p, *cd = convert_to_type_ex(IS_STRING, GET_STATIC_PROP(contentDisposition), &cd_p);
 		
 		RETVAL_ZVAL(cd, 1, 0);
@@ -654,13 +503,7 @@ PHP_METHOD(HttpResponse, getContentDisposition)
 /* }}} */
 
 /* {{{ proto static bool HttpResponse::setETag(string etag)
- *
- * Set a custom ETag.  Use this only if you know what you're doing.
- * 
- * Expects an unquoted string as parameter containing the ETag.
- * 
- * Returns TRUE on success, or FALSE on failure.
- */
+	Set a custom ETag.  Use this only if you know what you're doing. */
 PHP_METHOD(HttpResponse, setETag)
 {
 	char *etag;
@@ -675,16 +518,12 @@ PHP_METHOD(HttpResponse, setETag)
 /* }}} */
 
 /* {{{ proto static string HttpResponse::getETag()
- *
- * Get calculated or previously set custom ETag.
- * 
- * Returns the calculated or previously set ETag as unquoted string.
- */
+	Get calculated or previously set custom ETag. */
 PHP_METHOD(HttpResponse, getETag)
 {
 	NO_ARGS;
 
-	IF_RETVAL_USED {
+	if (return_value_used) {
 		zval *etag_p, *etag = convert_to_type_ex(IS_STRING, GET_STATIC_PROP(eTag), &etag_p);
 		
 		RETVAL_ZVAL(etag, 1, 0);
@@ -697,14 +536,7 @@ PHP_METHOD(HttpResponse, getETag)
 /* }}} */
 
 /* {{{ proto static bool HttpResponse::setLastModified(int timestamp)
- *
- * Set a custom Last-Modified date.
- * 
- * Expects an unix timestamp as parameter representing the last modification
- * time of the sent entity.
- * 
- * Returns TRUE on success, or FALSE on failure.
- */
+	Set a custom Last-Modified date. */
 PHP_METHOD(HttpResponse, setLastModified)
 {
 	long lm;
@@ -718,16 +550,12 @@ PHP_METHOD(HttpResponse, setLastModified)
 /* }}} */
 
 /* {{{ proto static int HttpResponse::getLastModified()
- *
- * Get calculated or previously set custom Last-Modified date.
- * 
- * Returns the calculated or previously set unix timestamp.
- */
+	Get calculated or previously set custom Last-Modified date. */
 PHP_METHOD(HttpResponse, getLastModified)
 {
 	NO_ARGS;
 	
-	IF_RETVAL_USED {
+	if (return_value_used) {
 		zval *lm_p, *lm = convert_to_type_ex(IS_LONG, GET_STATIC_PROP(lastModified), &lm_p);
 		
 		RETVAL_ZVAL(lm, 1, 0);
@@ -740,19 +568,7 @@ PHP_METHOD(HttpResponse, getLastModified)
 /* }}} */
 
 /* {{{ proto static bool HttpResponse::setThrottleDelay(double seconds)
- *
- * Sets the throttle delay for use with HttpResponse::setBufferSize().
- * 
- * Provides a basic throttling mechanism, which will yield the current process
- * resp. thread until the entity has been completely sent, though.
- * 
- * Note: This doesn't really work with the FastCGI SAPI.
- *
- * Expects a double parameter specifying the seconds too sleep() after
- * each chunk sent.
- * 
- * Returns TRUE on success, or FALSE on failure.
- */
+	Sets the throttle delay for use with HttpResponse::setBufferSize(). */
 PHP_METHOD(HttpResponse, setThrottleDelay)
 {
 	double seconds;
@@ -765,16 +581,12 @@ PHP_METHOD(HttpResponse, setThrottleDelay)
 /* }}} */
 
 /* {{{ proto static double HttpResponse::getThrottleDelay()
- *
- * Get the current throttle delay.
- * 
- * Returns a double representing the throttle delay in seconds.
- */
+	Get the current throttle delay. */
 PHP_METHOD(HttpResponse, getThrottleDelay)
 {
 	NO_ARGS;
 
-	IF_RETVAL_USED {
+	if (return_value_used) {
 		zval *delay_p, *delay = convert_to_type_ex(IS_DOUBLE, GET_STATIC_PROP(throttleDelay), &delay_p);
 		
 		RETVAL_ZVAL(delay, 1, 0);
@@ -787,18 +599,7 @@ PHP_METHOD(HttpResponse, getThrottleDelay)
 /* }}} */
 
 /* {{{ proto static bool HttpResponse::setBufferSize(int bytes)
- *
- * Sets the send buffer size for use with HttpResponse::setThrottleDelay().
- * 
- * Provides a basic throttling mechanism, which will yield the current process
- * resp. thread until the entity has been completely sent, though.
- * 
- * Note: This doesn't really work with the FastCGI SAPI.
- *
- * Expects an int parameter representing the chunk size in bytes.
- * 
- * Returns TRUE on success, or FALSE on failure.
- */
+	Sets the send buffer size for use with HttpResponse::setThrottleDelay(). */
 PHP_METHOD(HttpResponse, setBufferSize)
 {
 	long bytes;
@@ -811,16 +612,12 @@ PHP_METHOD(HttpResponse, setBufferSize)
 /* }}} */
 
 /* {{{ proto static int HttpResponse::getBufferSize()
- *
- * Get current buffer size.
- * 
- * Returns an int representing the current buffer size in bytes.
- */
+	Get current buffer size. */
 PHP_METHOD(HttpResponse, getBufferSize)
 {
 	NO_ARGS;
 
-	IF_RETVAL_USED {
+	if (return_value_used) {
 		zval *size_p, *size = convert_to_type_ex(IS_LONG, GET_STATIC_PROP(bufferSize), &size_p);
 		
 		RETVAL_ZVAL(size, 1, 0);
@@ -833,20 +630,13 @@ PHP_METHOD(HttpResponse, getBufferSize)
 /* }}} */
 
 /* {{{ proto static bool HttpResponse::setData(mixed data)
- *
- * Set the data to be sent.
- * 
- * Expects one parameter, which will be converted to a string and contains 
- * the data to send.
- * 
- * Returns TRUE on success, or FALSE on failure.
- */
+	Set the data to be sent. */
 PHP_METHOD(HttpResponse, setData)
 {
 	char *etag;
 	zval *the_data;
 
-	if (SUCCESS != zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "z", &the_data)) {
+	if (SUCCESS != zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "z/", &the_data)) {
 		RETURN_FALSE;
 	}
 	if (Z_TYPE_P(the_data) != IS_STRING) {
@@ -869,16 +659,12 @@ PHP_METHOD(HttpResponse, setData)
 /* }}} */
 
 /* {{{ proto static string HttpResponse::getData()
- *
- * Get the previously set data to be sent.
- * 
- * Returns a string containing the previously set data to send.
- */
+	Get the previously set data to be sent. */
 PHP_METHOD(HttpResponse, getData)
 {
 	NO_ARGS;
 
-	IF_RETVAL_USED {
+	if (return_value_used) {
 		zval *the_data = GET_STATIC_PROP(data);
 		
 		RETURN_ZVAL(the_data, 1, 0);
@@ -887,14 +673,7 @@ PHP_METHOD(HttpResponse, getData)
 /* }}} */
 
 /* {{{ proto static bool HttpResponse::setStream(resource stream)
- *
- * Set the resource to be sent.
- * 
- * Expects a resource parameter referencing an already opened stream from
- * which the data to send will be read.
- * 
- * Returns TRUE on success, or FALSE on failure.
- */
+	Set the resource to be sent. */
 PHP_METHOD(HttpResponse, setStream)
 {
 	char *etag;
@@ -928,16 +707,12 @@ PHP_METHOD(HttpResponse, setStream)
 /* }}} */
 
 /* {{{ proto static resource HttpResponse::getStream()
- *
- * Get the previously set resource to be sent.
- * 
- * Returns the previously set resource.
- */
+	Get the previously set resource to be sent. */
 PHP_METHOD(HttpResponse, getStream)
 {
 	NO_ARGS;
 
-	IF_RETVAL_USED {
+	if (return_value_used) {
 		zval *stream_p;
 		
 		RETVAL_RESOURCE(Z_LVAL_P(convert_to_type_ex(IS_LONG, GET_STATIC_PROP(stream), &stream_p)));
@@ -950,13 +725,7 @@ PHP_METHOD(HttpResponse, getStream)
 /* }}} */
 
 /* {{{ proto static bool HttpResponse::setFile(string file)
- *
- * Set the file to be sent.
- * 
- * Expects a string as parameter, specifying the path to the file to send.
- * 
- * Returns TRUE on success, or FALSE on failure.
- */
+	Set the file to be sent. */
 PHP_METHOD(HttpResponse, setFile)
 {
 	char *the_file, *etag;
@@ -987,16 +756,12 @@ PHP_METHOD(HttpResponse, setFile)
 /* }}} */
 
 /* {{{ proto static string HttpResponse::getFile()
- *
- * Get the previously set file to be sent.
- * 
- * Returns the previously set path to the file to send as string.
- */
+	Get the previously set file to be sent. */
 PHP_METHOD(HttpResponse, getFile)
 {
 	NO_ARGS;
 
-	IF_RETVAL_USED {
+	if (return_value_used) {
 		zval *the_file_p, *the_file = convert_to_type_ex(IS_STRING, GET_STATIC_PROP(file), &the_file_p);
 		
 		RETVAL_ZVAL(the_file, 1, 0);
@@ -1009,29 +774,7 @@ PHP_METHOD(HttpResponse, getFile)
 /* }}} */
 
 /* {{{ proto static bool HttpResponse::send([bool clean_ob = true])
- *
- * Finally send the entity.
- * 
- * Accepts an optional boolean parameter, specifying whether the output
- * buffers should be discarded prior sending.  A successful caching attempt
- * will cause a script termination, and write a log entry if the INI setting
- * http.cache_log is set.
- * 
- * Returns TRUE on success, or FALSE on failure.
- * 
- * Throws HttpHeaderException, HttpResponseException if http.only_exceptions is TRUE.
- *
- * Example:
- * <pre>
- * <?php
- * HttpResponse::setCache(true);
- * HttpResponse::setContentType('application/pdf');
- * HttpResponse::setContentDisposition("$user.pdf", false);
- * HttpResponse::setFile('sheet.pdf');
- * HttpResponse::send();
- * ?>
- * </pre>
- */
+	Finally send the entity. */
 PHP_METHOD(HttpResponse, send)
 {
 	zval *sent;
@@ -1092,7 +835,11 @@ PHP_METHOD(HttpResponse, send)
 		cctl = convert_to_type_ex(IS_STRING, GET_STATIC_PROP(cacheControl), &cctl_p);
 		
 		if (Z_LVAL_P(lmod) || Z_STRLEN_P(etag)) {
-			http_send_cache_control(Z_STRVAL_P(cctl), Z_STRLEN_P(cctl));
+			if (Z_STRLEN_P(cctl)) {
+				http_send_cache_control(Z_STRVAL_P(cctl), Z_STRLEN_P(cctl));
+			} else {
+				http_send_cache_control(HTTP_DEFAULT_CACHECONTROL, lenof(HTTP_DEFAULT_CACHECONTROL));
+			}
 			if (Z_STRLEN_P(etag)) {
 				http_send_etag(Z_STRVAL_P(etag), Z_STRLEN_P(etag));
 			}
@@ -1148,14 +895,10 @@ PHP_METHOD(HttpResponse, send)
 	}
 
 	/* gzip */
-	HTTP_G->send.deflate.encoding = zval_is_true(GET_STATIC_PROP(gzip));
+	HTTP_G->send.deflate.response = zval_is_true(GET_STATIC_PROP(gzip));
 	
-	/* start ob */
-	php_start_ob_buffer(NULL, HTTP_G->send.buffer_size, 0 TSRMLS_CC);
-
 	/* send */
-	switch (Z_LVAL_P(GET_STATIC_PROP(mode)))
-	{
+	switch (Z_LVAL_P(GET_STATIC_PROP(mode))) {
 		case SEND_DATA:
 		{
 			zval *zdata_p, *zdata = convert_to_type_ex(IS_STRING, GET_STATIC_PROP(data), &zdata_p);
@@ -1187,17 +930,7 @@ PHP_METHOD(HttpResponse, send)
 /* }}} */
 
 /* {{{ proto static void HttpResponse::capture()
- *
- * Capture script output.
- *
- * Example:
- * <pre>
- * <?php
- * HttpResponse::setCache(true);
- * HttpResponse::capture();
- * // script follows
- * ?>
- * </pre>
+	Capture script output.
  */
 PHP_METHOD(HttpResponse, capture)
 {
@@ -1208,7 +941,7 @@ PHP_METHOD(HttpResponse, capture)
 	UPD_STATIC_PROP(long, catch, 1);
 
 	php_end_ob_buffers(0 TSRMLS_CC);
-	php_start_ob_buffer(NULL, 40960, 0 TSRMLS_CC);
+	php_start_ob_buffer(NULL, 0, 0 TSRMLS_CC);
 
 	/* register shutdown function */
 	{