Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:44296 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 74337 invoked from network); 16 Jun 2009 08:07:52 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 16 Jun 2009 08:07:52 -0000 Authentication-Results: pb1.pair.com header.from=derick@php.net; sender-id=unknown Authentication-Results: pb1.pair.com smtp.mail=derick@php.net; spf=unknown; sender-id=unknown Received-SPF: unknown (pb1.pair.com: domain php.net does not designate 82.94.239.7 as permitted sender) X-PHP-List-Original-Sender: derick@php.net X-Host-Fingerprint: 82.94.239.7 mail.jdi.nl Received: from [82.94.239.7] ([82.94.239.7:52308] helo=mail.jdi-ict.nl) by pb1.pair.com (ecelerity 2.1.1.9-wez r(12769M)) with ESMTP id B5/A9-18251-4D2573A4 for ; Tue, 16 Jun 2009 04:07:50 -0400 Received: from localhost (localhost [127.0.0.1]) by mail.jdi-ict.nl (8.13.7/8.12.11) with ESMTP id n5G87jgq029457; Tue, 16 Jun 2009 10:07:45 +0200 X-Virus-Scanned: by amavisd-new at jdi-ict.nl Received: from localhost (localhost [127.0.0.1]) by jdi.jdi-ict.nl (8.13.7/8.12.11) with ESMTP id n5G87ea4029445; Tue, 16 Jun 2009 10:07:42 +0200 Date: Tue, 16 Jun 2009 10:07:40 +0200 (CEST) X-X-Sender: derick@kossu.ez.no To: Stanislav Malyshev cc: "'PHP Internals'" In-Reply-To: <4A31462B.1070302@zend.com> Message-ID: References: <4A31462B.1070302@zend.com> User-Agent: Alpine 2.00 (DEB 1167 2008-08-23) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-3.0 (jdi.jdi-ict.nl [127.0.0.1]); Tue, 16 Jun 2009 10:07:42 +0200 (CEST) Subject: Re: bug #48247 again From: derick@php.net (Derick Rethans) On Thu, 11 Jun 2009, Stanislav Malyshev wrote: > Bug #48247 appears to be not entirely fixed still, it still produces a torrent > of warnings in certain configs (namely, take php.ini-production and edit it to > enable error_log, don't touch anything else - I get 400K worth of warnings). > Attached patch should fix it. Please tell if there's some objection to it, > otherwise I'll commit it tomorrow. The attached patch does fix one of the symtomps of the issue, but not the real problem. The real problem here is that from an error handler, another error is thrown. Instead of fixing this as a hack in any extension that has influence on formatting of error messages or error handling (through overriding the callback), a fix should be done in the error handling mechanism in order to prevent an error message generating another error message. regards, Derick -- http://derickrethans.nl | http://ezcomponents.org | http://xdebug.org twitter: @derickr