Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:80822
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 74.125.82.49 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <53F8A1FF-C809-4DDF-9C6F-6916C3E4F044@ajf.me>
References: <53F8A1FF-C809-4DDF-9C6F-6916C3E4F044@ajf.me>
Date: Mon, 19 Jan 2015 15:07:14 +0100
Message-ID: <CAF+90c_kkD0xb_1=W=1bt00LfbXaN+JAn=kd7=MGmiFu_dTWXg@mail.gmail.com>
To: Andrea Faulds <ajf@ajf.me>
Cc: PHP internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=089e010d8a76e42628050d01d6bc
Subject: Re: [PHP-DEV] [RFC] Combined Comparison (Spaceship) Operator
From: nikita.ppv@gmail.com (Nikita Popov)

--089e010d8a76e42628050d01d6bc
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

On Mon, Jan 19, 2015 at 9:28 AM, Andrea Faulds <ajf@ajf.me> wrote:

> Good morning,
>
> This is a reboot of Davey Shafik=E2=80=99s RFC (with permission). After r=
ecent
> discussions about sort functions, I was inspired to bring this back up, a=
s
> I think it would be a useful feature.
>
> The RFC is here: https://wiki.php.net/rfc/combined-comparison-operator
>
> Thanks!
>

I like the idea behind this (exposing internal compare_function), but I
don't want to have an extra operator for a minor use case. Please just
introduce a function instead.

The operator is also less clear ... if I see "$a <=3D> $b" and do not happe=
n
to be familiar with another language implementing this operator, I won't
have any idea what it does (the closest thing <=3D> could be is the
equivalence operator from logic, and that's not the case here). On the
other hand something like compare($a, $b) is pretty obvious. If there is
concern about clashing with user-defined functions, lets namespace it.

A function can also be directly used as a callback. Furthermore a function
would fit in well with existing comparison functions like strcmp etc.

Nikita

Nikita

--089e010d8a76e42628050d01d6bc--