Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:101650
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: error (pb1.pair.com: domain duncanc.co.uk from 209.85.216.181 cause and error)
MIME-Version: 1.0
In-Reply-To: <CAP=4hCK7Wb-m+RO7prV01n=+hNewQnJaNb6EBO_EHQ66NjoynA@mail.gmail.com>
References: <CAP=4hCK7Wb-m+RO7prV01n=+hNewQnJaNb6EBO_EHQ66NjoynA@mail.gmail.com>
Date: Mon, 22 Jan 2018 09:52:51 +0000
Message-ID: <CAHE1EdsB7ZjmmmfLz20b6Z1q2XfYhnCyT9js3MKOer6bRWzOnA@mail.gmail.com>
To: Gabriel Caruso <carusogabriel34@gmail.com>
Cc: "internals@lists.php.net" <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="94eb2c0671f2c5590305635a643a"
Subject: Re: [PHP-DEV] [RFC] is_countable
From: php@duncanc.co.uk (Craig Duncan)

--94eb2c0671f2c5590305635a643a
Content-Type: text/plain; charset="UTF-8"

Hi Gabriel,

I proposed a similar function at the time of introducing the warning, but
it doesn't look like I got around to creating an RFC for it:
https://github.com/php/php-src/pull/2206/files

During this work we realised that objects don't have to implement the
'\Countable' interface to be successfully handled by 'count()',
internal/extension classes can implement a 'count_elements' handler and
ignore the interface.

I'd suggest that 'is_countable()' should be consistent with 'count()'.

Thanks,
Craig

--94eb2c0671f2c5590305635a643a--