Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:112397
Content-Type: multipart/alternative;
	boundary="Apple-Mail=_A8BA29EF-9CD4-43D7-BB58-8CA72D0574E2"
Mime-Version: 1.0 (Mac OS X Mail 13.4 \(3608.120.23.2.4\))
Date: Thu, 3 Dec 2020 14:30:15 +0100
References: <34FA3E20-9D06-4D08-937E-4216FEF35504@gmail.com>
To: PHP Internals <internals@lists.php.net>
In-Reply-To: <34FA3E20-9D06-4D08-937E-4216FEF35504@gmail.com>
Message-ID: <7E98C687-7A24-49E7-A14C-3D68B4B3419E@gmail.com>
Subject: Missing warning for stat(), etc. when invoked with empty string
From: claude.pache@gmail.com (Claude Pache)

--Apple-Mail=_A8BA29EF-9CD4-43D7-BB58-8CA72D0574E2
Content-Transfer-Encoding: quoted-printable
Content-Type: text/plain;
	charset=utf-8

[Of course, I meant `stat()`, not `stats()`. Resending the message with =
correct spelling, including in subject line. Sorry.]


Hi,

`stat()` and friends invoked with empty string or null as argument, =
return `false` (indicating failure), but do not emit the Warning that is =
expected on failure. See: https://3v4l.org/jXC2N =
<https://3v4l.org/jXC2N>=20

That discrepancy between the two ways of indicating failure is =
problematic. Indeed, whenever I use one of those functions without =
prefixing it by @, I expect that it will either return a value of =
expected type (`array` in case of `stat()`), or run the panic procedure =
of my error handler.

I propose to ensure that those functions always emit a Warning in each =
and every situation they return false/null (provided, of course, that =
those functions are not supposed to return `false/null` in absence of =
failure, and that their purpose is not to check for failure condition as =
in `file_exists()`).

=E2=80=94Claude


--Apple-Mail=_A8BA29EF-9CD4-43D7-BB58-8CA72D0574E2--