Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:112376
To: Silvio Sparapano <ssilvio@libero.it>, Josh Bruce <josh@joshbruce.dev>,
 Nikita Popov <nikita.ppv@gmail.com>
Cc: PHP internals <internals@lists.php.net>
References: <CAF+90c9SQfb8G8gQ4o4o4thXauZx1qAeV_YZrC0wtTo9s+ornQ@mail.gmail.com>
 <B5D2C484-1DD9-4FC7-9A2C-1E023994AB6E@joshbruce.dev>
 <fdfca53a8e0905a6949be0f4e29d0067@smtp-32.iol.local>
Message-ID: <ecaffeaf-0339-b263-5bbf-76191701f15f@gmx.de>
Date: Wed, 2 Dec 2020 17:52:15 +0100
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:78.0) Gecko/20100101
 Thunderbird/78.5.0
MIME-Version: 1.0
In-Reply-To: <fdfca53a8e0905a6949be0f4e29d0067@smtp-32.iol.local>
Content-Type: text/plain; charset=utf-8
Content-Language: de-DE
Content-Transfer-Encoding: quoted-printable
Subject: Re: R: [PHP-DEV] [RFC] Deprecate passing null to
 non-nullableargumentsof internal functions
From: cmbecker69@gmx.de ("Christoph M. Becker")

On 02.12.2020 at 17:34, Silvio Sparapano wrote:

> Hello,
> IMHO it=E2=80=99s only a declaration issue.
>
> In other terms, if documentation of strlen says:
>
> strlen=C2=A0(=C2=A0string=C2=A0$string=C2=A0) :=C2=A0int
>
> then I mean that $string MUST be a string, and return value MUST be an i=
nt.
>
> But then we found that if $string is null, zero is returned. If $string =
is an array, null is returned =E2=80=A6
> We simply need consistency between how a function is documented, and how=
 it really works.
> I mean, it may also be convenient for strlen to accept a parameter type =
other than string (such as null), but it must be well documented.

It is reasonably well documented:

| If the parameters given to a function are not what it expects, such as
| passing an array where a string is expected, the return value of the
| function is undefined. In this case it will likely return NULL but
| this is just a convention, and cannot be relied upon.

[1] <https://www.php.net/manual/en/functions.internal.php>

Regards,
Christoph