From 938e68d003ae91f3c95b53ae2a234534ff72f438 Mon Sep 17 00:00:00 2001 From: Linus Groh Date: Fri, 6 May 2022 19:34:59 +0200 Subject: [PATCH] LibJS: Remove type assertion comment from IsValidISODate This is an editorial change in the Temporal spec. See: https://github.com/tc39/proposal-temporal/commit/ddb5652 --- .../Libraries/LibJS/Runtime/Temporal/PlainDate.cpp | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/Userland/Libraries/LibJS/Runtime/Temporal/PlainDate.cpp b/Userland/Libraries/LibJS/Runtime/Temporal/PlainDate.cpp index 1243776e4c..f571dd5c11 100644 --- a/Userland/Libraries/LibJS/Runtime/Temporal/PlainDate.cpp +++ b/Userland/Libraries/LibJS/Runtime/Temporal/PlainDate.cpp @@ -374,24 +374,22 @@ ThrowCompletionOr regulate_iso_date(GlobalObject& global_object, double // 3.5.5 IsValidISODate ( year, month, day ), https://tc39.es/proposal-temporal/#sec-temporal-isvalidisodate bool is_valid_iso_date(i32 year, u8 month, u8 day) { - // 1. Assert: year, month, and day are integers. - - // 2. If month < 1 or month > 12, then + // 1. If month < 1 or month > 12, then if (month < 1 || month > 12) { // a. Return false. return false; } - // 3. Let daysInMonth be ! ISODaysInMonth(year, month). + // 2. Let daysInMonth be ! ISODaysInMonth(year, month). auto days_in_month = iso_days_in_month(year, month); - // 4. If day < 1 or day > daysInMonth, then + // 3. If day < 1 or day > daysInMonth, then if (day < 1 || day > days_in_month) { // a. Return false. return false; } - // 5. Return true. + // 4. Return true. return true; }