> -----Original Message-----
> From: p...@golemon.com [mailto:p...@golemon.com] On Behalf Of Sara
> Golemon
> Sent: Sunday, December 10, 2017 1:50 AM
> To: Andreas Hennings <andr...@dqxtech.net>
> Cc: PHP internals <internals@lists.php.net>
> Subject: Re: [PHP-DEV] instanceof survives non-object variables, but crashes
> on non-object constants.
> 
> On Sat, Dec 9, 2017 at 1:28 AM, Andreas Hennings <andr...@dqxtech.net>
> wrote:
> > The following (https://3v4l.org/A2Tp6) is ok, it simply returns false:
> >
> >     $x = 1;
> >     $x instanceof \stdClass;
> >
> >
> > The following (https://3v4l.org/IdSBu) gives a fatal error:
> >
> >     1 instanceof \stdclass;
> >
> > t think this behavior is inconsistent, and we should consider changing it.
> >
> > There is one option, and it is BC.
> >
> > - Let 1 instanceof \stdClass return false, instead of crashing. ->
> > seems BC
> >
> This.  There's nothing undefinable about "1 instanceof \stdClass", the clear
> and obvious answer is: "No, it's not an instance of \stdClass, it's an 
> integer."
> The runtime result of false makes sense, the compile time error doesn't.
> Let's relax the compile time error (fold it into a const false if you truly 
> must)
> and call it done.  There's no need to be hostile to users just because the 
> code
> *seems* silly.

+1

Zeev

Reply via email to