RFR: 8265418: Clean-up redundant null-checks of Class.getPackageName()

Сергей Цыпанов github.com+10835776+stsypanov at openjdk.java.net
Mon Apr 26 08:00:40 UTC 2021


On Mon, 19 Apr 2021 14:45:58 GMT, Alan Bateman <alanb at openjdk.org> wrote:

>> As discussed in https://github.com/openjdk/jdk/pull/3464 we can clean-up null-checks remaining after [8142968](https://bugs.openjdk.java.net/browse/JDK-8142968) as Class.getPackageName() never returns null.
>
> src/java.base/share/classes/java/util/concurrent/atomic/AtomicIntegerFieldUpdater.java line 458:
> 
>> 456:         private static boolean isSamePackage(Class<?> class1, Class<?> class2) {
>> 457:             return class1.getClassLoader() == class2.getClassLoader()
>> 458:                    && class1.getPackageName().equals(class2.getPackageName());
> 
> If the j.u.c.atomic code is changed then it will need to be coordinated with Doug Lea's repo. The rest of the changes are good and thanks for following up from the comments in the other issue.

@AlanBateman if DL is not responding, will it be ok to just revert the changes related to `java.util.concurrent`?

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

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


More information about the core-libs-dev mailing list