Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:94871
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: error (pb1.pair.com: domain daveyshafik.com from 209.85.216.178 cause and error)
MIME-Version: 1.0
Sender: me@daveyshafik.com
Date: Fri, 5 Aug 2016 19:26:19 -0700
Message-ID: <CANaX5ncVr5idYUtVtgjEYxRwjabi3f6Bg1-cKtoGvym=Ck1PLA@mail.gmail.com>
To: "internals@lists.php.net" <internals@lists.php.net>, Dmitry Stogov <dmitry@zend.com>
Content-Type: multipart/alternative; boundary=001a11c003868e7d7405395de885
Subject: Change to Too Few Args Exception RFC
From: davey@php.net (Davey Shafik)

--001a11c003868e7d7405395de885
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

Hey all,

I know this is a little late in the process, but it's something I've
noticed while prepping some content around 7.1.

This RFC: https://wiki.php.net/rfc/too_few_args

Passed, and has been implemented, but I feel that throwing an `\Error`
exception is a mistake. I think we should another more concrete exception
class for this error:

`\TooFewArgumentsError extends \Error`

A use case where this may trivially occur is where you are using argument
unpacking and the unpacked array is too small. Writing this, just looks bad=
:

try {
   foo(=E2=80=A6 $args);
} catch (\Error $e) { }

compared:

try {
   foo(=E2=80=A6 $args);
} catch (\TooFewArgumentsError $e) { }

Thoughts? Dmitry?

Given the tiny change this is, and that is backwards compatible with the
original RFC, I would like to add this to 7.1 for beta3.

I think I can make this change myself.

- Davey

--001a11c003868e7d7405395de885--