RFR: JDK-8223914: specification of j.l.c.MethodTypeDesc::of should document better the exceptions thrown
Roger Riggs
Roger.Riggs at oracle.com
Mon May 20 18:00:45 UTC 2019
Hi Vicente,
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
null;
It cannot happen and if it does, its more of an internal error than a
regular NPE
since it should not be possible to create a ClassDesc with a null
descriptor string.
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.
Thanks, Roger
On 05/17/2019 12:55 PM, Vicente Romero wrote:
> Please review simple fix for [1] at [2] plus the CSR at [3]. 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.
>
> Thanks,
> Vicente
>
> [1] https://bugs.openjdk.java.net/browse/JDK-8223914
> [2] http://cr.openjdk.java.net/~vromero/8223914/webrev.00/
> [3] https://bugs.openjdk.java.net/browse/JDK-8224136
More information about the core-libs-dev
mailing list