RFR (S) 8073479: Replace obj.getClass hacks with Objects.requireNonNull
Aleksey Shipilev
aleksey.shipilev at oracle.com
Thu Feb 19 15:02:31 UTC 2015
Hi Peter,
On 02/19/2015 05:59 PM, Peter Levart wrote:
> On 02/19/2015 03:51 PM, Aleksey Shipilev wrote:
>> Hi,
>>
>> Please review and sponsor this cleanup fix:
>> http://cr.openjdk.java.net/~shade/8073479/webrev.00/
>> https://bugs.openjdk.java.net/browse/JDK-8073479
>>
>> Summary: use Objects.requireNonNull instead of object.getClass to check
>> for nullity.
>>
>> Testing:
>> - jdk9-dev build on Linux x86_64/release
>> - JPRT (modulo some freetype and javac failures)
>>
>> Thank you,
>> -Aleksey.
>>
>> P.S. I am going to submit another one against langtools, should it be
>> reviewed on this list as well?
>>
>
> Hi Aleksey,
>
> Is profile pollution problem already fixed? Can Objects.requireNonNull
> be used in performance critical code without fear of performance
> regression?
What profile pollution? Objects.requireNonNull is a static method. See
also the link in bug description:
http://cr.openjdk.java.net/~shade/scratch/NullChecks.java
-Aleksey.
More information about the core-libs-dev
mailing list