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.