<i18n dev> RFR: 8280470: Confusing instanceof check in HijrahChronology.range

Naoto Sato naoto at openjdk.java.net
Fri Jan 21 17:29:43 UTC 2022


On Mon, 17 Jan 2022 21:02:35 GMT, Andrey Turbanov <aturbanov at openjdk.org> wrote:

> Parameter `ChronoField field` is checked by `if (field instanceof ChronoField)`. Such check is confusing, because only one case, when this could be `false` is when `field == null`.
> But if condition is not satisfied we will get immediate NullPointerException anyway in `return field.range();`.

Marked as reviewed by naoto (Reviewer).

-------------

PR: https://git.openjdk.java.net/jdk/pull/7118


More information about the i18n-dev mailing list