Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:1126 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 24201 invoked from network); 1 May 2003 11:26:42 -0000 Received: from unknown (HELO bass.serverhost.net) (216.71.84.17) by pb1.pair.com with SMTP; 1 May 2003 11:26:42 -0000 Received: from exocet (ns.tunebiz.net [61.202.202.2]) by bass.serverhost.net (8.11.6/8.11.6) with SMTP id h41BQ9D17487; Thu, 1 May 2003 07:26:09 -0400 Message-ID: <200305011126.h41BQ9D17487@bass.serverhost.net> Date: Thu, 01 May 2003 20:26:14 +0900 To: Wez Furlong Cc: internals@lists.php.net In-Reply-To: References: <200305011054.h41AsQ811798@bass.serverhost.net> X-Mailer: Datula version 1.51.09 for Windows Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Subject: Re: stream_set_write_buffer() does not work From: fujimoto@techstyle.jp (Masaki Fujimoto) Ah, I understand the status of this issue (that will do for me), and I agree w/ every point you mentioned. Wez Furlong : >I am aware of this issue, however, reverting leaves us at the mercy of >problems on solaris. > >I do plan to implement a write buffer, however, introducing that into >the 4.3.x branch is probably a bad idea at this late stage. > >While I don't think that stream_set_write_buffer()/set_file_buffer() >having no effect is such a bad thing (most php scripts don't have any >need to call this function), I am concerned about the overall >performance of writes in the absence of a user-space write buffer. > >So I'm looking into a solution at the moment; there is no need to update >the documentation at this point. > >--Wez. > > >On Thu, 1 May 2003, Masaki Fujimoto wrote: > >> Hello, >> >> I noticed that stream_set_write_buffer() is no more available on local files >> because of the change made on 2003/03/18 to main/streams.c (1.125.2.41 -> >> 1.125.2.42: use a file descriptor instead of a file pointer). >> >> As a result of this change, of course stream_set_write_buffer() and >> set_file_buffer() does not work anymore in most of the cases because user-level >> I/O buffer functions like setvbuf() does not work on file descriptors, though >> I'm not sure if this is the "probelm" or not. >> >> If this is an intentional result, fixing documents (== add note like "stream_set >> _write_buffer does not work on local files, bla bla bla" and fix a sample code) >> will do, but if not, maybe we should implement php-level I/O buffer or revert >> this change (I'm not sure reverting this change makes another problem, >> though:) >> >> Regards, >> >> Masaki Fujimoto >> fujimoto@tunebiz.net >> fujimoto@php.net >> >> >> -- Masaki Fujimoto fujimoto@techstyle.jp / fujimoto@php.net http://techstyle.jp/