Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:76461
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.192.42 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <CAESVnVr0J5qyAc-L0Yt_3Ahab5LVgtUErgoKsSPvgwe=QzppTw@mail.gmail.com>
References: <CAESVnVr0J5qyAc-L0Yt_3Ahab5LVgtUErgoKsSPvgwe=QzppTw@mail.gmail.com>
Date: Wed, 13 Aug 2014 09:47:38 +0200
Message-ID: <CAH-PCH4t1bnGwOmitU4cS4W--==8sZRhw1X-DkV0Y7zNj8E3kw@mail.gmail.com>
To: Sara Golemon <pollita@php.net>
Cc: PHP internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=001a11339280859c7c05007e007c
Subject: Re: [PHP-DEV] [RFC] Disallow multiple default blocks in a single
 switch statement
From: tyra3l@gmail.com (Ferenc Kovacs)

--001a11339280859c7c05007e007c
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

On Wed, Aug 6, 2014 at 6:38 AM, Sara Golemon <pollita@php.net> wrote:

> https://wiki.php.net/rfc/switch.default.multiple
>
> 'Cause this code is silly (even if it had case blocks), but we allow it:
>
> switch ($expr) {
>   default:
>     notExecuted();
>     break;
>   default:
>     executed();
> }
>
> --
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: http://www.php.net/unsub.php
>
>
Hi,

sorry to jump in this late, but I'm not sure that it is a good idea to only
reject the multiple default blocks but keep the ability to have the same
case multiple times:
http://3v4l.org/eZdPU
in this regard I think the current status is more consistent as it would
after merging this patch.
and I also think that this isn't an important enough issue to warrant a BC
break (albeit this is the better kind of BC: probably doesn't effect too
many people, and they will be clearly notified about the error at compile
time) so I voted no based on this two thing.


--=20
Ferenc Kov=C3=A1cs
@Tyr43l - http://tyrael.hu

--001a11339280859c7c05007e007c--