From fb47a873400d7a5483abc8ffdd1939f08180bd16 Mon Sep 17 00:00:00 2001 From: Linus Groh Date: Wed, 3 Aug 2022 22:46:38 +0100 Subject: [PATCH] LibJS: Add previously skipped assertion in RegulateTime The spec issue preventing us from doing this in the past has long been fixed. --- Userland/Libraries/LibJS/Runtime/Temporal/PlainTime.cpp | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/Userland/Libraries/LibJS/Runtime/Temporal/PlainTime.cpp b/Userland/Libraries/LibJS/Runtime/Temporal/PlainTime.cpp index 4715e0330b..a25e5583a1 100644 --- a/Userland/Libraries/LibJS/Runtime/Temporal/PlainTime.cpp +++ b/Userland/Libraries/LibJS/Runtime/Temporal/PlainTime.cpp @@ -160,8 +160,7 @@ ThrowCompletionOr regulate_time(GlobalObject& global_object, doubl auto& vm = global_object.vm(); // 1. Assert: hour, minute, second, millisecond, microsecond and nanosecond are integers. - // NOTE: As the spec is currently written this assertion can fail, these are either integers _or_ infinity. - // See https://github.com/tc39/proposal-temporal/issues/1672. + VERIFY(trunc(hour) == hour && trunc(minute) == minute && trunc(second) == second && trunc(millisecond) == millisecond && trunc(microsecond) == microsecond && trunc(nanosecond) == nanosecond); // 2. Assert: overflow is either "constrain" or "reject". // NOTE: Asserted by the VERIFY_NOT_REACHED at the end