mirrors/ladybird
0
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2025-08-08 01:00:05 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 9

LibJS: Remove the Intl.PluralRules GetOperands AO

Browse source 
This is an editorial change in the ECMA-402 spec. See:
c7bca3a
This commit is contained in:
Timothy Flynn 2024-12-04 13:52:05 -05:00 • committed by Andreas Kling
commit e5ec8441c9
Notes: github-actions[bot] 2024-12-05 08:50:54 +00:00
1 changed files with 9 additions and 10 deletions
Download patch file Download diff file Expand all files Collapse all files

19
Libraries/LibJS/Runtime/Intl/PluralRules.cpp
View file

@ -1,5 +1,5 @@
/* /*
* Copyright (c) 2022-2024, Tim Flynn <trflynn89@serenityos.org> * Copyright (c) 2022-2024, Tim Flynn <trflynn89@ladybird.org>
* *
* SPDX-License-Identifier: BSD-2-Clause * SPDX-License-Identifier: BSD-2-Clause
*/ */
@ -16,7 +16,7 @@ PluralRules::PluralRules(Object& prototype)
{ {
} }
// 16.5.4 ResolvePlural ( pluralRules, n ), https://tc39.es/ecma402/#sec-resolveplural // 16.5.2 ResolvePlural ( pluralRules, n ), https://tc39.es/ecma402/#sec-resolveplural
Unicode::PluralCategory resolve_plural(PluralRules const& plural_rules, Value number) Unicode::PluralCategory resolve_plural(PluralRules const& plural_rules, Value number)
{ {
// 1. If n is not a finite Number, then // 1. If n is not a finite Number, then
@ -26,17 +26,16 @@ Unicode::PluralCategory resolve_plural(PluralRules const& plural_rules, Value nu
return Unicode::PluralCategory::Other; return Unicode::PluralCategory::Other;
} }
// 2. Let locale be pluralRules.[[Locale]]. // 2. Let res be FormatNumericToString(pluralRules, ℝ(n)).
// 3. Let type be pluralRules.[[Type]]. // 3. Let s be res.[[FormattedString]].
// 4. Let res be FormatNumericToString(pluralRules, ℝ(n)). // 4. Let locale be pluralRules.[[Locale]].
// 5. Let s be res.[[FormattedString]]. // 5. Let type be pluralRules.[[Type]].
// 6. Let operands be GetOperands(s). // 6. Let p be PluralRuleSelect(locale, type, s).
// 7. Let p be PluralRuleSelect(locale, type, n, operands). // 7. Return the Record { [[PluralCategory]]: p, [[FormattedString]]: s }.
// 8. Return the Record { [[PluralCategory]]: p, [[FormattedString]]: s }.
return plural_rules.formatter().select_plural(number.as_double()); return plural_rules.formatter().select_plural(number.as_double());
} }
// 16.5.6 ResolvePluralRange ( pluralRules, x, y ), https://tc39.es/ecma402/#sec-resolveplural // 16.5.4 ResolvePluralRange ( pluralRules, x, y ), https://tc39.es/ecma402/#sec-resolveplural
ThrowCompletionOr<Unicode::PluralCategory> resolve_plural_range(VM& vm, PluralRules const& plural_rules, Value start, Value end) ThrowCompletionOr<Unicode::PluralCategory> resolve_plural_range(VM& vm, PluralRules const& plural_rules, Value start, Value end)
{ {
// 1. If x is NaN or y is NaN, throw a RangeError exception. // 1. If x is NaN or y is NaN, throw a RangeError exception.