Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:116480
MIME-Version: 1.0
References: <99cc5fab-42ff-f865-cdfc-8942557c86b8@gmx.de>
In-Reply-To: <99cc5fab-42ff-f865-cdfc-8942557c86b8@gmx.de>
Date: Mon, 22 Nov 2021 22:06:47 +0100
Message-ID: <CAF+90c_4hNKU2-srqNE3JXePkEUsho=s_hVq3qeAOQNPcepRfg@mail.gmail.com>
To: Andreas <syntheme@gmx.de>
Cc: internals@lists.php.net
Content-Type: multipart/alternative; boundary="000000000000ad6df805d16701df"
Subject: Re: [PHP-DEV] PHP 8.2 proposal: "match", allow "default" as
 conditional_expression, e.g. 'en_US', 'en_GB', default => loadDefaults()
From: nikita.ppv@gmail.com (Nikita Popov)

--000000000000ad6df805d16701df
Content-Type: text/plain; charset="UTF-8"

On Mon, Nov 22, 2021 at 7:18 PM Andreas <syntheme@gmx.de> wrote:

> Hi internals,
>
> This is a proposal to allow to append the `default` pattern by comma to
> the end of the last match branch. (Like a conditional_expression)
>
> This allows to re-use the return_expression if required and avoids code
> duplication.
>
> PROPOSAL: PHP 8.2
>
> <?php
> return match ($locale) {
>      'de_DE', 'de_CH', 'de_AT' => loadGermanLanguageSettings(),
>      'en_US', 'en_GB', default => loadDefaultLanguageSettings(),
> };
> ?>
>

Isn't this equivalent to just this?

<?php
return match ($locale) {
     'de_DE', 'de_CH', 'de_AT' => loadGermanLanguageSettings(),
     default => loadDefaultLanguageSettings(),
};

'en_US' and 'en_GB' will already go to the default branch if they're not
listed explicitly.

Regards,
Nikita

--000000000000ad6df805d16701df--