Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:22341 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 30589 invoked by uid 1010); 11 Mar 2006 20:01:59 -0000 Delivered-To: ezmlm-scan-internals@lists.php.net Delivered-To: ezmlm-internals@lists.php.net Received: (qmail 30574 invoked from network); 11 Mar 2006 20:01:59 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 11 Mar 2006 20:01:59 -0000 X-Host-Fingerprint: 80.123.98.46 unknown Received: from ([80.123.98.46:5180] helo=localhost.localdomain) by pb1.pair.com (ecelerity 2.0 beta r(6323M)) with SMTP id C1/95-21161-5BC23144 for ; Sat, 11 Mar 2006 15:01:57 -0500 Message-ID: To: internals@lists.php.net Date: Sat, 11 Mar 2006 21:02:08 +0100 User-Agent: Mozilla Thunderbird 1.0.7 (X11/20051013) X-Accept-Language: en-us, en MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-Posted-By: 80.123.98.46 Subject: Please review: new output control implementation From: mike@php.net (Michael Wallner) Please review my attempt [0] to reimplement the output control code and make your opinion, support or objections public. A patch [1] to try, the API [2] and the actual code [3] are online for review. There's alse a README [4] explaining the what-why-how. Remaining issues: - Getting rid of ext/zlib related code in SAPI.c (and elsewhere?) as this can be handled then from within the output handler. Make the gz handlers benefit from the new implementation. - ob_implicit_flush(): In the very original implementation (and in the documentation) ob_implicit_flush disabled output buffering, but it didn't in the recent code; restore that functionality? - Look into solving the bit about xmlrpc errors in main.c. References: [0] http://dev.iworks.at/PATCHES/output/ [1] http://dev.iworks.at/PATCHES/output/ob.patch.txt.gz [2] http://dev.iworks.at/PATCHES/output/php_output.h.txt [3] http://dev.iworks.at/PATCHES/output/output.c.txt [4] http://dev.iworks.at/PATCHES/output/README.NEW-OUTPUT-API Thanks, -- Michael - http://dev.iworks.at/ext-http/http-functions.html.gz