Re: cvs: php4 /ext/standard basic_functions.c var.c /main

From: Date: Thu, 03 Oct 2002 14:13:57 +0000
Subject: Re: cvs: php4 /ext/standard basic_functions.c var.c /main
References: 1 2  Groups: php.cvs 
Request: Send a blank email to php-cvs+get-14485@lists.php.net to get a copy of this message
At 17:00 03/10/2002, Yasuo Ohgaki wrote:
Zeev Suraski wrote:
Yasuo, Can you explain how come the output buffering code pays any attention to implicit_flush, when I repeatedly said that output buffering and implicit_flush are COMPLETELY unrelated?
This patch has nothing to do with implicit_flush ini directive.
-       if (OG(active_ob_buffer).chunk_size
-                       && OG(active_ob_buffer).text_length >= OG(active_ob_buffer).chunk_size) {
+       /* If implicit_flush is On, send contents to next buffer and return. */
+       if (OG(implicit_flush) || OG(active_ob_buffer).chunk_size
+               && OG(active_ob_buffer).text_length >= OG(active_ob_buffer).chunk_size)
+       {
                zval *output_handler = OG(active_ob_buffer).output_handler;
How does it have nothing to do with the implicit_flush INI directive, if turning this directive on calls php_start_implicit_flush(), which in turn sets OG(implicit_flush), and in turn used in that if?? Why is it necessary to explicitly disable output in all of the places where you added set_status() calls?? Zeev

« previous php.cvs (#14485) next »