Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:68675
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.219.42 as permitted sender)
MIME-Version: 1.0
Date: Thu, 29 Aug 2013 11:33:24 +0200
Message-ID: <CAF+90c_P6AVw1fB7BWeQSHdhLDNcWDnehZ2j7Hk6dzznZfKF=w@mail.gmail.com>
To: PHP internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=089e0139fc76337e8a04e512cc7f
Subject: Signature compatibility: Number of arguments
From: nikita.ppv@gmail.com (Nikita Popov)

--089e0139fc76337e8a04e512cc7f
Content-Type: text/plain; charset=ISO-8859-1

Hi internals!

This is a spinoff from the variadics thread. Quoting Stas:

> I also think this:
>     public function query($query, ...$params)
>     public function query(...$params)
> should be legal too.

This is a general issue in PHP that we might want to fix: Currently a
method A is not compatible with a method B if A has less arguments than B.

E.g. both of the following are incompatible signatures:

    public function foo($bar)
    public function foo()

    public function foo($bar = NULL)
    public function foo()

This doesn't really make sense. Removing parameters doesn't violate LSP,
because in PHP it is legal to pass more arguments than declared. (Only
adding additional required parameters would violate LSP.)

Is it okay to change this?

Thanks,
Nikita

--089e0139fc76337e8a04e512cc7f--