Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:76153 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 69673 invoked from network); 25 Jul 2014 23:26:27 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 25 Jul 2014 23:26:27 -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 108.166.43.99 as permitted sender) X-PHP-List-Original-Sender: smalyshev@sugarcrm.com X-Host-Fingerprint: 108.166.43.99 smtp99.ord1c.emailsrvr.com Linux 2.6 Received: from [108.166.43.99] ([108.166.43.99:54199] helo=smtp99.ord1c.emailsrvr.com) by pb1.pair.com (ecelerity 2.1.1.9-wez r(12769M)) with ESMTP id 4D/21-08559-F97E2D35 for ; Fri, 25 Jul 2014 19:26:26 -0400 Received: from localhost (localhost.localdomain [127.0.0.1]) by smtp13.relay.ord1c.emailsrvr.com (SMTP Server) with ESMTP id AD2DC3807D9; Fri, 25 Jul 2014 19:26:27 -0400 (EDT) X-Virus-Scanned: OK Received: by smtp13.relay.ord1c.emailsrvr.com (Authenticated sender: smalyshev-AT-sugarcrm.com) with ESMTPSA id F18593807AC; Fri, 25 Jul 2014 19:26:26 -0400 (EDT) X-Sender-Id: smalyshev@sugarcrm.com Received: from Stass-MacBook-Pro.local ([UNAVAILABLE]. [74.85.23.222]) (using TLSv1.2 with cipher DHE-RSA-AES128-SHA) by 0.0.0.0:465 (trex/5.2.10); Fri, 25 Jul 2014 23:26:27 GMT Message-ID: <53D2E7A2.9010209@sugarcrm.com> Date: Fri, 25 Jul 2014 16:26:26 -0700 Organization: SugarCRM User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.7; rv:31.0) Gecko/20100101 Thunderbird/31.0 MIME-Version: 1.0 To: Ferenc Kovacs CC: Alexey Zakhlestin , Marco Pivetta , Sebastian Bergmann , Julien Pauli , Remi Collet , PHP Internals References: <53A1C722.9060501@fedoraproject.org> <53A21137.6010705@sugarcrm.com> <53A2A9BD.1070603@sugarcrm.com> <53A3874E.20704@sugarcrm.com> <53A62AFF.4080302@sugarcrm.com> <53B10D59.4060206@sugarcrm.com> <53D2DA3B.9020308@sugarcrm.com> In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Subject: Re: [PHP-DEV] Problems with the fix for the BC break introduced in 5.4.29 and 5.5.13 From: smalyshev@sugarcrm.com (Stas Malyshev) Hi! > yeah, that would work ofc, but as these libs seems to have instanitate > arbitrary classes, that would require either generating files on the fly > and including them or simply evaling them, but of those are a bit > dirtier than using Reflection for the same job. True but that's what phpunit, etc. are doing for mocks anyway, aren't they? > true, but it can also be used to argue for loosening the restriction, > why restrict something from Reflection, which is already possible from > simple class extension. I agree, probably makes sense to allow it if you can do it anyway. Reflection is not something you can trigger without explicit codding (unlike O: thing) so it's fine with me. > a nice thing from OOP POV and also will cause problems if/when we > introduce a reflection method removing final from classes/methods (this > was already proposed not that long ago with a working patch but was > turned down because other reasons). That probably wouldn't be a good idea, especially for internal classes. -- Stanislav Malyshev, Software Architect SugarCRM: http://www.sugarcrm.com/