Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:102471
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.128.195 as permitted sender)
To: internals@lists.php.net
References: <CAF+90c_xCxxakMBpU45Xow1BkhS+O7ZxBENQMSEY+CrSa_C0YA@mail.gmail.com>
 <17c367f8-3aeb-277f-36c3-f019e6d0d27a@blueyonder.co.uk>
Message-ID: <94ff9afc-b3b0-0ae2-55bd-ce6c3936967d@gmail.com>
Date: Tue, 26 Jun 2018 21:49:29 +0100
User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101
 Thunderbird/52.8.0
MIME-Version: 1.0
In-Reply-To: <17c367f8-3aeb-277f-36c3-f019e6d0d27a@blueyonder.co.uk>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 8bit
Content-Language: en-GB
Subject: Re: [PHP-DEV] [RFC] Deprecate and remove continue targeting switch
From: rowan.collins@gmail.com (Rowan Collins)

On 26/06/2018 20:28, niel wrote:
> On 24/06/18 17:16, Nikita Popov wrote:
>> Hi internals,
>>
>> Another small deprecation for your consideration...
>>
>> https://wiki.php.net/rfc/continue_on_switch_deprecation
>>
>> Regards,
>> Nikita
>>
> IMO the proposed illustrations will lead to as much, if not more, WTF 
> from other language users as current behaviour. Why? Because the 
> programmer still has to account for the 'switch' in how continue 
> works. This will just cause a BC problem for older code without adding 
> anything for people new to PHP.
>
> My suggestion, is to make the break (no pun intended) be clean. 
> Deprecate use of continue from usage to escape switch completely in 
> next PHP, then remove that functionality in PHP 8.
>

Do you mean that this code would be valid in both PHP 7 and PHP 8, but 
mean different things?

while($outer) {
     while($inner) {
         switch($foo) {
             case 1:
                 continue 2;
         }
     }
}

If so, I think that would be even more confusing.

Regards,

-- 
Rowan Collins
[IMSoP]