Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:89390
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 74.125.82.48 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <84.DA.57156.C6B94565@pb1.pair.com>
References: <56547DFE.3080407@php.net>
	<C8.C9.57156.EB494565@pb1.pair.com>
	<565496FA.2010604@gmail.com>
	<84.DA.57156.C6B94565@pb1.pair.com>
Date: Tue, 24 Nov 2015 17:29:22 +0000
Message-ID: <CABHuAWw_H04w8TYRv4ZV1gaG3uxPVWY7jMm=mNu3TON-A3WMxA@mail.gmail.com>
To: Andrea Faulds <ajf@ajf.me>
Cc: PHP internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=089e010d84e8b9b0cd05254cae38
Subject: Re: [PHP-DEV] Re: Scalar Type Declaration Syntax Weirdness
From: t.carnage@gmail.com (Chris Riley)

--089e010d84e8b9b0cd05254cae38
Content-Type: text/plain; charset=UTF-8

On 24 November 2015 at 17:16, Andrea Faulds <ajf@ajf.me> wrote:

> Hi Stas,
>
> Stanislav Malyshev wrote:
>
>> Hi!
>>
>>        <?php
>>>>        function a(\int $i) {}
>>>>
>>>>    Is it intentional that the \ in front of the "int" is allowed? IMHO,
>>>> this
>>>>    confusing notation must not be allowed.
>>>>
>>>
>>> This is weird and I'd consider it a bug. You can't do \array or
>>> \callable, and if I saw \int, I'd think it meant a class of that name
>>> rather than a scalar type.
>>>
>>
>> I would assume \int means class named "int", as opposed to "int" type.
>>
>
> That's also what I'd expect. However, "int" is not allowed as a class name
> in PHP 7. And unfortunately what the code Sebastian posted sctually does is
> act as an integer type hint, not as a class type hint.
>
> Can this be fixed for 7.0.0?
>>>
>>
>> I don't think this would be a good idea. We're in the final stretch of
>> release cycle, and should not do any non-urgent fixes. This does not
>> look urgent. It can wait for 7.0.1.
>>
>
> It can't wait for 7.0.1, because banning this would be a
> backwards-compatibility break with 7.0.0. We have to fix it in 7.0.0 or not
> fix it ever.
>
> Thanks.
>
> --
> Andrea Faulds
> http://ajf.me/
>
> --
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: http://www.php.net/unsub.php
>
>
Could we not just document it as \int is not allowed and throws a fatal
error. Then we'd just be fixing a bug in documented behaviour.

~C

--089e010d84e8b9b0cd05254cae38--