RFR: JDK-8224158: assertion related to NPE at DynamicCallSiteDesc::withArgs should be reworded

Vicente Romero vicente.romero at oracle.com
Wed May 29 14:49:34 UTC 2019


Please review fix for [1] at [2] and the CSR at [3]. This is a simple 
fix that is just rewording an assertion at method 
java.lang.constant.DynamicCallSiteDesc::withArgs. The fix is simply:

diff -r dd321e3596c0 -r 78f3b29fb255 
src/java.base/share/classes/java/lang/constant/DynamicCallSiteDesc.java
--- 
a/src/java.base/share/classes/java/lang/constant/DynamicCallSiteDesc.java 
Wed May 29 13:58:05 2019 +0100
+++ 
b/src/java.base/share/classes/java/lang/constant/DynamicCallSiteDesc.java 
Wed May 29 09:14:55 2019 -0400
@@ -156,7 +156,7 @@
       *                      to the bootstrap, that would appear in the
       *                      {@code BootstrapMethods} attribute
       * @return the nominal descriptor
-     * @throws NullPointerException if any parameter is null
+     * @throws NullPointerException if its argument or its contents are 
{@code null}
       */
      public DynamicCallSiteDesc withArgs(ConstantDesc... bootstrapArgs) {
          return new DynamicCallSiteDesc(bootstrapMethod, 
invocationName, invocationType, bootstrapArgs);

Thanks,
Vicente

[1] https://bugs.openjdk.java.net/browse/JDK-8224158
[2] http://cr.openjdk.java.net/~vromero/8224158/webrev.00/
[3] https://bugs.openjdk.java.net/browse/JDK-8224985


More information about the core-libs-dev mailing list