Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:64936 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 11025 invoked from network); 14 Jan 2013 16:36:47 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 14 Jan 2013 16:36:47 -0000 Authentication-Results: pb1.pair.com header.from=dmitry@zend.com; sender-id=unknown Authentication-Results: pb1.pair.com smtp.mail=dmitry@zend.com; spf=unknown; sender-id=unknown Received-SPF: unknown (pb1.pair.com: domain zend.com does not designate 209.85.214.173 as permitted sender) X-PHP-List-Original-Sender: dmitry@zend.com X-Host-Fingerprint: 209.85.214.173 mail-ob0-f173.google.com Received: from [209.85.214.173] ([209.85.214.173:38840] helo=mail-ob0-f173.google.com) by pb1.pair.com (ecelerity 2.1.1.9-wez r(12769M)) with ESMTP id 26/E3-22727-D1434F05 for ; Mon, 14 Jan 2013 11:36:46 -0500 Received: by mail-ob0-f173.google.com with SMTP id xn12so4093834obc.18 for ; Mon, 14 Jan 2013 08:36:43 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type:x-gm-message-state; bh=i3vUp/Fg3U5dhMp1MnwACNlGJdrgwQ/wElwHooIjoos=; b=SttANAsNBrAX+FSqR1xgrV286koz/BgqfH2o3S4mZauD3LUC4PbMsbAkFNJBZDgnie ggooH5vmCwqZatI0ITXAMvh68YxvjN7uHfJG0AilqYRfoHvdrxVmSfLToaxjfwgBMTd/ 5cNwYKgNKZPO1iuzmCdo8azPkn6dBmdr+rUMu72EWqLibRianKWUoXG1xnMhrL/CznxI dHki1ROJn6nK9A4UdzeQwPglNIKO78zLH1xUFOewoCp96yiSTVYCjWHPi6gFieUkjVGD 5mCQe9Hgt5nLZFwJNqAGJFyfgCKjJUYFeMLC6yW8i0YT9RNWVjMEPx2+9GhpDsy7chLg PmgQ== MIME-Version: 1.0 Received: by 10.60.0.138 with SMTP id 10mr51401102oee.142.1358181403053; Mon, 14 Jan 2013 08:36:43 -0800 (PST) Received: by 10.182.97.198 with HTTP; Mon, 14 Jan 2013 08:36:42 -0800 (PST) In-Reply-To: <50F3BFBA.9070701@sugarcrm.com> References: <50F3BFBA.9070701@sugarcrm.com> Date: Mon, 14 Jan 2013 19:36:42 +0300 Message-ID: To: Stas Malyshev Cc: PHP Internals Content-Type: multipart/alternative; boundary=e89a8fb204c411cea904d342403d X-Gm-Message-State: ALoCoQlJ8qiDKiYsVYbVgaI6z/W0nWtsjO1Pqg9WAs5183QmAQfpbANAOlm3YPsM2n9kkQS5T0y4wp7pC3KtcVVfpetvAiCsLu1Jvwhb/X23u6IWJOTadF2Ncv2Obq2WpfWmXyxaWzFK Subject: Re: Fix for duplicate magic methods calls (bug #63462) From: dmitry@zend.com (Dmitry Stogov) --e89a8fb204c411cea904d342403d Content-Type: text/plain; charset=UTF-8 Hi Stas, Sorry for delay, I'll able to take a look only tomorrow or after tomorrow. Thanks. Dmitry. On Monday, January 14, 2013, Stas Malyshev wrote: > Hi! > > I made a fix for bug #63462 - https://github.com/php/php-src/pull/258 - > which changes a bit how we do guards by unmangling the names before > applying guards. This is a slight change of behavior and also means that > all private vars with the same name would use the same guard (all > protected and public vars with the same name are the same var anyway, so > they should be doing it in any case). > Since it's a behavior change in core, I'd appreciate a second pair of > eyes to see if nothing is broken by it. All tests pass but better safe > then sorry. If no problems found in it, I'll commit it soon. > > Thanks, > -- > Stanislav Malyshev, Software Architect > SugarCRM: http://www.sugarcrm.com/ > (408)454-6900 ext. 227 > --e89a8fb204c411cea904d342403d--