summaryrefslogtreecommitdiff
path: root/Userland/Libraries/LibJS
diff options
context:
space:
mode:
authorLinus Groh <mail@linusgroh.de>2022-05-06 20:45:25 +0200
committerLinus Groh <mail@linusgroh.de>2022-05-06 22:32:47 +0200
commitf7c9bd0760d21e1c32d59cb62fe36728736b47bd (patch)
tree6efb90cf7551e6b22970c86effef9ed8bbe65325 /Userland/Libraries/LibJS
parentb9b3d01bea2b8e5455f17eedf1acd6ac9ffe31ec (diff)
downloadserenity-f7c9bd0760d21e1c32d59cb62fe36728736b47bd.zip
LibJS: Convert remaining Date AOs using JS::Value as in/output to double
There was an awful lot of JS::Value <-> double conversion going on, even through these AOs only work with number values anyway. They don't need a global object either as they won't allocate or throw, that was simply to pass it to infallible calls of ToIntegerOrInfinity.
Diffstat (limited to 'Userland/Libraries/LibJS')
-rw-r--r--Userland/Libraries/LibJS/Runtime/Date.cpp60
-rw-r--r--Userland/Libraries/LibJS/Runtime/Date.h8
-rw-r--r--Userland/Libraries/LibJS/Runtime/DateConstructor.cpp76
-rw-r--r--Userland/Libraries/LibJS/Runtime/DatePrototype.cpp479
-rw-r--r--Userland/Libraries/LibJS/Runtime/DatePrototype.h4
-rw-r--r--Userland/Libraries/LibJS/Runtime/Intl/DateTimeFormat.cpp34
-rw-r--r--Userland/Libraries/LibJS/Runtime/Intl/DateTimeFormat.h14
-rw-r--r--Userland/Libraries/LibJS/Runtime/Intl/DateTimeFormatFunction.cpp8
-rw-r--r--Userland/Libraries/LibJS/Runtime/Intl/DateTimeFormatPrototype.cpp20
-rw-r--r--Userland/Libraries/LibJS/Runtime/Temporal/PlainDateTime.cpp8
10 files changed, 357 insertions, 354 deletions
diff --git a/Userland/Libraries/LibJS/Runtime/Date.cpp b/Userland/Libraries/LibJS/Runtime/Date.cpp
index 078036b2d2..9d68e369b1 100644
--- a/Userland/Libraries/LibJS/Runtime/Date.cpp
+++ b/Userland/Libraries/LibJS/Runtime/Date.cpp
@@ -300,25 +300,25 @@ double utc_time(double time)
}
// 21.4.1.11 MakeTime ( hour, min, sec, ms ), https://tc39.es/ecma262/#sec-maketime
-Value make_time(GlobalObject& global_object, Value hour, Value min, Value sec, Value ms)
+double make_time(double hour, double min, double sec, double ms)
{
// 1. If hour is not finite or min is not finite or sec is not finite or ms is not finite, return NaN.
- if (!hour.is_finite_number() || !min.is_finite_number() || !sec.is_finite_number() || !ms.is_finite_number())
- return js_nan();
+ if (!isfinite(hour) || !isfinite(min) || !isfinite(sec) || !isfinite(ms))
+ return NAN;
// 2. Let h be ๐”ฝ(! ToIntegerOrInfinity(hour)).
- auto h = MUST(hour.to_integer_or_infinity(global_object));
+ auto h = to_integer_or_infinity(hour);
// 3. Let m be ๐”ฝ(! ToIntegerOrInfinity(min)).
- auto m = MUST(min.to_integer_or_infinity(global_object));
+ auto m = to_integer_or_infinity(min);
// 4. Let s be ๐”ฝ(! ToIntegerOrInfinity(sec)).
- auto s = MUST(sec.to_integer_or_infinity(global_object));
+ auto s = to_integer_or_infinity(sec);
// 5. Let milli be ๐”ฝ(! ToIntegerOrInfinity(ms)).
- auto milli = MUST(ms.to_integer_or_infinity(global_object));
+ auto milli = to_integer_or_infinity(ms);
// 6. Let t be ((h * msPerHour + m * msPerMinute) + s * msPerSecond) + milli, performing the arithmetic according to IEEE 754-2019 rules (that is, as if using the ECMAScript operators * and +).
// NOTE: C++ arithmetic abides by IEEE 754 rules
auto t = ((h * ms_per_hour + m * ms_per_minute) + s * ms_per_second) + milli;
// 7. Return t.
- return Value(t);
+ return t;
}
// Day(t), https://tc39.es/ecma262/#eqn-Day
@@ -335,69 +335,69 @@ double time_within_day(double time)
}
// 21.4.1.12 MakeDay ( year, month, date ), https://tc39.es/ecma262/#sec-makeday
-Value make_day(GlobalObject& global_object, Value year, Value month, Value date)
+double make_day(double year, double month, double date)
{
// 1. If year is not finite or month is not finite or date is not finite, return NaN.
- if (!year.is_finite_number() || !month.is_finite_number() || !date.is_finite_number())
- return js_nan();
+ if (!isfinite(year) || !isfinite(month) || !isfinite(date))
+ return NAN;
// 2. Let y be ๐”ฝ(! ToIntegerOrInfinity(year)).
- auto y = MUST(year.to_integer_or_infinity(global_object));
+ auto y = to_integer_or_infinity(year);
// 3. Let m be ๐”ฝ(! ToIntegerOrInfinity(month)).
- auto m = MUST(month.to_integer_or_infinity(global_object));
+ auto m = to_integer_or_infinity(month);
// 4. Let dt be ๐”ฝ(! ToIntegerOrInfinity(date)).
- auto dt = MUST(date.to_integer_or_infinity(global_object));
+ auto dt = to_integer_or_infinity(date);
// 5. Let ym be y + ๐”ฝ(floor(โ„(m) / 12)).
auto ym = y + floor(m / 12);
// 6. If ym is not finite, return NaN.
- if (!Value(ym).is_finite_number())
- return js_nan();
+ if (!isfinite(ym))
+ return NAN;
// 7. Let mn be ๐”ฝ(โ„(m) modulo 12).
auto mn = modulo(m, 12);
// 8. Find a finite time value t such that YearFromTime(t) is ym and MonthFromTime(t) is mn and DateFromTime(t) is 1๐”ฝ; but if this is not possible (because some argument is out of range), return NaN.
if (!AK::is_within_range<int>(ym) || !AK::is_within_range<int>(mn + 1))
- return js_nan();
+ return NAN;
// FIXME: We are avoiding AK::years_to_days_since_epoch here because it is implemented by looping over
// the range [1970, ym), which will spin for any time value with an extremely large year.
auto t = time_from_year(ym) + (day_of_year(static_cast<int>(ym), static_cast<int>(mn) + 1, 1) * ms_per_day);
// 9. Return Day(t) + dt - 1๐”ฝ.
- return Value(day(static_cast<double>(t)) + dt - 1);
+ return day(static_cast<double>(t)) + dt - 1;
}
// 21.4.1.13 MakeDate ( day, time ), https://tc39.es/ecma262/#sec-makedate
-Value make_date(Value day, Value time)
+double make_date(double day, double time)
{
// 1. If day is not finite or time is not finite, return NaN.
- if (!day.is_finite_number() || !time.is_finite_number())
- return js_nan();
+ if (!isfinite(day) || !isfinite(time))
+ return NAN;
// 2. Let tv be day ร— msPerDay + time.
- auto tv = Value(day.as_double() * ms_per_day + time.as_double());
+ auto tv = day * ms_per_day + time;
// 3. If tv is not finite, return NaN.
- if (!tv.is_finite_number())
- return js_nan();
+ if (!isfinite(tv))
+ return NAN;
// 4. Return tv.
return tv;
}
// 21.4.1.14 TimeClip ( time ), https://tc39.es/ecma262/#sec-timeclip
-Value time_clip(GlobalObject& global_object, Value time)
+double time_clip(double time)
{
// 1. If time is not finite, return NaN.
- if (!time.is_finite_number())
- return js_nan();
+ if (!isfinite(time))
+ return NAN;
// 2. If abs(โ„(time)) > 8.64 ร— 10^15, return NaN.
- if (fabs(time.as_double()) > 8.64E15)
- return js_nan();
+ if (fabs(time) > 8.64E15)
+ return NAN;
// 3. Return ๐”ฝ(! ToIntegerOrInfinity(time)).
- return Value(MUST(time.to_integer_or_infinity(global_object)));
+ return to_integer_or_infinity(time);
}
}
diff --git a/Userland/Libraries/LibJS/Runtime/Date.h b/Userland/Libraries/LibJS/Runtime/Date.h
index 25748c9030..6bcbf2d792 100644
--- a/Userland/Libraries/LibJS/Runtime/Date.h
+++ b/Userland/Libraries/LibJS/Runtime/Date.h
@@ -63,9 +63,9 @@ double local_time(double time);
double utc_time(double time);
double day(double);
double time_within_day(double);
-Value make_time(GlobalObject& global_object, Value hour, Value min, Value sec, Value ms);
-Value make_day(GlobalObject& global_object, Value year, Value month, Value date);
-Value make_date(Value day, Value time);
-Value time_clip(GlobalObject& global_object, Value time);
+double make_time(double hour, double min, double sec, double ms);
+double make_day(double year, double month, double date);
+double make_date(double day, double time);
+double time_clip(double time);
}
diff --git a/Userland/Libraries/LibJS/Runtime/DateConstructor.cpp b/Userland/Libraries/LibJS/Runtime/DateConstructor.cpp
index 9db88ebc88..f375e7b975 100644
--- a/Userland/Libraries/LibJS/Runtime/DateConstructor.cpp
+++ b/Userland/Libraries/LibJS/Runtime/DateConstructor.cpp
@@ -1,5 +1,5 @@
/*
- * Copyright (c) 2020, Linus Groh <linusg@serenityos.org>
+ * Copyright (c) 2020-2022, Linus Groh <linusg@serenityos.org>
* Copyright (c) 2020, Nico Weber <thakis@chromium.org>
* Copyright (c) 2021, Petrรณczi Zoltรกn <petroczizoltan@tutanota.com>
* Copyright (c) 2022, Tim Flynn <trflynn89@serenityos.org>
@@ -23,7 +23,7 @@
namespace JS {
// 21.4.3.2 Date.parse ( string ), https://tc39.es/ecma262/#sec-date.parse
-static Value parse_simplified_iso8601(GlobalObject& global_object, String const& iso_8601)
+static double parse_simplified_iso8601(String const& iso_8601)
{
// 21.4.1.15 Date Time String Format, https://tc39.es/ecma262/#sec-date-time-string-format
GenericLexer lexer(iso_8601);
@@ -126,7 +126,7 @@ static Value parse_simplified_iso8601(GlobalObject& global_object, String const&
auto lex_time = [&]() { return lex_hours_minutes(hours, minutes) && (!lexer.consume_specific(':') || lex_seconds_milliseconds()) && lex_timezone(); };
if (!lex_date() || (lexer.consume_specific('T') && !lex_time()) || !lexer.is_eof()) {
- return js_nan();
+ return NAN;
}
// We parsed a valid date simplified ISO 8601 string.
@@ -144,22 +144,22 @@ static Value parse_simplified_iso8601(GlobalObject& global_object, String const&
else if (timezone == '+')
time_ms -= *timezone_hours * 3'600'000 + *timezone_minutes * 60'000;
- return time_clip(global_object, Value(time_ms));
+ return time_clip(time_ms);
}
-static Value parse_date_string(GlobalObject& global_object, String const& date_string)
+static double parse_date_string(String const& date_string)
{
- auto value = parse_simplified_iso8601(global_object, date_string);
- if (value.is_finite_number())
+ auto value = parse_simplified_iso8601(date_string);
+ if (isfinite(value))
return value;
// Date.parse() is allowed to accept an arbitrary number of implementation-defined formats.
// Parse formats of this type: "Wed Apr 17 23:08:53 +0000 2019"
auto maybe_datetime = Core::DateTime::parse("%a %b %e %T %z %Y", date_string);
if (maybe_datetime.has_value())
- return Value(1000.0 * maybe_datetime.value().timestamp());
+ return 1000.0 * maybe_datetime->timestamp();
- return js_nan();
+ return NAN;
}
DateConstructor::DateConstructor(GlobalObject& global_object)
@@ -200,20 +200,20 @@ ThrowCompletionOr<Object*> DateConstructor::construct(FunctionObject& new_target
auto& vm = this->vm();
auto& global_object = this->global_object();
- Value date_value;
+ double date_value;
// 2. Let numberOfArgs be the number of elements in values.
// 3. If numberOfArgs = 0, then
if (vm.argument_count() == 0) {
// a. Let dv be the time value (UTC) identifying the current time.
auto now = AK::Time::now_realtime().to_milliseconds();
- date_value = Value(static_cast<double>(now));
+ date_value = static_cast<double>(now);
}
// 4. Else if numberOfArgs = 1, then
else if (vm.argument_count() == 1) {
// a. Let value be values[0].
auto value = vm.argument(0);
- Value time_value;
+ double time_value;
// b. If Type(value) is Object and value has a [[DateValue]] internal slot, then
if (value.is_object() && is<Date>(value.as_object())) {
@@ -229,28 +229,28 @@ ThrowCompletionOr<Object*> DateConstructor::construct(FunctionObject& new_target
if (primitive.is_string()) {
// 1. Assert: The next step never returns an abrupt completion because Type(v) is String.
// 2. Let tv be the result of parsing v as a date, in exactly the same manner as for the parse method (21.4.3.2).
- time_value = parse_date_string(global_object, primitive.as_string().string());
+ time_value = parse_date_string(primitive.as_string().string());
}
// iii. Else,
else {
// 1. Let tv be ? ToNumber(v).
- time_value = TRY(primitive.to_number(global_object));
+ time_value = TRY(primitive.to_number(global_object)).as_double();
}
}
// d. Let dv be TimeClip(tv).
- date_value = time_clip(global_object, time_value);
+ date_value = time_clip(time_value);
}
// 5. Else,
else {
// a. Assert: numberOfArgs โ‰ฅ 2.
// b. Let y be ? ToNumber(values[0]).
- auto year = TRY(vm.argument(0).to_number(global_object));
+ auto year = TRY(vm.argument(0).to_number(global_object)).as_double();
// c. Let m be ? ToNumber(values[1]).
- auto month = TRY(vm.argument(1).to_number(global_object));
+ auto month = TRY(vm.argument(1).to_number(global_object)).as_double();
- auto arg_or = [&vm, &global_object](size_t i, i32 fallback) -> ThrowCompletionOr<Value> {
- return vm.argument_count() > i ? vm.argument(i).to_number(global_object) : Value(fallback);
+ auto arg_or = [&vm, &global_object](size_t i, double fallback) -> ThrowCompletionOr<double> {
+ return vm.argument_count() > i ? TRY(vm.argument(i).to_number(global_object)).as_double() : fallback;
};
// d. If numberOfArgs > 2, let dt be ? ToNumber(values[2]); else let dt be 1๐”ฝ.
@@ -266,28 +266,28 @@ ThrowCompletionOr<Object*> DateConstructor::construct(FunctionObject& new_target
// i. If y is NaN, let yr be NaN.
// j. Else,
- if (!year.is_nan()) {
+ if (!isnan(year)) {
// i. Let yi be ! ToIntegerOrInfinity(y).
- auto year_double = MUST(year.to_integer_or_infinity(global_object));
+ auto year_integer = to_integer_or_infinity(year);
// ii. If 0 โ‰ค yi โ‰ค 99, let yr be 1900๐”ฝ + ๐”ฝ(yi); otherwise, let yr be y.
- if (0 <= year_double && year_double <= 99)
- year = Value(1900 + year_double);
+ if (0 <= year_integer && year_integer <= 99)
+ year = 1900 + year_integer;
}
// k. Let finalDate be MakeDate(MakeDay(yr, m, dt), MakeTime(h, min, s, milli)).
- auto day = make_day(global_object, year, month, date);
- auto time = make_time(global_object, hours, minutes, seconds, milliseconds);
+ auto day = make_day(year, month, date);
+ auto time = make_time(hours, minutes, seconds, milliseconds);
auto final_date = make_date(day, time);
// l. Let dv be TimeClip(UTC(finalDate)).
- date_value = time_clip(global_object, Value(utc_time(final_date.as_double())));
+ date_value = time_clip(utc_time(final_date));
}
// 6. Let O be ? OrdinaryCreateFromConstructor(NewTarget, "%Date.prototype%", ยซ [[DateValue]] ยป).
// 7. Set O.[[DateValue]] to dv.
// 8. Return O.
- return TRY(ordinary_create_from_constructor<Date>(global_object, new_target, &GlobalObject::date_prototype, date_value.as_double()));
+ return TRY(ordinary_create_from_constructor<Date>(global_object, new_target, &GlobalObject::date_prototype, date_value));
}
// 21.4.3.1 Date.now ( ), https://tc39.es/ecma262/#sec-date.now
@@ -306,18 +306,18 @@ JS_DEFINE_NATIVE_FUNCTION(DateConstructor::parse)
auto date_string = TRY(vm.argument(0).to_string(global_object));
- return parse_date_string(global_object, date_string);
+ return Value(parse_date_string(date_string));
}
// 21.4.3.4 Date.UTC ( year [ , month [ , date [ , hours [ , minutes [ , seconds [ , ms ] ] ] ] ] ] ), https://tc39.es/ecma262/#sec-date.utc
JS_DEFINE_NATIVE_FUNCTION(DateConstructor::utc)
{
- auto arg_or = [&vm, &global_object](size_t i, i32 fallback) -> ThrowCompletionOr<Value> {
- return vm.argument_count() > i ? vm.argument(i).to_number(global_object) : Value(fallback);
+ auto arg_or = [&vm, &global_object](size_t i, double fallback) -> ThrowCompletionOr<double> {
+ return vm.argument_count() > i ? TRY(vm.argument(i).to_number(global_object)).as_double() : fallback;
};
// 1. Let y be ? ToNumber(year).
- auto year = TRY(vm.argument(0).to_number(global_object));
+ auto year = TRY(vm.argument(0).to_number(global_object)).as_double();
// 2. If month is present, let m be ? ToNumber(month); else let m be +0๐”ฝ.
auto month = TRY(arg_or(1, 0));
// 3. If date is present, let dt be ? ToNumber(date); else let dt be 1๐”ฝ.
@@ -333,19 +333,19 @@ JS_DEFINE_NATIVE_FUNCTION(DateConstructor::utc)
// 8. If y is NaN, let yr be NaN.
// 9. Else,
- if (!year.is_nan()) {
+ if (!isnan(year)) {
// a. Let yi be ! ToIntegerOrInfinity(y).
- auto year_double = MUST(year.to_integer_or_infinity(global_object));
+ auto year_integer = to_integer_or_infinity(year);
// b. If 0 โ‰ค yi โ‰ค 99, let yr be 1900๐”ฝ + ๐”ฝ(yi); otherwise, let yr be y.
- if (0 <= year_double && year_double <= 99)
- year = Value(1900 + year_double);
+ if (0 <= year_integer && year_integer <= 99)
+ year = 1900 + year_integer;
}
// 10. Return TimeClip(MakeDate(MakeDay(yr, m, dt), MakeTime(h, min, s, milli))).
- auto day = make_day(global_object, year, month, date);
- auto time = make_time(global_object, hours, minutes, seconds, milliseconds);
- return time_clip(global_object, make_date(day, time));
+ auto day = make_day(year, month, date);
+ auto time = make_time(hours, minutes, seconds, milliseconds);
+ return Value(time_clip(make_date(day, time)));
}
}
diff --git a/Userland/Libraries/LibJS/Runtime/DatePrototype.cpp b/Userland/Libraries/LibJS/Runtime/DatePrototype.cpp
index b466c3aeca..23bab1fa9a 100644
--- a/Userland/Libraries/LibJS/Runtime/DatePrototype.cpp
+++ b/Userland/Libraries/LibJS/Runtime/DatePrototype.cpp
@@ -99,12 +99,12 @@ void DatePrototype::initialize(GlobalObject& global_object)
}
// thisTimeValue ( value ), https://tc39.es/ecma262/#thistimevalue
-ThrowCompletionOr<Value> this_time_value(GlobalObject& global_object, Value value)
+ThrowCompletionOr<double> this_time_value(GlobalObject& global_object, Value value)
{
// 1. If Type(value) is Object and value has a [[DateValue]] internal slot, then
if (value.is_object() && is<Date>(value.as_object())) {
// a. Return value.[[DateValue]].
- return Value(static_cast<Date&>(value.as_object()).date_value());
+ return static_cast<Date&>(value.as_object()).date_value();
}
// 2. Throw a TypeError exception.
@@ -119,11 +119,11 @@ JS_DEFINE_NATIVE_FUNCTION(DatePrototype::get_date)
auto time = TRY(this_time_value(global_object, vm.this_value(global_object)));
// 2. If t is NaN, return NaN.
- if (time.is_nan())
- return time;
+ if (isnan(time))
+ return js_nan();
// 3. Return DateFromTime(LocalTime(t)).
- return Value(date_from_time(local_time(time.as_double())));
+ return Value(date_from_time(local_time(time)));
}
// 21.4.4.3 Date.prototype.getDay ( ), https://tc39.es/ecma262/#sec-date.prototype.getday
@@ -133,11 +133,11 @@ JS_DEFINE_NATIVE_FUNCTION(DatePrototype::get_day)
auto time = TRY(this_time_value(global_object, vm.this_value(global_object)));
// 2. If t is NaN, return NaN.
- if (time.is_nan())
- return time;
+ if (isnan(time))
+ return js_nan();
// 3. Return WeekDay(LocalTime(t)).
- return Value(week_day(local_time(time.as_double())));
+ return Value(week_day(local_time(time)));
}
// 21.4.4.4 Date.prototype.getFullYear ( ), https://tc39.es/ecma262/#sec-date.prototype.getfullyear
@@ -147,11 +147,11 @@ JS_DEFINE_NATIVE_FUNCTION(DatePrototype::get_full_year)
auto time = TRY(this_time_value(global_object, vm.this_value(global_object)));
// 2. If t is NaN, return NaN.
- if (time.is_nan())
- return time;
+ if (isnan(time))
+ return js_nan();
// 3. Return YearFromTime(LocalTime(t)).
- return Value(year_from_time(local_time(time.as_double())));
+ return Value(year_from_time(local_time(time)));
}
// 21.4.4.5 Date.prototype.getHours ( ), https://tc39.es/ecma262/#sec-date.prototype.gethours
@@ -161,11 +161,11 @@ JS_DEFINE_NATIVE_FUNCTION(DatePrototype::get_hours)
auto time = TRY(this_time_value(global_object, vm.this_value(global_object)));
// 2. If t is NaN, return NaN.
- if (time.is_nan())
- return time;
+ if (isnan(time))
+ return js_nan();
// 3. Return HourFromTime(LocalTime(t)).
- return Value(hour_from_time(local_time(time.as_double())));
+ return Value(hour_from_time(local_time(time)));
}
// 21.4.4.6 Date.prototype.getMilliseconds ( ), https://tc39.es/ecma262/#sec-date.prototype.getmilliseconds
@@ -175,11 +175,11 @@ JS_DEFINE_NATIVE_FUNCTION(DatePrototype::get_milliseconds)
auto time = TRY(this_time_value(global_object, vm.this_value(global_object)));
// 2. If t is NaN, return NaN.
- if (time.is_nan())
- return time;
+ if (isnan(time))
+ return js_nan();
// 3. Return msFromTime(LocalTime(t)).
- return Value(ms_from_time(local_time(time.as_double())));
+ return Value(ms_from_time(local_time(time)));
}
// 21.4.4.7 Date.prototype.getMinutes ( ), https://tc39.es/ecma262/#sec-date.prototype.getminutes
@@ -189,11 +189,11 @@ JS_DEFINE_NATIVE_FUNCTION(DatePrototype::get_minutes)
auto time = TRY(this_time_value(global_object, vm.this_value(global_object)));
// 2. If t is NaN, return NaN.
- if (time.is_nan())
- return time;
+ if (isnan(time))
+ return js_nan();
// 3. Return MinFromTime(LocalTime(t)).
- return Value(min_from_time(local_time(time.as_double())));
+ return Value(min_from_time(local_time(time)));
}
// 21.4.4.8 Date.prototype.getMonth ( ), https://tc39.es/ecma262/#sec-date.prototype.getmonth
@@ -203,11 +203,11 @@ JS_DEFINE_NATIVE_FUNCTION(DatePrototype::get_month)
auto time = TRY(this_time_value(global_object, vm.this_value(global_object)));
// 2. If t is NaN, return NaN.
- if (time.is_nan())
- return time;
+ if (isnan(time))
+ return js_nan();
// 3. Return MonthFromTime(LocalTime(t)).
- return Value(month_from_time(local_time(time.as_double())));
+ return Value(month_from_time(local_time(time)));
}
// 21.4.4.9 Date.prototype.getSeconds ( ), https://tc39.es/ecma262/#sec-date.prototype.getseconds
@@ -217,18 +217,18 @@ JS_DEFINE_NATIVE_FUNCTION(DatePrototype::get_seconds)
auto time = TRY(this_time_value(global_object, vm.this_value(global_object)));
// 2. If t is NaN, return NaN.
- if (time.is_nan())
- return time;
+ if (isnan(time))
+ return js_nan();
// 3. Return SecFromTime(LocalTime(t)).
- return Value(sec_from_time(local_time(time.as_double())));
+ return Value(sec_from_time(local_time(time)));
}
// 21.4.4.10 Date.prototype.getTime ( ), https://tc39.es/ecma262/#sec-date.prototype.gettime
JS_DEFINE_NATIVE_FUNCTION(DatePrototype::get_time)
{
// 1. Return ? thisTimeValue(this value).
- return this_time_value(global_object, vm.this_value(global_object));
+ return Value(TRY(this_time_value(global_object, vm.this_value(global_object))));
}
// 21.4.4.11 Date.prototype.getTimezoneOffset ( ), https://tc39.es/ecma262/#sec-date.prototype.gettimezoneoffset
@@ -238,11 +238,11 @@ JS_DEFINE_NATIVE_FUNCTION(DatePrototype::get_timezone_offset)
auto time = TRY(this_time_value(global_object, vm.this_value(global_object)));
// 2. If t is NaN, return NaN.
- if (time.is_nan())
+ if (isnan(time))
return js_nan();
// 3. Return (t - LocalTime(t)) / msPerMinute.
- return Value((time.as_double() - local_time(time.as_double())) / ms_per_minute);
+ return Value((time - local_time(time)) / ms_per_minute);
}
// 21.4.4.12 Date.prototype.getUTCDate ( ), https://tc39.es/ecma262/#sec-date.prototype.getutcdate
@@ -252,11 +252,11 @@ JS_DEFINE_NATIVE_FUNCTION(DatePrototype::get_utc_date)
auto time = TRY(this_time_value(global_object, vm.this_value(global_object)));
// 2. If t is NaN, return NaN.
- if (time.is_nan())
- return time;
+ if (isnan(time))
+ return js_nan();
// 3. Return DateFromTime(t).
- return Value(date_from_time(time.as_double()));
+ return Value(date_from_time(time));
}
// 21.4.4.13 Date.prototype.getUTCDay ( ), https://tc39.es/ecma262/#sec-date.prototype.getutcday
@@ -266,11 +266,11 @@ JS_DEFINE_NATIVE_FUNCTION(DatePrototype::get_utc_day)
auto time = TRY(this_time_value(global_object, vm.this_value(global_object)));
// 2. If t is NaN, return NaN.
- if (time.is_nan())
- return time;
+ if (isnan(time))
+ return js_nan();
// 3. Return WeekDay(t).
- return Value(week_day(time.as_double()));
+ return Value(week_day(time));
}
// 21.4.4.14 Date.prototype.getUTCFullYear ( ), https://tc39.es/ecma262/#sec-date.prototype.getutcfullyear
@@ -280,11 +280,11 @@ JS_DEFINE_NATIVE_FUNCTION(DatePrototype::get_utc_full_year)
auto time = TRY(this_time_value(global_object, vm.this_value(global_object)));
// 2. If t is NaN, return NaN.
- if (time.is_nan())
- return time;
+ if (isnan(time))
+ return js_nan();
// 3. Return YearFromTime(t).
- return Value(year_from_time(time.as_double()));
+ return Value(year_from_time(time));
}
// 21.4.4.15 Date.prototype.getUTCHours ( ), https://tc39.es/ecma262/#sec-date.prototype.getutchours
@@ -294,11 +294,11 @@ JS_DEFINE_NATIVE_FUNCTION(DatePrototype::get_utc_hours)
auto time = TRY(this_time_value(global_object, vm.this_value(global_object)));
// 2. If t is NaN, return NaN.
- if (time.is_nan())
- return time;
+ if (isnan(time))
+ return js_nan();
// 3. Return HourFromTime(t).
- return Value(hour_from_time(time.as_double()));
+ return Value(hour_from_time(time));
}
// 21.4.4.16 Date.prototype.getUTCMilliseconds ( ), https://tc39.es/ecma262/#sec-date.prototype.getutcmilliseconds
@@ -308,11 +308,11 @@ JS_DEFINE_NATIVE_FUNCTION(DatePrototype::get_utc_milliseconds)
auto time = TRY(this_time_value(global_object, vm.this_value(global_object)));
// 2. If t is NaN, return NaN.
- if (time.is_nan())
- return time;
+ if (isnan(time))
+ return js_nan();
// 3. Return msFromTime(t).
- return Value(ms_from_time(time.as_double()));
+ return Value(ms_from_time(time));
}
// 21.4.4.17 Date.prototype.getUTCMinutes ( ), https://tc39.es/ecma262/#sec-date.prototype.getutcminutes
@@ -322,11 +322,11 @@ JS_DEFINE_NATIVE_FUNCTION(DatePrototype::get_utc_minutes)
auto time = TRY(this_time_value(global_object, vm.this_value(global_object)));
// 2. If t is NaN, return NaN.
- if (time.is_nan())
- return time;
+ if (isnan(time))
+ return js_nan();
// 3. Return MinFromTime(t).
- return Value(min_from_time(time.as_double()));
+ return Value(min_from_time(time));
}
// 21.4.4.18 Date.prototype.getUTCMonth ( ), https://tc39.es/ecma262/#sec-date.prototype.getutcmonth
@@ -336,11 +336,11 @@ JS_DEFINE_NATIVE_FUNCTION(DatePrototype::get_utc_month)
auto time = TRY(this_time_value(global_object, vm.this_value(global_object)));
// 2. If t is NaN, return NaN.
- if (time.is_nan())
- return time;
+ if (isnan(time))
+ return js_nan();
// 3. Return MonthFromTime(t).
- return Value(month_from_time(time.as_double()));
+ return month_from_time(time);
}
// 21.4.4.19 Date.prototype.getUTCSeconds ( ), https://tc39.es/ecma262/#sec-date.prototype.getutcseconds
@@ -350,32 +350,31 @@ JS_DEFINE_NATIVE_FUNCTION(DatePrototype::get_utc_seconds)
auto time = TRY(this_time_value(global_object, vm.this_value(global_object)));
// 2. If t is NaN, return NaN.
- if (time.is_nan())
- return time;
+ if (isnan(time))
+ return js_nan();
// 3. Return SecFromTime(t).
- return Value(sec_from_time(time.as_double()));
+ return Value(sec_from_time(time));
}
-template<typename T>
-static ThrowCompletionOr<Value> argument_or_value(GlobalObject& global_object, size_t index, T fallback)
+static ThrowCompletionOr<double> argument_or_number(GlobalObject& global_object, size_t index, double fallback)
{
auto& vm = global_object.vm();
if (vm.argument_count() > index)
- return vm.argument(index).to_number(global_object);
+ return TRY(vm.argument(index).to_number(global_object)).as_double();
- return Value(fallback);
+ return fallback;
}
-static ThrowCompletionOr<Optional<Value>> argument_or_empty(GlobalObject& global_object, size_t index)
+static ThrowCompletionOr<Optional<double>> argument_or_empty(GlobalObject& global_object, size_t index)
{
auto& vm = global_object.vm();
if (vm.argument_count() > index)
- return TRY(vm.argument(index).to_number(global_object));
+ return TRY(vm.argument(index).to_number(global_object)).as_double();
- return Optional<Value> {};
+ return Optional<double> {};
}
// 21.4.4.20 Date.prototype.setDate ( date ), https://tc39.es/ecma262/#sec-date.prototype.setdate
@@ -385,31 +384,31 @@ JS_DEFINE_NATIVE_FUNCTION(DatePrototype::set_date)
auto this_time = TRY(this_time_value(global_object, vm.this_value(global_object)));
// 2. Let dt be ? ToNumber(date).
- auto date = TRY(vm.argument(0).to_number(global_object));
+ auto date = TRY(vm.argument(0).to_number(global_object)).as_double();
// 3. If t is NaN, return NaN.
- if (this_time.is_nan())
+ if (isnan(this_time))
return js_nan();
// 4. Set t to LocalTime(t).
- auto time = local_time(this_time.as_double());
+ auto time = local_time(this_time);
// 5. Let newDate be MakeDate(MakeDay(YearFromTime(t), MonthFromTime(t), dt), TimeWithinDay(t)).
- auto year = Value(year_from_time(time));
- auto month = Value(month_from_time(time));
+ auto year = year_from_time(time);
+ auto month = month_from_time(time);
- auto day = make_day(global_object, year, month, date);
- auto new_date = make_date(day, Value(time_within_day(time)));
+ auto day = make_day(year, month, date);
+ auto new_date = make_date(day, time_within_day(time));
// 6. Let u be TimeClip(UTC(newDate)).
- new_date = time_clip(global_object, Value(utc_time(new_date.as_double())));
+ new_date = time_clip(utc_time(new_date));
// 7. Set the [[DateValue]] internal slot of this Date object to u.
auto* this_object = MUST(typed_this_object(global_object));
- this_object->set_date_value(new_date.as_double());
+ this_object->set_date_value(new_date);
// 8. Return u.
- return new_date;
+ return Value(new_date);
}
// 21.4.4.21 Date.prototype.setFullYear ( year [ , month [ , date ] ] ), https://tc39.es/ecma262/#sec-date.prototype.setfullyear
@@ -419,32 +418,32 @@ JS_DEFINE_NATIVE_FUNCTION(DatePrototype::set_full_year)
auto this_time = TRY(this_time_value(global_object, vm.this_value(global_object)));
// 2. Let y be ? ToNumber(year).
- auto year = TRY(vm.argument(0).to_number(global_object));
+ auto year = TRY(vm.argument(0).to_number(global_object)).as_double();
// 3. If t is NaN, set t to +0๐”ฝ; otherwise, set t to LocalTime(t).
double time = 0;
- if (!this_time.is_nan())
- time = local_time(this_time.as_double());
+ if (!isnan(this_time))
+ time = local_time(this_time);
// 4. If month is not present, let m be MonthFromTime(t); otherwise, let m be ? ToNumber(month).
- auto month = TRY(argument_or_value(global_object, 1, month_from_time(time)));
+ auto month = TRY(argument_or_number(global_object, 1, month_from_time(time)));
// 5. If date is not present, let dt be DateFromTime(t); otherwise, let dt be ? ToNumber(date).
- auto date = TRY(argument_or_value(global_object, 2, date_from_time(time)));
+ auto date = TRY(argument_or_number(global_object, 2, date_from_time(time)));
// 6. Let newDate be MakeDate(MakeDay(y, m, dt), TimeWithinDay(t)).
- auto day = make_day(global_object, year, month, date);
- auto new_date = make_date(day, Value(time_within_day(time)));
+ auto day = make_day(year, month, date);
+ auto new_date = make_date(day, time_within_day(time));
// 7. Let u be TimeClip(UTC(newDate)).
- new_date = time_clip(global_object, Value(utc_time(new_date.as_double())));
+ new_date = time_clip(utc_time(new_date));
// 8. Set the [[DateValue]] internal slot of this Date object to u.
auto* this_object = MUST(typed_this_object(global_object));
- this_object->set_date_value(new_date.as_double());
+ this_object->set_date_value(new_date);
// 9. Return u.
- return new_date;
+ return Value(new_date);
}
// 21.4.4.22 Date.prototype.setHours ( hour [ , min [ , sec [ , ms ] ] ] ), https://tc39.es/ecma262/#sec-date.prototype.sethours
@@ -454,7 +453,7 @@ JS_DEFINE_NATIVE_FUNCTION(DatePrototype::set_hours)
auto this_time = TRY(this_time_value(global_object, vm.this_value(global_object)));
// 2. Let h be ? ToNumber(hour).
- auto hour = TRY(vm.argument(0).to_number(global_object));
+ auto hour = TRY(vm.argument(0).to_number(global_object)).as_double();
// 3. If min is present, let m be ? ToNumber(min).
auto minute = TRY(argument_or_empty(global_object, 1));
@@ -466,37 +465,37 @@ JS_DEFINE_NATIVE_FUNCTION(DatePrototype::set_hours)
auto millisecond = TRY(argument_or_empty(global_object, 3));
// 6. If t is NaN, return NaN.
- if (this_time.is_nan())
+ if (isnan(this_time))
return js_nan();
// 7. Set t to LocalTime(t).
- auto time = local_time(this_time.as_double());
+ auto time = local_time(this_time);
// 8. If min is not present, let m be MinFromTime(t).
if (!minute.has_value())
- minute = Value(min_from_time(time));
+ minute = min_from_time(time);
// 9. If sec is not present, let s be SecFromTime(t).
if (!second.has_value())
- second = Value(sec_from_time(time));
+ second = sec_from_time(time);
// 10. If ms is not present, let milli be msFromTime(t).
if (!millisecond.has_value())
- millisecond = Value(ms_from_time(time));
+ millisecond = ms_from_time(time);
// 11. Let date be MakeDate(Day(t), MakeTime(h, m, s, milli)).
- auto new_time = make_time(global_object, hour, *minute, *second, *millisecond);
- auto date = make_date(Value(day(time)), new_time);
+ auto new_time = make_time(hour, *minute, *second, *millisecond);
+ auto date = make_date(day(time), new_time);
// 12. Let u be TimeClip(UTC(date)).
- date = time_clip(global_object, Value(utc_time(date.as_double())));
+ date = time_clip(utc_time(date));
// 13. Set the [[DateValue]] internal slot of this Date object to u.
auto* this_object = MUST(typed_this_object(global_object));
- this_object->set_date_value(date.as_double());
+ this_object->set_date_value(date);
// 14. Return u.
- return date;
+ return Value(date);
}
// 21.4.4.23 Date.prototype.setMilliseconds ( ms ), https://tc39.es/ecma262/#sec-date.prototype.setmilliseconds
@@ -506,32 +505,32 @@ JS_DEFINE_NATIVE_FUNCTION(DatePrototype::set_milliseconds)
auto this_time = TRY(this_time_value(global_object, vm.this_value(global_object)));
// 2. Set ms to ? ToNumber(ms).
- auto millisecond = TRY(vm.argument(0).to_number(global_object));
+ auto millisecond = TRY(vm.argument(0).to_number(global_object)).as_double();
// 3. If t is NaN, return NaN.
- if (this_time.is_nan())
+ if (isnan(this_time))
return js_nan();
// 4. Set t to LocalTime(t).
- auto time = local_time(this_time.as_double());
+ auto time = local_time(this_time);
// 5. Let time be MakeTime(HourFromTime(t), MinFromTime(t), SecFromTime(t), ms).
- auto hour = Value(hour_from_time(time));
- auto minute = Value(min_from_time(time));
- auto second = Value(sec_from_time(time));
+ auto hour = hour_from_time(time);
+ auto minute = min_from_time(time);
+ auto second = sec_from_time(time);
- auto new_time = make_time(global_object, hour, minute, second, millisecond);
+ auto new_time = make_time(hour, minute, second, millisecond);
// 6. Let u be TimeClip(UTC(MakeDate(Day(t), time))).
- auto date = make_date(Value(day(time)), new_time);
- date = time_clip(global_object, Value(utc_time(date.as_double())));
+ auto date = make_date(day(time), new_time);
+ date = time_clip(utc_time(date));
// 7. Set the [[DateValue]] internal slot of this Date object to u.
auto* this_object = MUST(typed_this_object(global_object));
- this_object->set_date_value(date.as_double());
+ this_object->set_date_value(date);
// 8. Return u.
- return date;
+ return Value(date);
}
// 21.4.4.24 Date.prototype.setMinutes ( min [ , sec [ , ms ] ] ), https://tc39.es/ecma262/#sec-date.prototype.setminutes
@@ -541,7 +540,7 @@ JS_DEFINE_NATIVE_FUNCTION(DatePrototype::set_minutes)
auto this_time = TRY(this_time_value(global_object, vm.this_value(global_object)));
// 2. Let m be ? ToNumber(min).
- auto minute = TRY(vm.argument(0).to_number(global_object));
+ auto minute = TRY(vm.argument(0).to_number(global_object)).as_double();
// 3. If sec is present, let s be ? ToNumber(sec).
auto second = TRY(argument_or_empty(global_object, 1));
@@ -550,35 +549,35 @@ JS_DEFINE_NATIVE_FUNCTION(DatePrototype::set_minutes)
auto millisecond = TRY(argument_or_empty(global_object, 2));
// 5. If t is NaN, return NaN.
- if (this_time.is_nan())
+ if (isnan(this_time))
return js_nan();
// 6. Set t to LocalTime(t).
- auto time = local_time(this_time.as_double());
+ auto time = local_time(this_time);
// 7. If sec is not present, let s be SecFromTime(t).
if (!second.has_value())
- second = Value(sec_from_time(time));
+ second = sec_from_time(time);
// 8. If ms is not present, let milli be msFromTime(t).
if (!millisecond.has_value())
- millisecond = Value(ms_from_time(time));
+ millisecond = ms_from_time(time);
// 9. Let date be MakeDate(Day(t), MakeTime(HourFromTime(t), m, s, milli)).
- auto hour = Value(hour_from_time(time));
+ auto hour = hour_from_time(time);
- auto new_time = make_time(global_object, hour, minute, *second, *millisecond);
- auto date = make_date(Value(day(time)), new_time);
+ auto new_time = make_time(hour, minute, *second, *millisecond);
+ auto date = make_date(day(time), new_time);
// 10. Let u be TimeClip(UTC(date)).
- date = time_clip(global_object, Value(utc_time(date.as_double())));
+ date = time_clip(utc_time(date));
// 11. Set the [[DateValue]] internal slot of this Date object to u.
auto* this_object = MUST(typed_this_object(global_object));
- this_object->set_date_value(date.as_double());
+ this_object->set_date_value(date);
// 12. Return u.
- return date;
+ return Value(date);
}
// 21.4.4.25 Date.prototype.setMonth ( month [ , date ] ), https://tc39.es/ecma262/#sec-date.prototype.setmonth
@@ -588,37 +587,37 @@ JS_DEFINE_NATIVE_FUNCTION(DatePrototype::set_month)
auto this_time = TRY(this_time_value(global_object, vm.this_value(global_object)));
// 2. Let m be ? ToNumber(month).
- auto month = TRY(vm.argument(0).to_number(global_object));
+ auto month = TRY(vm.argument(0).to_number(global_object)).as_double();
// 3. If date is present, let dt be ? ToNumber(date).
auto date = TRY(argument_or_empty(global_object, 1));
// 4. If t is NaN, return NaN.
- if (this_time.is_nan())
+ if (isnan(this_time))
return js_nan();
// 5. Set t to LocalTime(t).
- auto time = local_time(this_time.as_double());
+ auto time = local_time(this_time);
// 6. If date is not present, let dt be DateFromTime(t).
if (!date.has_value())
- date = Value(date_from_time(time));
+ date = date_from_time(time);
// 7. Let newDate be MakeDate(MakeDay(YearFromTime(t), m, dt), TimeWithinDay(t)).
- auto year = Value(year_from_time(time));
+ auto year = year_from_time(time);
- auto day = make_day(global_object, year, month, *date);
- auto new_date = make_date(day, Value(time_within_day(time)));
+ auto day = make_day(year, month, *date);
+ auto new_date = make_date(day, time_within_day(time));
// 8. Let u be TimeClip(UTC(newDate)).
- new_date = time_clip(global_object, Value(utc_time(new_date.as_double())));
+ new_date = time_clip(utc_time(new_date));
// 9. Set the [[DateValue]] internal slot of this Date object to u.
auto* this_object = MUST(typed_this_object(global_object));
- this_object->set_date_value(new_date.as_double());
+ this_object->set_date_value(new_date);
// 10. Return u.
- return new_date;
+ return Value(new_date);
}
// 21.4.4.26 Date.prototype.setSeconds ( sec [ , ms ] ), https://tc39.es/ecma262/#sec-date.prototype.setseconds
@@ -628,58 +627,58 @@ JS_DEFINE_NATIVE_FUNCTION(DatePrototype::set_seconds)
auto this_time = TRY(this_time_value(global_object, vm.this_value(global_object)));
// 2. Let s be ? ToNumber(sec).
- auto second = TRY(vm.argument(0).to_number(global_object));
+ auto second = TRY(vm.argument(0).to_number(global_object)).as_double();
// 3. If ms is present, let milli be ? ToNumber(ms).
auto millisecond = TRY(argument_or_empty(global_object, 1));
// 4. If t is NaN, return NaN.
- if (this_time.is_nan())
+ if (isnan(this_time))
return js_nan();
// 5. Set t to LocalTime(t).
- auto time = local_time(this_time.as_double());
+ auto time = local_time(this_time);
// 6. If ms is not present, let milli be msFromTime(t).
if (!millisecond.has_value())
- millisecond = Value(ms_from_time(time));
+ millisecond = ms_from_time(time);
// 7. Let date be MakeDate(Day(t), MakeTime(HourFromTime(t), MinFromTime(t), s, milli)).
- auto hour = Value(hour_from_time(time));
- auto minute = Value(min_from_time(time));
+ auto hour = hour_from_time(time);
+ auto minute = min_from_time(time);
- auto new_time = make_time(global_object, hour, minute, second, *millisecond);
- auto new_date = make_date(Value(day(time)), new_time);
+ auto new_time = make_time(hour, minute, second, *millisecond);
+ auto new_date = make_date(day(time), new_time);
// 8. Let u be TimeClip(UTC(date)).
- new_date = time_clip(global_object, Value(utc_time(new_date.as_double())));
+ new_date = time_clip(utc_time(new_date));
// 9. Set the [[DateValue]] internal slot of this Date object to u.
auto* this_object = MUST(typed_this_object(global_object));
- this_object->set_date_value(new_date.as_double());
+ this_object->set_date_value(new_date);
// 10. Return u.
- return new_date;
+ return Value(new_date);
}
// 21.4.4.27 Date.prototype.setTime ( time ), https://tc39.es/ecma262/#sec-date.prototype.settime
JS_DEFINE_NATIVE_FUNCTION(DatePrototype::set_time)
{
// 1. Perform ? thisTimeValue(this value).
- auto time = TRY(this_time_value(global_object, vm.this_value(global_object)));
+ TRY(this_time_value(global_object, vm.this_value(global_object)));
// 2. Let t be ? ToNumber(time).
- time = TRY(vm.argument(0).to_number(global_object));
+ auto time = TRY(vm.argument(0).to_number(global_object)).as_double();
// 3. Let v be TimeClip(t).
- time = time_clip(global_object, time);
+ time = time_clip(time);
// 4. Set the [[DateValue]] internal slot of this Date object to v.
auto* this_object = MUST(typed_this_object(global_object));
- this_object->set_date_value(time.as_double());
+ this_object->set_date_value(time);
// 5. Return v.
- return time;
+ return Value(time);
}
// 21.4.4.28 Date.prototype.setUTCDate ( date ), https://tc39.es/ecma262/#sec-date.prototype.setutcdate
@@ -689,28 +688,28 @@ JS_DEFINE_NATIVE_FUNCTION(DatePrototype::set_utc_date)
auto time = TRY(this_time_value(global_object, vm.this_value(global_object)));
// 2. Let dt be ? ToNumber(date).
- auto date = TRY(vm.argument(0).to_number(global_object));
+ auto date = TRY(vm.argument(0).to_number(global_object)).as_double();
// 3. If t is NaN, return NaN.
- if (time.is_nan())
+ if (isnan(time))
return js_nan();
// 4. Let newDate be MakeDate(MakeDay(YearFromTime(t), MonthFromTime(t), dt), TimeWithinDay(t)).
- auto year = Value(year_from_time(time.as_double()));
- auto month = Value(month_from_time(time.as_double()));
+ auto year = year_from_time(time);
+ auto month = month_from_time(time);
- auto day = make_day(global_object, year, month, date);
- auto new_date = make_date(day, Value(time_within_day(time.as_double())));
+ auto day = make_day(year, month, date);
+ auto new_date = make_date(day, time_within_day(time));
// 5. Let v be TimeClip(newDate).
- new_date = time_clip(global_object, new_date);
+ new_date = time_clip(new_date);
// 6. Set the [[DateValue]] internal slot of this Date object to v.
auto* this_object = MUST(typed_this_object(global_object));
- this_object->set_date_value(new_date.as_double());
+ this_object->set_date_value(new_date);
// 7. Return v.
- return new_date;
+ return Value(new_date);
}
// 21.4.4.29 Date.prototype.setUTCFullYear ( year [ , month [ , date ] ] ), https://tc39.es/ecma262/#sec-date.prototype.setutcfullyear
@@ -721,31 +720,31 @@ JS_DEFINE_NATIVE_FUNCTION(DatePrototype::set_utc_full_year)
// 2. If t is NaN, set t to +0๐”ฝ.
double time = 0;
- if (!this_time.is_nan())
- time = this_time.as_double();
+ if (!isnan(this_time))
+ time = this_time;
// 3. Let y be ? ToNumber(year).
- auto year = TRY(vm.argument(0).to_number(global_object));
+ auto year = TRY(vm.argument(0).to_number(global_object)).as_double();
// 4. If month is not present, let m be MonthFromTime(t); otherwise, let m be ? ToNumber(month).
- auto month = TRY(argument_or_value(global_object, 1, month_from_time(time)));
+ auto month = TRY(argument_or_number(global_object, 1, month_from_time(time)));
// 5. If date is not present, let dt be DateFromTime(t); otherwise, let dt be ? ToNumber(date).
- auto date = TRY(argument_or_value(global_object, 2, date_from_time(time)));
+ auto date = TRY(argument_or_number(global_object, 2, date_from_time(time)));
// 6. Let newDate be MakeDate(MakeDay(y, m, dt), TimeWithinDay(t)).
- auto day = make_day(global_object, year, month, date);
- auto new_date = make_date(day, Value(time_within_day(time)));
+ auto day = make_day(year, month, date);
+ auto new_date = make_date(day, time_within_day(time));
// 7. Let v be TimeClip(newDate).
- new_date = time_clip(global_object, new_date);
+ new_date = time_clip(new_date);
// 8. Set the [[DateValue]] internal slot of this Date object to v.
auto* this_object = MUST(typed_this_object(global_object));
- this_object->set_date_value(new_date.as_double());
+ this_object->set_date_value(new_date);
// 9. Return v.
- return new_date;
+ return Value(new_date);
}
// 21.4.4.30 Date.prototype.setUTCHours ( hour [ , min [ , sec [ , ms ] ] ] ), https://tc39.es/ecma262/#sec-date.prototype.setutchours
@@ -755,7 +754,7 @@ JS_DEFINE_NATIVE_FUNCTION(DatePrototype::set_utc_hours)
auto time = TRY(this_time_value(global_object, vm.this_value(global_object)));
// 2. Let h be ? ToNumber(hour).
- auto hour = TRY(vm.argument(0).to_number(global_object));
+ auto hour = TRY(vm.argument(0).to_number(global_object)).as_double();
// 3. If min is present, let m be ? ToNumber(min).
auto minute = TRY(argument_or_empty(global_object, 1));
@@ -767,34 +766,34 @@ JS_DEFINE_NATIVE_FUNCTION(DatePrototype::set_utc_hours)
auto millisecond = TRY(argument_or_empty(global_object, 3));
// 6. If t is NaN, return NaN.
- if (time.is_nan())
+ if (isnan(time))
return js_nan();
// 7. If min is not present, let m be MinFromTime(t).
if (!minute.has_value())
- minute = Value(min_from_time(time.as_double()));
+ minute = min_from_time(time);
// 8. If sec is not present, let s be SecFromTime(t).
if (!second.has_value())
- second = Value(sec_from_time(time.as_double()));
+ second = sec_from_time(time);
// 9. If ms is not present, let milli be msFromTime(t).
if (!millisecond.has_value())
- millisecond = Value(ms_from_time(time.as_double()));
+ millisecond = ms_from_time(time);
// 10. Let date be MakeDate(Day(t), MakeTime(h, m, s, milli)).
- auto new_time = make_time(global_object, hour, *minute, *second, *millisecond);
- auto date = make_date(Value(day(time.as_double())), new_time);
+ auto new_time = make_time(hour, *minute, *second, *millisecond);
+ auto date = make_date(day(time), new_time);
// 11. Let v be TimeClip(date).
- date = time_clip(global_object, date);
+ date = time_clip(date);
// 12. Set the [[DateValue]] internal slot of this Date object to v.
auto* this_object = MUST(typed_this_object(global_object));
- this_object->set_date_value(date.as_double());
+ this_object->set_date_value(date);
// 13. Return v.
- return date;
+ return Value(date);
}
// 21.4.4.31 Date.prototype.setUTCMilliseconds ( ms ), https://tc39.es/ecma262/#sec-date.prototype.setutcmilliseconds
@@ -804,29 +803,29 @@ JS_DEFINE_NATIVE_FUNCTION(DatePrototype::set_utc_milliseconds)
auto time = TRY(this_time_value(global_object, vm.this_value(global_object)));
// 2. Set ms to ? ToNumber(ms).
- auto millisecond = TRY(vm.argument(0).to_number(global_object));
+ auto millisecond = TRY(vm.argument(0).to_number(global_object)).as_double();
// 3. If t is NaN, return NaN.
- if (time.is_nan())
+ if (isnan(time))
return js_nan();
// 4. Let time be MakeTime(HourFromTime(t), MinFromTime(t), SecFromTime(t), ms).
- auto hour = Value(hour_from_time(time.as_double()));
- auto minute = Value(min_from_time(time.as_double()));
- auto second = Value(sec_from_time(time.as_double()));
+ auto hour = hour_from_time(time);
+ auto minute = min_from_time(time);
+ auto second = sec_from_time(time);
- auto new_time = make_time(global_object, hour, minute, second, millisecond);
+ auto new_time = make_time(hour, minute, second, millisecond);
// 5. Let v be TimeClip(MakeDate(Day(t), time)).
- auto date = make_date(Value(day(time.as_double())), new_time);
- date = time_clip(global_object, date);
+ auto date = make_date(day(time), new_time);
+ date = time_clip(date);
// 6. Set the [[DateValue]] internal slot of this Date object to v.
auto* this_object = MUST(typed_this_object(global_object));
- this_object->set_date_value(date.as_double());
+ this_object->set_date_value(date);
// 7. Return v.
- return date;
+ return Value(date);
}
// 21.4.4.32 Date.prototype.setUTCMinutes ( min [ , sec [ , ms ] ] ), https://tc39.es/ecma262/#sec-date.prototype.setutcminutes
@@ -836,7 +835,7 @@ JS_DEFINE_NATIVE_FUNCTION(DatePrototype::set_utc_minutes)
auto time = TRY(this_time_value(global_object, vm.this_value(global_object)));
// 2. Let m be ? ToNumber(min).
- auto minute = TRY(vm.argument(0).to_number(global_object));
+ auto minute = TRY(vm.argument(0).to_number(global_object)).as_double();
// 3. If sec is present, let s be ? ToNumber(sec).
auto second = TRY(argument_or_empty(global_object, 1));
@@ -845,32 +844,32 @@ JS_DEFINE_NATIVE_FUNCTION(DatePrototype::set_utc_minutes)
auto millisecond = TRY(argument_or_empty(global_object, 2));
// 5. If t is NaN, return NaN.
- if (time.is_nan())
+ if (isnan(time))
return js_nan();
// 6. If sec is not present, let s be SecFromTime(t).
if (!second.has_value())
- second = Value(sec_from_time(time.as_double()));
+ second = sec_from_time(time);
// 7. If ms is not present, let milli be msFromTime(t).
if (!millisecond.has_value())
- millisecond = Value(ms_from_time(time.as_double()));
+ millisecond = ms_from_time(time);
// 8. Let date be MakeDate(Day(t), MakeTime(HourFromTime(t), m, s, milli)).
- auto hour = Value(hour_from_time(time.as_double()));
+ auto hour = hour_from_time(time);
- auto new_time = make_time(global_object, hour, minute, *second, *millisecond);
- auto date = make_date(Value(day(time.as_double())), new_time);
+ auto new_time = make_time(hour, minute, *second, *millisecond);
+ auto date = make_date(day(time), new_time);
// 9. Let v be TimeClip(date).
- date = time_clip(global_object, date);
+ date = time_clip(date);
// 10. Set the [[DateValue]] internal slot of this Date object to v.
auto* this_object = MUST(typed_this_object(global_object));
- this_object->set_date_value(date.as_double());
+ this_object->set_date_value(date);
// 11. Return v.
- return date;
+ return Value(date);
}
// 21.4.4.33 Date.prototype.setUTCMonth ( month [ , date ] ), https://tc39.es/ecma262/#sec-date.prototype.setutcmonth
@@ -880,34 +879,34 @@ JS_DEFINE_NATIVE_FUNCTION(DatePrototype::set_utc_month)
auto time = TRY(this_time_value(global_object, vm.this_value(global_object)));
// 2. Let m be ? ToNumber(month).
- auto month = TRY(vm.argument(0).to_number(global_object));
+ auto month = TRY(vm.argument(0).to_number(global_object)).as_double();
// 3. If date is present, let dt be ? ToNumber(date).
auto date = TRY(argument_or_empty(global_object, 1));
// 4. If t is NaN, return NaN.
- if (time.is_nan())
+ if (isnan(time))
return js_nan();
// 5. If date is not present, let dt be DateFromTime(t).
if (!date.has_value())
- date = Value(date_from_time(time.as_double()));
+ date = date_from_time(time);
// 6. Let newDate be MakeDate(MakeDay(YearFromTime(t), m, dt), TimeWithinDay(t)).
- auto year = Value(year_from_time(time.as_double()));
+ auto year = year_from_time(time);
- auto day = make_day(global_object, year, month, *date);
- auto new_date = make_date(day, Value(time_within_day(time.as_double())));
+ auto day = make_day(year, month, *date);
+ auto new_date = make_date(day, time_within_day(time));
// 7. Let v be TimeClip(newDate).
- new_date = time_clip(global_object, new_date);
+ new_date = time_clip(new_date);
// 8. Set the [[DateValue]] internal slot of this Date object to v.
auto* this_object = MUST(typed_this_object(global_object));
- this_object->set_date_value(new_date.as_double());
+ this_object->set_date_value(new_date);
// 9. Return v.
- return new_date;
+ return Value(new_date);
}
// 21.4.4.34 Date.prototype.setUTCSeconds ( sec [ , ms ] ), https://tc39.es/ecma262/#sec-date.prototype.setutcseconds
@@ -917,35 +916,35 @@ JS_DEFINE_NATIVE_FUNCTION(DatePrototype::set_utc_seconds)
auto time = TRY(this_time_value(global_object, vm.this_value(global_object)));
// 2. Let s be ? ToNumber(sec).
- auto second = TRY(vm.argument(0).to_number(global_object));
+ auto second = TRY(vm.argument(0).to_number(global_object)).as_double();
// 3. If ms is present, let milli be ? ToNumber(ms).
auto millisecond = TRY(argument_or_empty(global_object, 1));
// 4. If t is NaN, return NaN.
- if (time.is_nan())
+ if (isnan(time))
return js_nan();
// 5. If ms is not present, let milli be msFromTime(t).
if (!millisecond.has_value())
- millisecond = Value(ms_from_time(time.as_double()));
+ millisecond = ms_from_time(time);
// 6. Let date be MakeDate(Day(t), MakeTime(HourFromTime(t), MinFromTime(t), s, milli)).
- auto hour = Value(hour_from_time(time.as_double()));
- auto minute = Value(min_from_time(time.as_double()));
+ auto hour = hour_from_time(time);
+ auto minute = min_from_time(time);
- auto new_time = make_time(global_object, hour, minute, second, *millisecond);
- auto new_date = make_date(Value(day(time.as_double())), new_time);
+ auto new_time = make_time(hour, minute, second, *millisecond);
+ auto new_date = make_date(day(time), new_time);
// 7. Let v be TimeClip(date).
- new_date = time_clip(global_object, new_date);
+ new_date = time_clip(new_date);
// 8. Set the [[DateValue]] internal slot of this Date object to v.
auto* this_object = MUST(typed_this_object(global_object));
- this_object->set_date_value(new_date.as_double());
+ this_object->set_date_value(new_date);
// 9. Return v.
- return new_date;
+ return Value(new_date);
}
// 21.4.4.35 Date.prototype.toDateString ( ), https://tc39.es/ecma262/#sec-date.prototype.todatestring
@@ -956,12 +955,12 @@ JS_DEFINE_NATIVE_FUNCTION(DatePrototype::to_date_string)
auto time = TRY(this_time_value(global_object, vm.this_value(global_object)));
// 3. If tv is NaN, return "Invalid Date".
- if (time.is_nan())
+ if (isnan(time))
return js_string(vm, "Invalid Date"sv);
// 4. Let t be LocalTime(tv).
// 5. Return DateString(t).
- return js_string(vm, date_string(local_time(time.as_double())));
+ return js_string(vm, date_string(local_time(time)));
}
// 21.4.4.36 Date.prototype.toISOString ( ), https://tc39.es/ecma262/#sec-date.prototype.toisostring
@@ -1006,7 +1005,7 @@ JS_DEFINE_NATIVE_FUNCTION(DatePrototype::to_locale_date_string)
auto time = TRY(this_time_value(global_object, vm.this_value(global_object)));
// 2. If x is NaN, return "Invalid Date".
- if (time.is_nan())
+ if (isnan(time))
return js_string(vm, "Invalid Date"sv);
// 3. Let options be ? ToDateTimeOptions(options, "date", "date").
@@ -1031,7 +1030,7 @@ JS_DEFINE_NATIVE_FUNCTION(DatePrototype::to_locale_string)
auto time = TRY(this_time_value(global_object, vm.this_value(global_object)));
// 2. If x is NaN, return "Invalid Date".
- if (time.is_nan())
+ if (isnan(time))
return js_string(vm, "Invalid Date"sv);
// 3. Let options be ? ToDateTimeOptions(options, "any", "all").
@@ -1056,7 +1055,7 @@ JS_DEFINE_NATIVE_FUNCTION(DatePrototype::to_locale_time_string)
auto time = TRY(this_time_value(global_object, vm.this_value(global_object)));
// 2. If x is NaN, return "Invalid Date".
- if (time.is_nan())
+ if (isnan(time))
return js_string(vm, "Invalid Date"sv);
// 3. Let options be ? ToDateTimeOptions(options, "time", "time").
@@ -1077,7 +1076,7 @@ JS_DEFINE_NATIVE_FUNCTION(DatePrototype::to_string)
auto time = TRY(this_time_value(global_object, vm.this_value(global_object)));
// 2. Return ToDateString(tv).
- return js_string(vm, JS::to_date_string(time.as_double()));
+ return js_string(vm, JS::to_date_string(time));
}
// 21.4.4.41.1 TimeString ( tv ), https://tc39.es/ecma262/#sec-timestring
@@ -1181,7 +1180,7 @@ JS_DEFINE_NATIVE_FUNCTION(DatePrototype::to_temporal_instant)
auto t = TRY(this_time_value(global_object, vm.this_value(global_object)));
// 2. Let ns be ? NumberToBigInt(t) ร— 10^6.
- auto* ns = TRY(number_to_bigint(global_object, t));
+ auto* ns = TRY(number_to_bigint(global_object, Value(t)));
ns = js_bigint(vm, ns->big_integer().multiplied_by(Crypto::UnsignedBigInteger { 1'000'000 }));
// 3. Return ! CreateTemporalInstant(ns).
@@ -1196,12 +1195,12 @@ JS_DEFINE_NATIVE_FUNCTION(DatePrototype::to_time_string)
auto time = TRY(this_time_value(global_object, vm.this_value(global_object)));
// 3. If tv is NaN, return "Invalid Date".
- if (time.is_nan())
+ if (isnan(time))
return js_string(vm, "Invalid Date"sv);
// 4. Let t be LocalTime(tv).
// 5. Return the string-concatenation of TimeString(t) and TimeZoneString(tv).
- auto string = String::formatted("{}{}", time_string(local_time(time.as_double())), time_zone_string(time.as_double()));
+ auto string = String::formatted("{}{}", time_string(local_time(time)), time_zone_string(time));
return js_string(vm, move(string));
}
@@ -1213,27 +1212,27 @@ JS_DEFINE_NATIVE_FUNCTION(DatePrototype::to_utc_string)
auto time = TRY(this_time_value(global_object, vm.this_value(global_object)));
// 3. If tv is NaN, return "Invalid Date".
- if (time.is_nan())
+ if (isnan(time))
return js_string(vm, "Invalid Date"sv);
// 4. Let weekday be the Name of the entry in Table 62 with the Number WeekDay(tv).
- auto weekday = short_day_names[week_day(time.as_double())];
+ auto weekday = short_day_names[week_day(time)];
// 5. Let month be the Name of the entry in Table 63 with the Number MonthFromTime(tv).
- auto month = short_month_names[month_from_time(time.as_double())];
+ auto month = short_month_names[month_from_time(time)];
// 6. Let day be ToZeroPaddedDecimalString(โ„(DateFromTime(tv)), 2).
- auto day = date_from_time(time.as_double());
+ auto day = date_from_time(time);
// 7. Let yv be YearFromTime(tv).
- auto year = year_from_time(time.as_double());
+ auto year = year_from_time(time);
// 8. If yv is +0๐”ฝ or yv > +0๐”ฝ, let yearSign be the empty String; otherwise, let yearSign be "-".
auto year_sign = year >= 0 ? ""sv : "-"sv;
// 9. Let paddedYear be ToZeroPaddedDecimalString(abs(โ„(yv)), 4).
// 10. Return the string-concatenation of weekday, ",", the code unit 0x0020 (SPACE), day, the code unit 0x0020 (SPACE), month, the code unit 0x0020 (SPACE), yearSign, paddedYear, the code unit 0x0020 (SPACE), and TimeString(tv).
- auto string = String::formatted("{}, {:02} {} {}{:04} {}", weekday, day, month, year_sign, abs(year), time_string(time.as_double()));
+ auto string = String::formatted("{}, {:02} {} {}{:04} {}", weekday, day, month, year_sign, abs(year), time_string(time));
return js_string(vm, move(string));
}
@@ -1264,11 +1263,11 @@ JS_DEFINE_NATIVE_FUNCTION(DatePrototype::get_year)
auto time = TRY(this_time_value(global_object, vm.this_value(global_object)));
// 2. If t is NaN, return NaN.
- if (time.is_nan())
- return time;
+ if (isnan(time))
+ return js_nan();
- // 3. Return Return YearFromTime(LocalTime(t)) - 1900๐”ฝ.
- return Value(year_from_time(local_time(time.as_double())) - 1900);
+ // 3. Return YearFromTime(LocalTime(t)) - 1900๐”ฝ.
+ return Value(year_from_time(local_time(time)) - 1900);
}
// B.2.4.2 Date.prototype.setYear ( year ), https://tc39.es/ecma262/#sec-date.prototype.setyear
@@ -1279,43 +1278,43 @@ JS_DEFINE_NATIVE_FUNCTION(DatePrototype::set_year)
// 2. If t is NaN, set t to +0๐”ฝ; otherwise, set t to LocalTime(t).
double time = 0;
- if (!this_time.is_nan())
- time = local_time(this_time.as_double());
+ if (!isnan(this_time))
+ time = local_time(this_time);
// 3. Let y be ? ToNumber(year).
- auto year = TRY(vm.argument(0).to_number(global_object));
+ auto year = TRY(vm.argument(0).to_number(global_object)).as_double();
auto* this_object = MUST(typed_this_object(global_object));
// 4. If y is NaN, then
- if (year.is_nan()) {
+ if (isnan(year)) {
// a. Set the [[DateValue]] internal slot of this Date object to NaN.
- this_object->set_date_value(js_nan().as_double());
+ this_object->set_date_value(NAN);
// b. Return NaN.
- return year;
+ return js_nan();
}
// 5. Let yi be ! ToIntegerOrInfinity(y).
- auto year_double = MUST(year.to_integer_or_infinity(global_object));
+ auto year_integer = to_integer_or_infinity(year);
// 6. If 0 โ‰ค yi โ‰ค 99, let yyyy be 1900๐”ฝ + ๐”ฝ(yi).
- if (0 <= year_double && year_double <= 99)
- year = Value(1900 + year_double);
+ if (0 <= year_integer && year_integer <= 99)
+ year = 1900 + year_integer;
// 7. Else, let yyyy be y.
// 8. Let d be MakeDay(yyyy, MonthFromTime(t), DateFromTime(t)).
- auto day = make_day(global_object, year, Value(month_from_time(time)), Value(date_from_time(time)));
+ auto day = make_day(year, month_from_time(time), date_from_time(time));
// 9. Let date be UTC(MakeDate(d, TimeWithinDay(t))).
- auto date = utc_time(make_date(day, Value(time_within_day(time))).as_double());
+ auto date = utc_time(make_date(day, time_within_day(time)));
// 10. Set the [[DateValue]] internal slot of this Date object to TimeClip(date).
- auto new_date = time_clip(global_object, Value(date));
- this_object->set_date_value(new_date.as_double());
+ auto new_date = time_clip(date);
+ this_object->set_date_value(new_date);
// 11. Return the value of the [[DateValue]] internal slot of this Date object.
- return new_date;
+ return Value(new_date);
}
// B.2.4.3 Date.prototype.toGMTString ( ), https://tc39.es/ecma262/#sec-date.prototype.togmtstring
diff --git a/Userland/Libraries/LibJS/Runtime/DatePrototype.h b/Userland/Libraries/LibJS/Runtime/DatePrototype.h
index 805b2a6bab..f3c75d2b15 100644
--- a/Userland/Libraries/LibJS/Runtime/DatePrototype.h
+++ b/Userland/Libraries/LibJS/Runtime/DatePrototype.h
@@ -1,5 +1,5 @@
/*
- * Copyright (c) 2020, Linus Groh <linusg@serenityos.org>
+ * Copyright (c) 2020-2022, Linus Groh <linusg@serenityos.org>
*
* SPDX-License-Identifier: BSD-2-Clause
*/
@@ -71,7 +71,7 @@ private:
JS_DECLARE_NATIVE_FUNCTION(symbol_to_primitive);
};
-ThrowCompletionOr<Value> this_time_value(GlobalObject& global_object, Value value);
+ThrowCompletionOr<double> this_time_value(GlobalObject& global_object, Value value);
String time_string(double time);
String date_string(double time);
String time_zone_string(double time);
diff --git a/Userland/Libraries/LibJS/Runtime/Intl/DateTimeFormat.cpp b/Userland/Libraries/LibJS/Runtime/Intl/DateTimeFormat.cpp
index 38456ff04b..10e15fe888 100644
--- a/Userland/Libraries/LibJS/Runtime/Intl/DateTimeFormat.cpp
+++ b/Userland/Libraries/LibJS/Runtime/Intl/DateTimeFormat.cpp
@@ -504,15 +504,15 @@ static Optional<StyleAndValue> find_calendar_field(StringView name, Unicode::Cal
}
// 11.5.6 FormatDateTimePattern ( dateTimeFormat, patternParts, x, rangeFormatOptions ), https://tc39.es/ecma402/#sec-formatdatetimepattern
-ThrowCompletionOr<Vector<PatternPartition>> format_date_time_pattern(GlobalObject& global_object, DateTimeFormat& date_time_format, Vector<PatternPartition> pattern_parts, Value time, Unicode::CalendarPattern const* range_format_options)
+ThrowCompletionOr<Vector<PatternPartition>> format_date_time_pattern(GlobalObject& global_object, DateTimeFormat& date_time_format, Vector<PatternPartition> pattern_parts, double time, Unicode::CalendarPattern const* range_format_options)
{
auto& vm = global_object.vm();
// 1. Let x be TimeClip(x).
- time = time_clip(global_object, time);
+ time = time_clip(time);
// 2. If x is NaN, throw a RangeError exception.
- if (time.is_nan())
+ if (isnan(time))
return vm.throw_completion<RangeError>(global_object, ErrorType::IntlInvalidTime);
// 3. Let locale be dateTimeFormat.[[Locale]].
@@ -567,7 +567,7 @@ ThrowCompletionOr<Vector<PatternPartition>> format_date_time_pattern(GlobalObjec
}
// 13. Let tm be ToLocalTime(x, dateTimeFormat.[[Calendar]], dateTimeFormat.[[TimeZone]]).
- auto local_time = TRY(to_local_time(global_object, time.as_double(), date_time_format.calendar(), date_time_format.time_zone()));
+ auto local_time = TRY(to_local_time(global_object, time, date_time_format.calendar(), date_time_format.time_zone()));
// 14. Let result be a new empty List.
Vector<PatternPartition> result;
@@ -788,7 +788,7 @@ ThrowCompletionOr<Vector<PatternPartition>> format_date_time_pattern(GlobalObjec
}
// 11.5.7 PartitionDateTimePattern ( dateTimeFormat, x ), https://tc39.es/ecma402/#sec-partitiondatetimepattern
-ThrowCompletionOr<Vector<PatternPartition>> partition_date_time_pattern(GlobalObject& global_object, DateTimeFormat& date_time_format, Value time)
+ThrowCompletionOr<Vector<PatternPartition>> partition_date_time_pattern(GlobalObject& global_object, DateTimeFormat& date_time_format, double time)
{
// 1. Let patternParts be PartitionPattern(dateTimeFormat.[[Pattern]]).
auto pattern_parts = partition_pattern(date_time_format.pattern());
@@ -801,7 +801,7 @@ ThrowCompletionOr<Vector<PatternPartition>> partition_date_time_pattern(GlobalOb
}
// 11.5.8 FormatDateTime ( dateTimeFormat, x ), https://tc39.es/ecma402/#sec-formatdatetime
-ThrowCompletionOr<String> format_date_time(GlobalObject& global_object, DateTimeFormat& date_time_format, Value time)
+ThrowCompletionOr<String> format_date_time(GlobalObject& global_object, DateTimeFormat& date_time_format, double time)
{
// 1. Let parts be ? PartitionDateTimePattern(dateTimeFormat, x).
auto parts = TRY(partition_date_time_pattern(global_object, date_time_format, time));
@@ -820,7 +820,7 @@ ThrowCompletionOr<String> format_date_time(GlobalObject& global_object, DateTime
}
// 11.5.9 FormatDateTimeToParts ( dateTimeFormat, x ), https://tc39.es/ecma402/#sec-formatdatetimetoparts
-ThrowCompletionOr<Array*> format_date_time_to_parts(GlobalObject& global_object, DateTimeFormat& date_time_format, Value time)
+ThrowCompletionOr<Array*> format_date_time_to_parts(GlobalObject& global_object, DateTimeFormat& date_time_format, double time)
{
auto& vm = global_object.vm();
@@ -891,33 +891,33 @@ ThrowCompletionOr<void> for_each_range_pattern_with_source(Unicode::CalendarRang
}
// 11.5.10 PartitionDateTimeRangePattern ( dateTimeFormat, x, y ), https://tc39.es/ecma402/#sec-partitiondatetimerangepattern
-ThrowCompletionOr<Vector<PatternPartitionWithSource>> partition_date_time_range_pattern(GlobalObject& global_object, DateTimeFormat& date_time_format, Value start, Value end)
+ThrowCompletionOr<Vector<PatternPartitionWithSource>> partition_date_time_range_pattern(GlobalObject& global_object, DateTimeFormat& date_time_format, double start, double end)
{
auto& vm = global_object.vm();
// 1. Let x be TimeClip(x).
- start = time_clip(global_object, start);
+ start = time_clip(start);
// 2. If x is NaN, throw a RangeError exception.
- if (start.is_nan())
+ if (isnan(start))
return vm.throw_completion<RangeError>(global_object, ErrorType::IntlInvalidTime);
// 3. Let y be TimeClip(y).
- end = time_clip(global_object, end);
+ end = time_clip(end);
// 4. If y is NaN, throw a RangeError exception.
- if (end.is_nan())
+ if (isnan(end))
return vm.throw_completion<RangeError>(global_object, ErrorType::IntlInvalidTime);
// 5. If x is greater than y, throw a RangeError exception.
- if (start.as_double() > end.as_double())
+ if (start > end)
return vm.throw_completion<RangeError>(global_object, ErrorType::IntlStartTimeAfterEndTime, start, end);
// 6. Let tm1 be ToLocalTime(x, dateTimeFormat.[[Calendar]], dateTimeFormat.[[TimeZone]]).
- auto start_local_time = TRY(to_local_time(global_object, start.as_double(), date_time_format.calendar(), date_time_format.time_zone()));
+ auto start_local_time = TRY(to_local_time(global_object, start, date_time_format.calendar(), date_time_format.time_zone()));
// 7. Let tm2 be ToLocalTime(y, dateTimeFormat.[[Calendar]], dateTimeFormat.[[TimeZone]]).
- auto end_local_time = TRY(to_local_time(global_object, end.as_double(), date_time_format.calendar(), date_time_format.time_zone()));
+ auto end_local_time = TRY(to_local_time(global_object, end, date_time_format.calendar(), date_time_format.time_zone()));
// 8. Let rangePatterns be dateTimeFormat.[[RangePatterns]].
auto range_patterns = date_time_format.range_patterns();
@@ -1121,7 +1121,7 @@ ThrowCompletionOr<Vector<PatternPartitionWithSource>> partition_date_time_range_
}
// 11.5.11 FormatDateTimeRange ( dateTimeFormat, x, y ), https://tc39.es/ecma402/#sec-formatdatetimerange
-ThrowCompletionOr<String> format_date_time_range(GlobalObject& global_object, DateTimeFormat& date_time_format, Value start, Value end)
+ThrowCompletionOr<String> format_date_time_range(GlobalObject& global_object, DateTimeFormat& date_time_format, double start, double end)
{
// 1. Let parts be ? PartitionDateTimeRangePattern(dateTimeFormat, x, y).
auto parts = TRY(partition_date_time_range_pattern(global_object, date_time_format, start, end));
@@ -1140,7 +1140,7 @@ ThrowCompletionOr<String> format_date_time_range(GlobalObject& global_object, Da
}
// 11.5.12 FormatDateTimeRangeToParts ( dateTimeFormat, x, y ), https://tc39.es/ecma402/#sec-formatdatetimerangetoparts
-ThrowCompletionOr<Array*> format_date_time_range_to_parts(GlobalObject& global_object, DateTimeFormat& date_time_format, Value start, Value end)
+ThrowCompletionOr<Array*> format_date_time_range_to_parts(GlobalObject& global_object, DateTimeFormat& date_time_format, double start, double end)
{
auto& vm = global_object.vm();
diff --git a/Userland/Libraries/LibJS/Runtime/Intl/DateTimeFormat.h b/Userland/Libraries/LibJS/Runtime/Intl/DateTimeFormat.h
index f4ffd3e7fa..eb17b0ea69 100644
--- a/Userland/Libraries/LibJS/Runtime/Intl/DateTimeFormat.h
+++ b/Userland/Libraries/LibJS/Runtime/Intl/DateTimeFormat.h
@@ -203,13 +203,13 @@ ThrowCompletionOr<Object*> to_date_time_options(GlobalObject& global_object, Val
Optional<Unicode::CalendarPattern> date_time_style_format(StringView data_locale, DateTimeFormat& date_time_format);
Optional<Unicode::CalendarPattern> basic_format_matcher(Unicode::CalendarPattern const& options, Vector<Unicode::CalendarPattern> formats);
Optional<Unicode::CalendarPattern> best_fit_format_matcher(Unicode::CalendarPattern const& options, Vector<Unicode::CalendarPattern> formats);
-ThrowCompletionOr<Vector<PatternPartition>> format_date_time_pattern(GlobalObject& global_object, DateTimeFormat& date_time_format, Vector<PatternPartition> pattern_parts, Value time, Unicode::CalendarPattern const* range_format_options);
-ThrowCompletionOr<Vector<PatternPartition>> partition_date_time_pattern(GlobalObject& global_object, DateTimeFormat& date_time_format, Value time);
-ThrowCompletionOr<String> format_date_time(GlobalObject& global_object, DateTimeFormat& date_time_format, Value time);
-ThrowCompletionOr<Array*> format_date_time_to_parts(GlobalObject& global_object, DateTimeFormat& date_time_format, Value time);
-ThrowCompletionOr<Vector<PatternPartitionWithSource>> partition_date_time_range_pattern(GlobalObject& global_object, DateTimeFormat& date_time_format, Value start, Value end);
-ThrowCompletionOr<String> format_date_time_range(GlobalObject& global_object, DateTimeFormat& date_time_format, Value start, Value end);
-ThrowCompletionOr<Array*> format_date_time_range_to_parts(GlobalObject& global_object, DateTimeFormat& date_time_format, Value start, Value end);
+ThrowCompletionOr<Vector<PatternPartition>> format_date_time_pattern(GlobalObject& global_object, DateTimeFormat& date_time_format, Vector<PatternPartition> pattern_parts, double time, Unicode::CalendarPattern const* range_format_options);
+ThrowCompletionOr<Vector<PatternPartition>> partition_date_time_pattern(GlobalObject& global_object, DateTimeFormat& date_time_format, double time);
+ThrowCompletionOr<String> format_date_time(GlobalObject& global_object, DateTimeFormat& date_time_format, double time);
+ThrowCompletionOr<Array*> format_date_time_to_parts(GlobalObject& global_object, DateTimeFormat& date_time_format, double time);
+ThrowCompletionOr<Vector<PatternPartitionWithSource>> partition_date_time_range_pattern(GlobalObject& global_object, DateTimeFormat& date_time_format, double start, double end);
+ThrowCompletionOr<String> format_date_time_range(GlobalObject& global_object, DateTimeFormat& date_time_format, double start, double end);
+ThrowCompletionOr<Array*> format_date_time_range_to_parts(GlobalObject& global_object, DateTimeFormat& date_time_format, double start, double end);
ThrowCompletionOr<LocalTime> to_local_time(GlobalObject& global_object, double time, StringView calendar, StringView time_zone);
template<typename Callback>
diff --git a/Userland/Libraries/LibJS/Runtime/Intl/DateTimeFormatFunction.cpp b/Userland/Libraries/LibJS/Runtime/Intl/DateTimeFormatFunction.cpp
index 82d4a8721b..114bf06916 100644
--- a/Userland/Libraries/LibJS/Runtime/Intl/DateTimeFormatFunction.cpp
+++ b/Userland/Libraries/LibJS/Runtime/Intl/DateTimeFormatFunction.cpp
@@ -44,19 +44,21 @@ ThrowCompletionOr<Value> DateTimeFormatFunction::call()
// 1. Let dtf be F.[[DateTimeFormat]].
// 2. Assert: Type(dtf) is Object and dtf has an [[InitializedDateTimeFormat]] internal slot.
+ double date_value;
+
// 3. If date is not provided or is undefined, then
if (date.is_undefined()) {
// a. Let x be ! Call(%Date.now%, undefined).
- date = MUST(JS::call(global_object, global_object.date_constructor_now_function(), js_undefined()));
+ date_value = MUST(JS::call(global_object, global_object.date_constructor_now_function(), js_undefined())).as_double();
}
// 4. Else,
else {
// a. Let x be ? ToNumber(date).
- date = TRY(date.to_number(global_object));
+ date_value = TRY(date.to_number(global_object)).as_double();
}
// 5. Return ? FormatDateTime(dtf, x).
- auto formatted = TRY(format_date_time(global_object, m_date_time_format, date));
+ auto formatted = TRY(format_date_time(global_object, m_date_time_format, date_value));
return js_string(vm, move(formatted));
}
diff --git a/Userland/Libraries/LibJS/Runtime/Intl/DateTimeFormatPrototype.cpp b/Userland/Libraries/LibJS/Runtime/Intl/DateTimeFormatPrototype.cpp
index 4fe76eb643..e1161f80d7 100644
--- a/Userland/Libraries/LibJS/Runtime/Intl/DateTimeFormatPrototype.cpp
+++ b/Userland/Libraries/LibJS/Runtime/Intl/DateTimeFormatPrototype.cpp
@@ -69,19 +69,21 @@ JS_DEFINE_NATIVE_FUNCTION(DateTimeFormatPrototype::format_to_parts)
// 2. Perform ? RequireInternalSlot(dtf, [[InitializedDateTimeFormat]]).
auto* date_time_format = TRY(typed_this_object(global_object));
+ double date_value;
+
// 3. If date is undefined, then
if (date.is_undefined()) {
// a. Let x be ! Call(%Date.now%, undefined).
- date = MUST(call(global_object, global_object.date_constructor_now_function(), js_undefined()));
+ date_value = MUST(call(global_object, global_object.date_constructor_now_function(), js_undefined())).as_double();
}
// 4. Else,
else {
// a. Let x be ? ToNumber(date).
- date = TRY(date.to_number(global_object));
+ date_value = TRY(date.to_number(global_object)).as_double();
}
// 5. Return ? FormatDateTimeToParts(dtf, x).
- return TRY(format_date_time_to_parts(global_object, *date_time_format, date));
+ return TRY(format_date_time_to_parts(global_object, *date_time_format, date_value));
}
// 11.3.5 Intl.DateTimeFormat.prototype.formatRange ( startDate, endDate ), https://tc39.es/ecma402/#sec-intl.datetimeformat.prototype.formatRange
@@ -101,13 +103,13 @@ JS_DEFINE_NATIVE_FUNCTION(DateTimeFormatPrototype::format_range)
return vm.throw_completion<TypeError>(global_object, ErrorType::IsUndefined, "endDate"sv);
// 4. Let x be ? ToNumber(startDate).
- start_date = TRY(start_date.to_number(global_object));
+ auto start_date_number = TRY(start_date.to_number(global_object)).as_double();
// 5. Let y be ? ToNumber(endDate).
- end_date = TRY(end_date.to_number(global_object));
+ auto end_date_number = TRY(end_date.to_number(global_object)).as_double();
// 6. Return ? FormatDateTimeRange(dtf, x, y).
- auto formatted = TRY(format_date_time_range(global_object, *date_time_format, start_date, end_date));
+ auto formatted = TRY(format_date_time_range(global_object, *date_time_format, start_date_number, end_date_number));
return js_string(vm, move(formatted));
}
@@ -128,13 +130,13 @@ JS_DEFINE_NATIVE_FUNCTION(DateTimeFormatPrototype::format_range_to_parts)
return vm.throw_completion<TypeError>(global_object, ErrorType::IsUndefined, "endDate"sv);
// 4. Let x be ? ToNumber(startDate).
- start_date = TRY(start_date.to_number(global_object));
+ auto start_date_number = TRY(start_date.to_number(global_object)).as_double();
// 5. Let y be ? ToNumber(endDate).
- end_date = TRY(end_date.to_number(global_object));
+ auto end_date_number = TRY(end_date.to_number(global_object)).as_double();
// 6. Return ? FormatDateTimeRangeToParts(dtf, x, y).
- return TRY(format_date_time_range_to_parts(global_object, *date_time_format, start_date, end_date));
+ return TRY(format_date_time_range_to_parts(global_object, *date_time_format, start_date_number, end_date_number));
}
// 11.3.7 Intl.DateTimeFormat.prototype.resolvedOptions ( ), https://tc39.es/ecma402/#sec-intl.datetimeformat.prototype.resolvedoptions
diff --git a/Userland/Libraries/LibJS/Runtime/Temporal/PlainDateTime.cpp b/Userland/Libraries/LibJS/Runtime/Temporal/PlainDateTime.cpp
index a7f92e4837..dbcab74172 100644
--- a/Userland/Libraries/LibJS/Runtime/Temporal/PlainDateTime.cpp
+++ b/Userland/Libraries/LibJS/Runtime/Temporal/PlainDateTime.cpp
@@ -53,19 +53,19 @@ BigInt* get_epoch_from_iso_parts(GlobalObject& global_object, i32 year, u8 month
VERIFY(is_valid_iso_date(year, month, day));
// 2. Let date be MakeDay(๐”ฝ(year), ๐”ฝ(month - 1), ๐”ฝ(day)).
- auto date = make_day(global_object, Value(year), Value(month - 1), Value(day));
+ auto date = make_day(year, month - 1, day);
// 3. Let time be MakeTime(๐”ฝ(hour), ๐”ฝ(minute), ๐”ฝ(second), ๐”ฝ(millisecond)).
- auto time = make_time(global_object, Value(hour), Value(minute), Value(second), Value(millisecond));
+ auto time = make_time(hour, minute, second, millisecond);
// 4. Let ms be MakeDate(date, time).
auto ms = make_date(date, time);
// 5. Assert: ms is finite.
- VERIFY(ms.is_finite_number());
+ VERIFY(isfinite(ms));
// 6. Return โ„ค(โ„(ms) ร— 10^6 + microsecond ร— 10^3 + nanosecond).
- return js_bigint(vm, Crypto::SignedBigInteger::create_from(static_cast<i64>(ms.as_double())).multiplied_by(Crypto::UnsignedBigInteger { 1'000'000 }).plus(Crypto::SignedBigInteger::create_from((i64)microsecond * 1000)).plus(Crypto::SignedBigInteger(nanosecond)));
+ return js_bigint(vm, Crypto::SignedBigInteger::create_from(static_cast<i64>(ms)).multiplied_by(Crypto::UnsignedBigInteger { 1'000'000 }).plus(Crypto::SignedBigInteger::create_from((i64)microsecond * 1000)).plus(Crypto::SignedBigInteger(nanosecond)));
}
// -864 * 10^19 - 864 * 10^11