Skip to content

Various fixes #107

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
ryzokuken merged 26 commits into from
Aug 3, 2022
Merged

Various fixes #107

Changes from 1 commit
Commits
Show all changes
26 commits
Select commit Hold shift + click to select a range
43b1115
Add missing check that 'numberingSystem' is not undefined
anba Jul 25, 2022
913a65c
GetDurationUnitOptions requires that prevStyle is a String value
anba Jul 25, 2022
c7b5ac8
Remove unnecessary initialisation for value and pass value as a Numbe…
anba Jul 25, 2022
ce0a0d3
Creating internal Intl objects is infallible
anba Jul 25, 2022
03cff7c
Add missing Oxford comma
anba Jul 25, 2022
6e7153d
Use correct NumberFormat variable
anba Jul 25, 2022
8f995d2
Correctly call MakePartsList with a List of Records
anba Jul 25, 2022
fdc27d6
Use computed slot names
anba Jul 25, 2022
7968d7c
Add a note that CreatePartsFromList will need to be changed
anba Jul 25, 2022
2318b2f
Follow PartitionDateTimeRangePattern instead of using 'Stop iteration'
anba Jul 25, 2022
1d54ece
Invert the condition instead of using 'Skip to the next iteration'
anba Jul 25, 2022
1297f00
Add missing definition for 'display' variable
anba Jul 25, 2022
6692e97
Don't append [[Unit]] field to the result list
anba Jul 25, 2022
e1a7fec
Make ToDurationRecord more similar to ToTemporalDurationRecord/ToTemp…
anba Jul 25, 2022
0c95440
Rename DurationSign to DurationRecordSign to avoid clashing with Temp…
anba Jul 25, 2022
cf89536
Use 'of' instead of 'in' to match ECMA-402
anba Jul 25, 2022
2fb3173
Change the operations from Temporal to be more like the rest of ECMA-402
anba Jul 25, 2022
7d0c057
Enumerate Record fields for iteration to match ECMA-402
anba Jul 25, 2022
a8681d3
Remove `next row` which is undefined for the last row
anba Jul 25, 2022
5789f68
Only append time separator when the next value will be displayed
anba Jul 25, 2022
1f87460
Add a TODO note about incorrect handling of the time separator symbol
anba Jul 25, 2022
8eb0100
Use "numeric" as the default style when the previous style is "numeri…
anba Jul 25, 2022
dfa1ce7
Use correct markup for string literals
anba Jul 25, 2022
a90d077
Use correct markup for null and number values
anba Jul 25, 2022
7fe6e79
Use correct markup for throwing a RangeError
anba Jul 25, 2022
8f0890b
Add missing dots
anba Jul 25, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion spec.emu
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -350,7 +350,7 @@ contributors: Ujjwal Sharma, Younies Mahmoud
1. Let _matcher_ be ? GetOption(_options_, *"localeMatcher"*, *"string"*, « *"lookup"*, *"best fit"* », *"best fit"*).
1. Let _numberingSystem_ be ? GetOption(_options_, *"numberingSystem"*, *"string"*, *undefined*, *undefined*).
1. If _numberingSystem_ is not *undefined, then
1. If _numberingSystem_ does not match the Unicode Locale Identifier `type` nonterminal, throw a `RangeError` exception.
1. If _numberingSystem_ does not match the Unicode Locale Identifier `type` nonterminal, throw a *RangeError* exception.
Copy link
Member

@ryzokuken ryzokuken Jul 26, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Regarding this and the previous two commits, I believe I switched from asterisks to backticks on @rbuckton's advice. That said, I'd be happy to do either as long as we have a clear contender.

Copy link
Contributor Author

@anba anba Jul 27, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The style changed at some point → tc39/ecma402#396, which links to other discussions and the corresponding PR for ECMA-262.

1. Let _opt_ be the Record { [[localeMatcher]]: _matcher_, [[nu]]: _numberingSystem_ }.
1. Let _r_ be ResolveLocale(%DurationFormat%.[[AvailableLocales]], _requestedLocales_, _opt_, %DurationFormat%.[[RelevantExtensionKeys]], %DurationFormat%.[[LocaleData]]).
1. Let _locale_ be r.[[locale]].
Expand Down