Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:125812
Feedback-ID: i8414410d:Fastmail
Precedence: bulk
MIME-Version: 1.0
Date: Sun, 20 Oct 2024 15:21:49 -0500
To: "php internals" <internals@lists.php.net>
Message-ID: <20340e0e-ddd0-44cd-b555-1402857eb20f@app.fastmail.com>
In-Reply-To: 
 <ZydAaayECTRNG8Ackhb0tDgYjmdFKnkW4p0ISWlLFP5My4CsJBR1KKTylrD14Ags2GUyBLoJPf68N6Z6LoVzjG-tru8YAbmi74W5xzSkJ8I=@gpb.moe>
References: 
 <ZydAaayECTRNG8Ackhb0tDgYjmdFKnkW4p0ISWlLFP5My4CsJBR1KKTylrD14Ags2GUyBLoJPf68N6Z6LoVzjG-tru8YAbmi74W5xzSkJ8I=@gpb.moe>
Subject: Re: [PHP-DEV] [RFC] Change behaviour of array sort functions to return a copy
 of the sorted array
Content-Type: text/plain
Content-Transfer-Encoding: 7bit
From: larry@garfieldtech.com ("Larry Garfield")

On Sun, Oct 20, 2024, at 12:42 PM, Gina P. Banyard wrote:
> Hello internals,
>
> I would like to propose a short RFC to make the return value of the 
> sort() and similar functions more useful:
> https://wiki.php.net/rfc/array-sort-return-array
>
> I intend for the discussion to last 2 weeks and then open the vote.
>
> Best regards,
>
> Gina P. Banyard

Like others, I'm skeptical of the behavior here not being all that clear.

What several other languages do, and what I have in the plan for collections if/when they ever happen, is sort() sorts in place, while sorted() returns a new value.  There's similarly reverse() (in place) and reversed() (new value returned).  

*sorted($arr) seems like it would be a lot less confusing, and consistent with what other languages (and hopefully future PHP) do.  (I don't know what that means for array_walk(), but I don't know what that would even return anyway.)

--Larry Garfield