[8u backport] RFR(XS): 8051803 [parfait] warnings from b114 for hotspot.src.share.vm

Vladimir Kozlov vladimir.kozlov at oracle.com
Wed Jul 23 20:55:00 UTC 2014


Changes are good.

David, you should use only original bug id in Subject and in the request 
text (nobody care about backport bug id). Also you have to point jdk9 
changeset:

http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/de9cd35a3d1f

And you should say if you need to change the fix for 8u or it is applied 
(hg export/import) cleanly. And backport it using hg export/import.

Thanks,
Vladimir

On 7/23/14 12:43 PM, David Chase wrote:
> Summary: it’s a parfait warning, where convoluted and perhaps delicate logic ensures that a NULL could “never” appear.
> Fix is to insert a null check and return the “nothing here” result (false) which (a) should never occur anyhow and
> (b) is safe and consistent behavior.
>
> webrev: http://cr.openjdk.java.net/~drchase/8051803/webrev.00/
>
> bug: https://bugs.openjdk.java.net/browse/JDK-8051803
>
> backport of: https://bugs.openjdk.java.net/browse/JDK-8028037
>
> old review discussion: http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2014-March/013684.html
>
> testing: local jtreg
>
> David
>


More information about the hotspot-compiler-dev mailing list