HI Jani,
I don't think it is a good think to merge this change at this stage in
5.3. 5.3 is now very stable and should have only bug fixes and minor
improvements. I would have like to have the new output API in 5.3, but
we failed to do it before 5.3.0 release began. But it is definitively
too late to do it in 5.3.
What I would suggest is to create a 5.3 branch to merge the API so we
can test it using 5.3 as a base, something like what we do for FPM.
Doing so will considerably increase the merge process for the next
major release (be 5.4 or whatever is the next major release number).
Thanks for your understanding,
Cheers,
--
Pierre
On Thu, Mar 11, 2010 at 11:24 AM, Jani Taskinen <jani@php.net> wrote:
> jani Thu, 11 Mar 2010 10:24:29 +0000>> Revision: http://svn.php.net/viewvc?view=revision&revision=296062>> Log:> MFH: Improved / fixed output buffering (Michael Wallner)>> Changed paths:> A php/php-src/branches/PHP_5_3/README.NEW-OUTPUT-API> U php/php-src/branches/PHP_5_3/Zend/zend_highlight.c> U php/php-src/branches/PHP_5_3/Zend/zend_indent.c> U php/php-src/branches/PHP_5_3/ext/iconv/iconv.c> U php/php-src/branches/PHP_5_3/ext/session/session.c> U php/php-src/branches/PHP_5_3/ext/soap/soap.c> U php/php-src/branches/PHP_5_3/ext/standard/basic_functions.c> U php/php-src/branches/PHP_5_3/ext/standard/exec.c> U php/php-src/branches/PHP_5_3/ext/standard/head.c> U php/php-src/branches/PHP_5_3/ext/standard/info.c> U
> php/php-src/branches/PHP_5_3/ext/standard/tests/general_functions/ob_get_flush_error.phpt> U php/php-src/branches/PHP_5_3/ext/standard/tests/general_functions/phpinfo.phpt> U php/php-src/branches/PHP_5_3/ext/standard/url_scanner_ex.c> U php/php-src/branches/PHP_5_3/ext/standard/url_scanner_ex.re> U php/php-src/branches/PHP_5_3/ext/standard/var.c> U php/php-src/branches/PHP_5_3/ext/tidy/php_tidy.h> U php/php-src/branches/PHP_5_3/ext/tidy/tidy.c> U php/php-src/branches/PHP_5_3/ext/zlib/CREDITS> U php/php-src/branches/PHP_5_3/ext/zlib/config0.m4> U php/php-src/branches/PHP_5_3/ext/zlib/php_zlib.h> U php/php-src/branches/PHP_5_3/ext/zlib/tests/005.phpt> U php/php-src/branches/PHP_5_3/ext/zlib/tests/006.phpt> U php/php-src/branches/PHP_5_3/ext/zlib/tests/007.phpt> U php/php-src/branches/PHP_5_3/ext/zlib/tests/gzcompress_basic1.phpt> U php/php-src/branches/PHP_5_3/ext/zlib/tests/gzcompress_error1.phpt> U php/php-src/branches/PHP_5_3/ext/zlib/tests/gzcompress_variation1.phpt> U php/php-src/branches/PHP_5_3/ext/zlib/tests/gzdeflate_basic1..phpt> U php/php-src/branches/PHP_5_3/ext/zlib/tests/gzdeflate_error1..phpt> U php/php-src/branches/PHP_5_3/ext/zlib/tests/gzdeflate_variation1.phpt> U php/php-src/branches/PHP_5_3/ext/zlib/tests/gzencode_basic1.phpt> U php/php-src/branches/PHP_5_3/ext/zlib/tests/gzencode_error1.phpt> U php/php-src/branches/PHP_5_3/ext/zlib/tests/gzencode_variation1.phpt> A php/php-src/branches/PHP_5_3/ext/zlib/tests/gzencode_variation2-win32.phpt> U php/php-src/branches/PHP_5_3/ext/zlib/tests/gzencode_variation2.phpt> U php/php-src/branches/PHP_5_3/ext/zlib/tests/gzfile_variation10.phpt> U php/php-src/branches/PHP_5_3/ext/zlib/tests/gzfilegzreadfile..phpt> U php/php-src/branches/PHP_5_3/ext/zlib/tests/gzinflate-bug42663.phpt> U php/php-src/branches/PHP_5_3/ext/zlib/tests/gzinflate_length..phpt> U php/php-src/branches/PHP_5_3/ext/zlib/tests/gzopen_variation5.phpt> U php/php-src/branches/PHP_5_3/ext/zlib/tests/gzopen_variation8.phpt> U php/php-src/branches/PHP_5_3/ext/zlib/tests/gzread_variation1.phpt> U php/php-src/branches/PHP_5_3/ext/zlib/tests/gzreadgzwrite.phpt> U php/php-src/branches/PHP_5_3/ext/zlib/tests/gzreadgzwriteplain.phpt> U php/php-src/branches/PHP_5_3/ext/zlib/tests/gzuncompress_basic1.phpt> U php/php-src/branches/PHP_5_3/ext/zlib/tests/gzuncompress_error1.phpt> U php/php-src/branches/PHP_5_3/ext/zlib/tests/gzwrite_basic.phpt> U php/php-src/branches/PHP_5_3/ext/zlib/tests/gzwrite_variation1.phpt> A php/php-src/branches/PHP_5_3/ext/zlib/tests/ob_001.phpt> A php/php-src/branches/PHP_5_3/ext/zlib/tests/ob_002.phpt> A php/php-src/branches/PHP_5_3/ext/zlib/tests/ob_003.phpt> A php/php-src/branches/PHP_5_3/ext/zlib/tests/ob_004.phpt> A php/php-src/branches/PHP_5_3/ext/zlib/tests/ob_005.phpt> U php/php-src/branches/PHP_5_3/ext/zlib/tests/readgzfile_variation10.phpt> U php/php-src/branches/PHP_5_3/ext/zlib/tests/readgzfile_variation14.phpt> U php/php-src/branches/PHP_5_3/ext/zlib/tests/readgzfile_variation4.phpt> U php/php-src/branches/PHP_5_3/ext/zlib/tests/readgzfile_variation5.phpt> U php/php-src/branches/PHP_5_3/ext/zlib/tests/readgzfile_variation6.phpt> U php/php-src/branches/PHP_5_3/ext/zlib/tests/readgzfile_variation7.phpt> U php/php-src/branches/PHP_5_3/ext/zlib/tests/zlib_filter_inflate2.phpt> U php/php-src/branches/PHP_5_3/ext/zlib/tests/zlib_scheme_copy_variation2.phpt> U php/php-src/branches/PHP_5_3/ext/zlib/zlib.c> U php/php-src/branches/PHP_5_3/ext/zlib/zlib_filter.c> U php/php-src/branches/PHP_5_3/ext/zlib/zlib_fopen_wrapper.c> U php/php-src/branches/PHP_5_3/main/SAPI.c> U php/php-src/branches/PHP_5_3/main/main.c> U php/php-src/branches/PHP_5_3/main/output.c> U php/php-src/branches/PHP_5_3/main/php.h> U php/php-src/branches/PHP_5_3/main/php_logos.c> U php/php-src/branches/PHP_5_3/main/php_output.h> U php/php-src/branches/PHP_5_3/sapi/apache/libpre.c> U php/php-src/branches/PHP_5_3/sapi/apache/mod_php5.c> U php/php-src/branches/PHP_5_3/sapi/apache/php_apache.c> U php/php-src/branches/PHP_5_3/sapi/apache/sapi_apache.c> U php/php-src/branches/PHP_5_3/sapi/apache2handler/php_functions.c> U php/php-src/branches/PHP_5_3/sapi/apache_hooks/mod_php5.c> U php/php-src/branches/PHP_5_3/sapi/apache_hooks/php_apache.c> U php/php-src/branches/PHP_5_3/sapi/apache_hooks/sapi_apache.c> U php/php-src/branches/PHP_5_3/sapi/cgi/cgi_main.c> U php/php-src/branches/PHP_5_3/sapi/cli/php_cli.c> U php/php-src/branches/PHP_5_3/sapi/cli/tests/007.phpt> U php/php-src/branches/PHP_5_3/sapi/cli/tests/008.phpt> U php/php-src/branches/PHP_5_3/sapi/cli/tests/009.phpt> U php/php-src/branches/PHP_5_3/sapi/cli/tests/011.phpt> U php/php-src/branches/PHP_5_3/sapi/cli/tests/012.phpt> U php/php-src/branches/PHP_5_3/sapi/cli/tests/014.phpt> U php/php-src/branches/PHP_5_3/sapi/milter/php_milter.c> U php/php-src/branches/PHP_5_3/sapi/nsapi/nsapi.c> U php/php-src/branches/PHP_5_3/tests/output/bug46897.phpt> U php/php-src/branches/PHP_5_3/tests/output/ob_011.phpt> U php/php-src/branches/PHP_5_3/tests/output/ob_013.phpt> U php/php-src/branches/PHP_5_3/tests/output/ob_017.phpt> U php/php-src/branches/PHP_5_3/tests/output/ob_clean_basic_001..phpt> U php/php-src/branches/PHP_5_3/tests/output/ob_end_clean_basic_001.phpt> U php/php-src/branches/PHP_5_3/tests/output/ob_end_flush_basic_001.phpt> U php/php-src/branches/PHP_5_3/tests/output/ob_flush_basic_001..phpt> U php/php-src/branches/PHP_5_3/tests/output/ob_get_clean_basic_001.phpt> U php/php-src/branches/PHP_5_3/tests/output/ob_get_level_basic_001.phpt> U php/php-src/branches/PHP_5_3/tests/output/ob_get_status.phpt> U php/php-src/branches/PHP_5_3/tests/output/ob_start_basic_004..phpt> U php/php-src/branches/PHP_5_3/tests/output/ob_start_basic_006..phpt> U php/php-src/branches/PHP_5_3/tests/output/ob_start_basic_unerasable_002.phpt> U php/php-src/branches/PHP_5_3/tests/output/ob_start_basic_unerasable_003.phpt> U php/php-src/branches/PHP_5_3/tests/output/ob_start_basic_unerasable_004.phpt> U php/php-src/branches/PHP_5_3/tests/output/ob_start_basic_unerasable_005.phpt> U php/php-src/branches/PHP_5_3/tests/output/ob_start_error_001..phpt> U php/php-src/branches/PHP_5_3/tests/output/ob_start_error_002..phpt> U php/php-src/branches/PHP_5_3/tests/output/ob_start_error_003..phpt> U php/php-src/branches/PHP_5_3/tests/output/ob_start_error_004..phpt>> <diffs exceeded maximum size>>> --> PHP CVS Mailing List (http://www.php.net/)> To unsubscribe, visit: http://www.php.net/unsub.php>--
Pierre
@pierrejoye | http://blog.thepimp.net | http://www.libgd.org