Skip to content

Commit

Permalink
Bug 1930464: Only use currency minor digits for standard notation. r=…
Browse files Browse the repository at this point in the history
…spidermonkey-reviewers,dminor

Implement the changes from <tc39/ecma402#925>.

Test coverage: <tc39/test262#4274>

Differential Revision: https://phabricator.services.mozilla.com/D228589
  • Loading branch information
anba committed Nov 26, 2024
1 parent 1d6ec02 commit 502d37f
Show file tree
Hide file tree
Showing 2 changed files with 65 additions and 68 deletions.
4 changes: 2 additions & 2 deletions js/src/builtin/intl/NumberFormat.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -125,7 +125,7 @@ const ClassSpec NumberFormatObject::classSpec_ = {
/**
* 15.1.1 Intl.NumberFormat ( [ locales [ , options ] ] )
*
* ES2024 Intl draft rev 74ca7099f103d143431b2ea422ae640c6f43e3e6
* ES2025 Intl draft rev 5ea95f8a98d660e94c177d6f5e88c6d2962123b1
*/
static bool NumberFormat(JSContext* cx, const CallArgs& args, bool construct) {
AutoJSConstructorProfilerEntry pseudoFrame(cx, "Intl.NumberFormat");
Expand All @@ -150,7 +150,7 @@ static bool NumberFormat(JSContext* cx, const CallArgs& args, bool construct) {
HandleValue locales = args.get(0);
HandleValue options = args.get(1);

// Step 3.
// Steps 3-33.
return intl::InitializeNumberFormatObject(cx, numberFormat, thisValue,
locales, options, args.rval());
}
Expand Down
Loading

0 comments on commit 502d37f

Please sign in to comment.