IsValidEraYearForCalendar: Don't check for minimum/maximum era year?

[anba]

This is supposed to work:

Temporal.PlainDate.from({calendar: "gregory", era: "ce", eraYear: -1, month: 1, day: 1}).toString()

But if I read IsValidEraYearForCalendar and NonISOCalendarDateToISO correctly, the current spec text requires a RangeError to be thrown from NonISOCalendarDateToISO?

NonISOCalendarDateToISO, step 1:

1. If fields.[[Era]] and fields.[[EraYear]] are not unset, then
       a. If IsValidEraYearForCalendar(calendar, fields.[[Era]], fields.[[EraYear]]) is false, throw a RangeError exception.

IsValidEraYearForCalendar, steps 4-7:

4. Let min be the value given in the "Minimum eraYear" column of r.
5. Let max be the value given in the "Maximum eraYear" column of r.
6. If eraYear < min, return false.
7. If eraYear > max, return false.

[Manishearth]

Hmm, yeah, I'm surprised by this line.

We've had consensus elsewhere that "out of range" values for eras are always fine, and even necessary to support Japanese;

See the second note on https://tc39.es/proposal-temporal/#sec-temporal-nonisoresolvefields

[ptomato]

Pretty sure this was a remnant of the original stage 2 text that I overlooked. Sorry about that!

[sffc]

Yeah, the spec is quite clear about era year being checked for being in range, but this doesn't align when my recollection of the intended behavior.