Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:102593
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.218.53 as permitted sender)
MIME-Version: 1.0
References: <CAF+90c_xCxxakMBpU45Xow1BkhS+O7ZxBENQMSEY+CrSa_C0YA@mail.gmail.com>
 <CAGa2bXYGv8R6GSX_M42TtqLmKDhJi3oLh6HRbePbsrdK0CKAKw@mail.gmail.com>
In-Reply-To: <CAGa2bXYGv8R6GSX_M42TtqLmKDhJi3oLh6HRbePbsrdK0CKAKw@mail.gmail.com>
Date: Thu, 5 Jul 2018 00:20:36 +0200
Message-ID: <CAJW__o1tx270d8inTJNz+7=k4NAA8paY1pRcTYeGF-TT28J41g@mail.gmail.com>
To: Yasuo Ohgaki <yohgaki@ohgaki.net>
Cc: Nikita Popov <nikita.ppv@gmail.com>, Internals <internals@lists.php.net>
Content-Type: text/plain; charset="UTF-8"
Subject: Re: [PHP-DEV] [RFC] Deprecate and remove continue targeting switch
From: kalle@php.net (Kalle Sommer Nielsen)

Den tor. 5. jul. 2018 kl. 00.15 skrev Yasuo Ohgaki <yohgaki@ohgaki.net>:
> Since the issue is incompatibility between current "continue" and "break",
> how about provide a tool replace "continue" to "break" where it is
> applicable.
> (Raise error for invalid "continue" usage also. This would be a bug most
> likely.)
>
> Then there wouldn't be compatibility issue anymore.

I like the idea of a tool for such, but I think its better to
implement it in some of the static analyzer projects like PHPStan and
the like to catch these.



-- 
regards,

Kalle Sommer Nielsen
kalle@php.net