Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:82578
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: error (pb1.pair.com: domain mabe.berlin from 80.237.132.167 cause and error)
Message-ID: <54DD815F.70504@mabe.berlin>
Date: Fri, 13 Feb 2015 05:45:19 +0100
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.4.0
MIME-Version: 1.0
To: internals@lists.php.net
References: <CAF+90c8iVRU8-er+05CJqLb7Pg2viR_-zgwf5jmVyfVWx-+-8Q@mail.gmail.com> <54DD1BB1.1090501@gmail.com>
In-Reply-To: <54DD1BB1.1090501@gmail.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 7bit
Subject: Re: [PHP-DEV] Remove $this from incompatible context
From: dev@mabe.berlin (Marc Bennewitz)

Hi Stas,

Am 12.02.2015 um 22:31 schrieb Stanislav Malyshev:
> Hi!
>
>> class A {
>>      // This is an *instance* method, but it doesn't actually use $this.
>>      // This kind of usage is very common in PHP 4 era code, where
>>      // "static" annotations weren't used
>>      function test() {
>>          echo "foo";
>>      }
>> }
>>
>> class B {
>>      function test2() {
>>          // This call would be forbidden because it assumes $this of class
>> B, when
>>          // calling an instance method of class A. However A::test() does
>> not actually
>>          // use $this!
>>          A::test();
>>      }
>> }
> IMHO, this should work. Maybe issue E_STRICT, but even then I'd think
> it's not really necessary, I can't see any purpose E_STRICT serves in
> this case - since $this is not used, no potential bug is averted, as the
> code works exactly as it was supposed to be working.

Such code will break, not in the first place but later on!
You propose that every instance method not using the variable $this 
internally will be magically a static method and can never ever be 
changed to use $this without an bc break.

> <snip>
>

Marc