Skip to content
This repository was archived by the owner on Oct 1, 2025. It is now read-only.

8201793: (ref) Reference object should not support cloning #6

Closed
wants to merge 3 commits into from
Closed

8201793: (ref) Reference object should not support cloning #6

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
365622e
8201793: (ref) Reference object should not support cloning
poonamparhar Apr 20, 2022
a096694
Fixed the javadoc comments
poonamparhar Apr 22, 2022
2014116
fixed merge conflict
poonamparhar Apr 28, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 3 additions & 1 deletion jdk/src/share/classes/java/lang/ref/Reference.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -249,7 +249,9 @@ public boolean enqueue() {
* Throws {@link CloneNotSupportedException}. A {@code Reference} cannot be
* meaningfully cloned. Construct a new {@code Reference} instead.
*
* @returns never returns normally
* @apiNote This method is defined in Java SE 8 Maintenance Release 4.
*
* @return never returns normally
* @throws CloneNotSupportedException always
*
* @since 8
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This method is only present in JDK 8u42 (RI) and 8u352 and later releases. It's not in existing 8u releases >= 8u45 and < 8u352. Neither 8 or 8u42 would cause confusion. A suggestion is to use @apiNote to indicate which MR this method is defined:

@apiNote This method is defined in Java SE 8 Maintenance Release 4.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The proposed @apinote looks good.

Expand Down