Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:77127
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.223.173 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <540F8DAA.5090202@sugarcrm.com>
References: <004d01cfcc63$e7e3ac40$b7ab04c0$@tutteli.ch>
	<3DB7DEC9-2F99-4DDB-95A9-70455E4BABCB@ajf.me>
	<540F8DAA.5090202@sugarcrm.com>
Date: Tue, 9 Sep 2014 21:01:45 -0500
Message-ID: <CALrkBjacsB5VpBAghAuc0LtoNqg7p=b4xSek=u-B6mRhzigpnw@mail.gmail.com>
To: Stas Malyshev <smalyshev@sugarcrm.com>
Cc: Andrea Faulds <ajf@ajf.me>, Robert Stoll <php@tutteli.ch>, 
	"internals@lists.php.net" <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=089e0111b2c61881390502ac6ffa
Subject: Re: [PHP-DEV] make casts more strict in PHP 7
From: pencap@gmail.com (Mike Willbanks)

--089e0111b2c61881390502ac6ffa
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

> Hi!
>
> > No, no it would not. PHP=E2=80=99s explicit casts cannot fail, and ther=
e is
> > absolutely no good reason to change this. If people want strict
> > casting, we can add new functions or operators for that specifically.
> > But to break explicit casts and make them sometimes fail would cause
> > innumerable bugs and backwards-compatibility ideas.
>
> I agree. If the developer explicitly states they want conversion, it
> should be conversion. For implicit casts, some tightening may be ok, but
> for explicit ones I think it would bring much more headaches than it's
> worth.
>

Precisely, it would cause a vast amount of headaches for many projects.
 Think of user input where you are expecting an integer and it must be a
positive integer (which would ultimately be a string); it is much faster to
do a quick check such as:

$foo =3D (int) $_GET['foo'];
if ($foo > 0) {
    ....
}

I'd personally not have to wrap more items around it due to a notice or
warning.

--089e0111b2c61881390502ac6ffa--