Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:122933
Content-Type: text/plain;
	charset=utf-8
Precedence: bulk
Mime-Version: 1.0 (Mac OS X Mail 16.0 \(3774.500.171.1.1\))
Subject: Re: [PHP-DEV] RFC idea: using the void type to control maximum arity
 of user-defined functions
In-Reply-To: <CA+pig-JBkZip5bEM0gSwcwNRO9NDKuRO2vu=iHnx2diBTv9Bng@mail.gmail.com>
Date: Thu, 4 Apr 2024 16:29:18 +0200
Cc: Pablo Rauzy <r@uzy.me>,
 internals@lists.php.net
Content-Transfer-Encoding: quoted-printable
Message-ID: <7B32AF65-CA40-40F5-BA59-CB5180EC4D7F@gmail.com>
References: <6299b649-c19b-4172-9632-2ef0a55d256d@uzy.me>
 <CA+pig-JBkZip5bEM0gSwcwNRO9NDKuRO2vu=iHnx2diBTv9Bng@mail.gmail.com>
To: Mark Trapp <mark@itafroma.com>
From: claude.pache@gmail.com (Claude Pache)



> Le 4 avr. 2024 =C3=A0 15:03, Mark Trapp <mark@itafroma.com> a =C3=A9crit=
 :
>=20
> On Thu, Apr 4, 2024 at 5:43=E2=80=AFAM Pablo Rauzy <r@uzy.me> wrote:
>=20
> I like this concept, but instead of introducing a new syntax, have you
> considered leveraging attributes in the same way that PHP 8.3
> introduced #[Override]?
>=20
> #[Nonvariadic]
> function foo () {}
> foo(42); // warning: foo() expects exactly 0 arguments, 1 given
>=20
> I think the intent would be clearer and it would avoid introducing a =
new syntax.
>=20

Indeed, except that I think it should be better to have *by default* the =
same behaviour between built-in functions and user-defined functions:

```php
function foo() { }

foo(42);
// in PHP 8.x: Deprecated: foo() expects expects exactly 0 arguments, 1 =
given
// in PHP 9: Fatal error: foo() expects expects exactly 0 arguments, 1 =
given
```

and, in case you want a variadic function:

```php
#[Variadic]
function foo() { }
```

Alternatively, we don=E2=80=99t really need to introduce a new =
attribute, because one can just write:

```php
function foo(...$args) { }
```


=E2=80=94Claude