Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:16267 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 74083 invoked by uid 1010); 17 May 2005 15:06:19 -0000 Delivered-To: ezmlm-scan-internals@lists.php.net Delivered-To: ezmlm-internals@lists.php.net Received: (qmail 42634 invoked from network); 17 May 2005 14:46:41 -0000 Received: from unknown (HELO pb1.pair.com) (127.0.0.1) by localhost with SMTP; 17 May 2005 14:46:41 -0000 X-Host-Fingerprint: 69.93.155.18 brian.3gupload.com Received: from ([69.93.155.18:48120] helo=brian.3gupload.com) by pb1.pair.com (ecelerity 1.2 r(5656M)) with SMTP id 48/56-01538-1D30A824 for ; Tue, 17 May 2005 10:46:41 -0400 Received: from [192.168.1.26] (internal.3gupload.com [68.253.30.206]) by brian.3gupload.com (Postfix) with ESMTP id 33D6C4F0BD5; Tue, 17 May 2005 07:21:04 -0700 (PDT) Message-ID: <4289FE0C.7060501@3gupload.com> Date: Tue, 17 May 2005 09:22:04 -0500 Organization: 3GUpload.com, Inc. User-Agent: Mozilla Thunderbird 1.0 (X11/20041207) X-Accept-Language: en-us, en MIME-Version: 1.0 To: Wez Furlong Cc: Derick Rethans , George Schlossnagle , PHP Developers Mailing List References: <4288F183.7010804@3gupload.com> <4289DDBA.8020809@3gupload.com> <4e89b426050517070932202006@mail.gmail.com> In-Reply-To: <4e89b426050517070932202006@mail.gmail.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [PHP-DEV] Patch for php_error_cb From: bmatheny@3gupload.com (Blake Matheny) After George's comments, I think I'll just write up an extension and throw it out there. I can still support a script:,file:,socket:,sql: convention for naming and meet all of my requirements. Thanks for the feedback. -Blake Wez Furlong wrote: > On 5/17/05, Blake Matheny wrote: >>- No need to add an extension every time you upgrade PHP > > But you still need to write a custom shared library, so this is not > such a great point. > >>- No additional patching (which an extension essentially requires) > > But you still need to write a custom shared library. > >>- Only adds a couple of extra operations (no performance hit) >>- Backwards compatible with existing php.ini > > I don't think those two points are relevant. > >>- Possibility for other prefixes (script:,file:,socket:,etc) > > This is the only plus point in your argument, but you should consider > that we already have a mechanism for logging to files, and no one has > asked to be able to log to anything else for quite some time. > > I think you really need to do some convincing to get the patch in, and > I don't see many benefits to the majority of PHP users with the patch > as it stands now. > > --Wez. -- Blake Matheny 3GUpload.com, Inc. Director of Technology 317-472-4962 (W) 317-201-2840 (C) bmatheny@3gupload.com