RFR: JDK-8223914: specification of j.l.c.MethodTypeDesc::of should document better the exceptions thrown
Roger.Riggs at oracle.com
Mon May 20 18:00:45 UTC 2019
In the CSR, the Summary should be about the change...
"...MethodTypeDesc::of should document all exceptions.
Avoid duplication between Summary and Problem.
I would omit the part about "content of parameter" or "its contents" is
It cannot happen and if it does, its more of an internal error than a
since it should not be possible to create a ClassDesc with a null
In the Code Review:
MethodTypeDesc.java: 68: as above, "or its contents are" -> "is "
there's no need to mention the contents.
MethodTypeDescTest.java: 264 and 274. The messages would be more useful
(if they ever were to happen)
and for the person reading the code if they describe what should not happen.
For example, "ClassDesc array should not be null" or ClassDesc should
not be null.
On 05/17/2019 12:55 PM, Vicente Romero wrote:
> Please review simple fix for  at  plus the CSR at . This fix
> is simply documenting all the missing cases in which method
> java.lang.constant.MethodTypeDesc::of can throw exceptions. A test has
> been added to cover the missing cases.
>  https://bugs.openjdk.java.net/browse/JDK-8223914
>  http://cr.openjdk.java.net/~vromero/8223914/webrev.00/
>  https://bugs.openjdk.java.net/browse/JDK-8224136
More information about the core-libs-dev