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();`.
------------- Commit messages: - [PATCH] Remove confusing instanceof check in HijrahChronology.range - [PATCH] Remove confusing instanceof check in HijrahChronology.range Changes: https://git.openjdk.java.net/jdk/pull/7118/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=7118&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8280470 Stats: 13 lines in 1 file changed: 0 ins; 4 del; 9 mod Patch: https://git.openjdk.java.net/jdk/pull/7118.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/7118/head:pull/7118 PR: https://git.openjdk.java.net/jdk/pull/7118
