From dc03529ffd590195526babc1d7d5524824df8256 Mon Sep 17 00:00:00 2001 From: Emanuele Torre Date: Thu, 30 Dec 2021 23:30:56 +0100 Subject: [PATCH] LibJS: Fix spec comment in Temporal::PlainDate::balance_iso_date() This matches the text of the spec, and is more correct since the variable is being updated, not defined it. See: https://github.com/tc39/proposal-temporal/commit/5ab1822 --- I also changed `test_year += 1` to `test_year++` for consistency with step 11.c that has the same description. --- Userland/Libraries/LibJS/Runtime/Temporal/PlainDate.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Userland/Libraries/LibJS/Runtime/Temporal/PlainDate.cpp b/Userland/Libraries/LibJS/Runtime/Temporal/PlainDate.cpp index d4dbc64084..946de23205 100644 --- a/Userland/Libraries/LibJS/Runtime/Temporal/PlainDate.cpp +++ b/Userland/Libraries/LibJS/Runtime/Temporal/PlainDate.cpp @@ -440,8 +440,8 @@ ISODate balance_iso_date(double year_, double month_, double day) // 9. NOTE: To deal with numbers of days greater than the number of days in a year, the following section adds years and subtracts days until the number of days is less than 366 or 365. - // 10. Let testYear be testYear + 1. - test_year += 1; + // 10. Set testYear to testYear + 1. + test_year++; // 11. Repeat, while day > ! ISODaysInYear(testYear), while (day > iso_days_in_year(test_year)) {