Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:61536 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 48887 invoked from network); 20 Jul 2012 00:58:16 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 20 Jul 2012 00:58:16 -0000 Authentication-Results: pb1.pair.com header.from=smalyshev@sugarcrm.com; sender-id=pass Authentication-Results: pb1.pair.com smtp.mail=smalyshev@sugarcrm.com; spf=pass; sender-id=pass Received-SPF: pass (pb1.pair.com: domain sugarcrm.com designates 67.192.241.173 as permitted sender) X-PHP-List-Original-Sender: smalyshev@sugarcrm.com X-Host-Fingerprint: 67.192.241.173 smtp173.dfw.emailsrvr.com Linux 2.6 Received: from [67.192.241.173] ([67.192.241.173:52932] helo=smtp173.dfw.emailsrvr.com) by pb1.pair.com (ecelerity 2.1.1.9-wez r(12769M)) with ESMTP id A2/B2-18983-82DA8005 for ; Thu, 19 Jul 2012 20:58:16 -0400 Received: from localhost (localhost.localdomain [127.0.0.1]) by smtp7.relay.dfw1a.emailsrvr.com (SMTP Server) with ESMTP id AE1BE2586F8; Thu, 19 Jul 2012 20:58:13 -0400 (EDT) X-Virus-Scanned: OK Received: by smtp7.relay.dfw1a.emailsrvr.com (Authenticated sender: smalyshev-AT-sugarcrm.com) with ESMTPSA id 4F41A2586F3; Thu, 19 Jul 2012 20:58:13 -0400 (EDT) Message-ID: <5008AD24.4020003@sugarcrm.com> Date: Thu, 19 Jul 2012 17:58:12 -0700 Organization: SugarCRM User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:14.0) Gecko/20120713 Thunderbird/14.0 MIME-Version: 1.0 To: Gustavo Lopes CC: Nikita Popov , Sara Golemon , "internals@lists.php.net" References: <5008A143.7010203@sugarcrm.com> In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Subject: Re: [PHP-DEV] zend_parse_parameters() improvements From: smalyshev@sugarcrm.com (Stas Malyshev) Hi! > And just to clear, I agree with Sara that ! (both for l/d/b and the > pointer types) should not be redefined to "parameter was passed". But it's > my understanding that that's not what you're planning. I didn't have yet a good generic way to handle l/d/b with 'default' - in many cases for now I prohibited using 'default' in current code in this context - but if we had some improvement that allows to do it - like your patch, though not with ! - it would be definitely an improvement. I'll be done with rest of ZEND_NUM_ARGS() cleanup probably by the end of next week, and then I could look into integrating your changes, I think it makes my patch work much better in some cases :) -- Stanislav Malyshev, Software Architect SugarCRM: http://www.sugarcrm.com/ (408)454-6900 ext. 227