On Tue, 22 Sep 2026 17:59:01 GMT, sbracely <[email protected]> wrote: >> src/java.base/share/classes/java/time/chrono/HijrahChronology.java line 948: >> >>> 946: >>> 947: if (length < 29 || length > 32) { >>> 948: throw new IllegalArgumentException("Invalid month >>> length in year: " + year); >> >> An even better message would call out the month and month length, but it is >> pretty trivial to identify those once the correct year is emitted, so it is >> fine as is too. > > Thanks. I'll update the message to include both the invalid length and the > year.
Updated to include year, month, and length. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/33015#discussion_r4075458113
