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