Checking instanceof in DualPivotQuicksort
Vladimir Yaroslavskiy
vlv.spb.ru at mail.ru
Thu Oct 24 12:11:21 UTC 2019
Hi Brent,
Looking at coverage of DualPivotQuicksort class, I found that
cases of unexpected type of given array are not invoked at all
(mentioned in my previous email).
I think that these "else" can be removed, because method
compute() of private classes are invoked from DualPivotQuicksort
class only and we can guarantee valid array types (int[], .. , double[]).
If you agree, could you please, remove such cases?
Replace lines 4147-4148:
throw new IllegalArgumentException(
"Unknown type of array: " + a.getClass().getName());
by return null;
remove lines 4099-4101:
} else {
throw new IllegalArgumentException(
"Unknown type of array: " + dst.getClass().getName());
and remove lines 4030-4033:
} else {
throw new IllegalArgumentException(
"Unknown type of array: " + a.getClass().getName());
-----
And doe the same for Sorting class: remove 12 cases
fail("Unknown type of array: " + ....
Thank you,
Vladimir
More information about the core-libs-dev
mailing list