Browse Source

LibJS: Simplify TemporalInstantString

This is an editorial change in the Temporal spec.

See: https://github.com/tc39/proposal-temporal/commit/271d71c
Linus Groh 3 năm trước cách đây
mục cha
commit
f1dafabbb9

+ 4 - 10
Userland/Libraries/LibJS/Runtime/Temporal/ISO8601.cpp

@@ -1145,19 +1145,13 @@ bool ISO8601Parser::parse_duration()
 bool ISO8601Parser::parse_temporal_instant_string()
 bool ISO8601Parser::parse_temporal_instant_string()
 {
 {
     // TemporalInstantString :
     // TemporalInstantString :
-    //     Date TimeZoneOffsetRequired
-    //     Date DateTimeSeparator TimeSpec TimeZoneOffsetRequired
+    //     Date TimeSpecSeparator[opt] TimeZoneOffsetRequired
     StateTransaction transaction { *this };
     StateTransaction transaction { *this };
     if (!parse_date())
     if (!parse_date())
         return false;
         return false;
-    if (!parse_time_zone_offset_required()) {
-        if (!parse_date_time_separator())
-            return false;
-        if (!parse_time_spec())
-            return false;
-        if (!parse_time_zone_offset_required())
-            return false;
-    }
+    (void)parse_time_spec_separator();
+    if (!parse_time_zone_offset_required())
+        return false;
     transaction.commit();
     transaction.commit();
     return true;
     return true;
 }
 }