From eaf6f20ed1e153770b57c30fd95b8d0a082cc439 Mon Sep 17 00:00:00 2001 From: Linus Groh Date: Wed, 15 Jun 2022 00:20:05 +0100 Subject: [PATCH] LibJS: Move assertion step around in Calendar.prototype.mergeFields This is an editorial change in the Temporal spec. See: https://github.com/tc39/proposal-temporal/commit/f790821 --- .../LibJS/Runtime/Temporal/CalendarPrototype.cpp | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/Userland/Libraries/LibJS/Runtime/Temporal/CalendarPrototype.cpp b/Userland/Libraries/LibJS/Runtime/Temporal/CalendarPrototype.cpp index b0b09084f9..a2242afc36 100644 --- a/Userland/Libraries/LibJS/Runtime/Temporal/CalendarPrototype.cpp +++ b/Userland/Libraries/LibJS/Runtime/Temporal/CalendarPrototype.cpp @@ -572,15 +572,15 @@ JS_DEFINE_NATIVE_FUNCTION(CalendarPrototype::merge_fields) // 2. Perform ? RequireInternalSlot(calendar, [[InitializedTemporalCalendar]]). auto* calendar = TRY(typed_this_object(global_object)); - // 3. Assert: calendar.[[Identifier]] is "iso8601". - VERIFY(calendar->identifier() == "iso8601"sv); - - // 4. Set fields to ? ToObject(fields). + // 3. Set fields to ? ToObject(fields). auto* fields = TRY(vm.argument(0).to_object(global_object)); - // 5. Set additionalFields to ? ToObject(additionalFields). + // 4. Set additionalFields to ? ToObject(additionalFields). auto* additional_fields = TRY(vm.argument(1).to_object(global_object)); + // 5. Assert: calendar.[[Identifier]] is "iso8601". + VERIFY(calendar->identifier() == "iso8601"sv); + // 6. Return ? DefaultMergeCalendarFields(fields, additionalFields). return TRY(default_merge_calendar_fields(global_object, *fields, *additional_fields)); }