On Mon, 8 Jan 2024 20:16:50 GMT, Chris Plummer <[email protected]> wrote:
> Fix some broken links and other minor cleanups:
>
> There are a few broken links in the javadoc for ObjectReference.java. For
> example:
>
>
> * Not all target VMs support this operation. See
> * VirtualMachine#canGetMonitorInfo to determine if the
> * operation is supported.
>
>
> Which should be:
>
>
> * Not all target VMs support this operation. See
> * {@link VirtualMachine#canGetMonitorInfo} to determine if the
> * operation is supported.
>
>
>
> The following does not need to include the link text:
>
> ` * {@link VirtualMachine#canGetInstanceInfo() canGetInstanceInfo()}`
>
> Instead just use:
>
> ` * {@link VirtualMachine#canGetInstanceInfo()}`
>
>
> There are many places where we embed a "see" rather than use @see. The
> formatting convention is:
>
> ` * @throws VMCannotBeModifiedException if the VirtualMachine is
> read-only - see {@link VirtualMachine#canBeModified()}.`
>
> However, there are a few places that are missing a space before the "see":
>
>
> * @throws VMCannotBeModifiedException if the VirtualMachine is read-only
> * -see {@link VirtualMachine#canBeModified()}.
I could use one more review. Thanks!
-------------
PR Comment: https://git.openjdk.org/jdk/pull/17311#issuecomment-1886029788
