[JSC] Temporal.PlainDate should validate input range
Created attachment 452687 [details] Patch
Comment on attachment 452687 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=452687&action=review > JSTests/ChangeLog:3 > + [JSC] Temporal.PlainDate should validate input range I think we’d want the test for the last valid as well as the first invalid.
Comment on attachment 452687 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=452687&action=review >> JSTests/ChangeLog:3 >> + [JSC] Temporal.PlainDate should validate input range > > I think we’d want the test for the last valid as well as the first invalid. Sounds good, added.
Created attachment 452776 [details] Patch
Created attachment 452778 [details] Patch
Created attachment 452781 [details] Patch
Comment on attachment 452781 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=452781&action=review review+ assuming the tests pass > JSTests/stress/temporal-plaindate.js:114 > +new Temporal.PlainDate(0x43530, 9, 13); Maybe a shouldBe about the string form? > JSTests/stress/temporal-plaindate.js:116 > +new Temporal.PlainDate(-0x425cd, 4, 20); Ditto.
Comment on attachment 452781 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=452781&action=review Thank you! >> JSTests/stress/temporal-plaindate.js:114 >> +new Temporal.PlainDate(0x43530, 9, 13); > > Maybe a shouldBe about the string form? Done. >> JSTests/stress/temporal-plaindate.js:116 >> +new Temporal.PlainDate(-0x425cd, 4, 20); > > Ditto. Done.
Committed r290282 (?): <https://commits.webkit.org/r290282>
<rdar://problem/89268246>