Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:101417
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: error (pb1.pair.com: domain kelunik.com from 81.169.146.163 cause and error)
MIME-Version: 1.0
References: <72392123-d37b-26df-6886-218f48205f8a@fleshgrinder.com> <5171148a-3554-35a3-ab72-370e1aac4a3f@php.net>
In-Reply-To: <5171148a-3554-35a3-ab72-370e1aac4a3f@php.net>
Date: Tue, 26 Dec 2017 15:36:54 +0000
Message-ID: <CANUQDCgVvgf+7UBiiJrg7dWBE1vzMcuDjEKWtvDrDrViDeeEOg@mail.gmail.com>
To: PHP Internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="001a11407d6810aa1a0561400e70"
Subject: Re: [PHP-DEV] [RFC] [DISCUSSION] Scalar Pseudo-type
From: me@kelunik.com (Niklas Keller)

--001a11407d6810aa1a0561400e70
Content-Type: text/plain; charset="UTF-8"

>
> Am 24.12.2017 um 15:34 schrieb Fleshgrinder:
> > I prepared a PR to add the `scalar` pseudo-type to PHP after the
> > discussions around adding a `mixed` pseudo-type. I strongly believe that
> > it makes sense to provide the most common primitive union types with
> > handy aliases even if we are going to add union types in the future to
> PHP.
>
> Thank you, Richard, for working on this.
>
> I spent a lot of time this year introducing scalar type declarations into
> existing code bases. In quite a few cases I was not able to do so because
> the existing code works with parameters that can be of two or more scalar
> types. With PHP 7.2, I can only document this outside of the code using
> @param annotations (until the code has been refactored to work with only
> one parameter type).
>
> With a "scalar" type declaration I would not have to fall back to @param
> annotations and could express the type in actual syntax.
>

If you want just two of those and not all, you'd still have to use PHPdoc
for those?

IIRC you voted against union types, which would be a better fit in that
case?

Regards, Niklas

>

--001a11407d6810aa1a0561400e70--