Skip to content

Remove unnecessary closing curly brackets in Javadoc #34679

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Conversation

ngocnhan-tran1996
Copy link
Contributor

No description provided.

Signed-off-by: Tran Ngoc Nhan <ngocnhan.tran1996@gmail.com>
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged or decided on label Mar 29, 2025
@sbrannen sbrannen self-assigned this Mar 29, 2025
@sbrannen sbrannen added type: documentation A documentation task and removed status: waiting-for-triage An issue we've not yet triaged or decided on labels Mar 29, 2025
@sbrannen sbrannen added this to the 6.2.6 milestone Mar 29, 2025
@sbrannen sbrannen changed the title Remove obsolete braces in javadocs Remove unnecessary closing curly brackets in Javadoc Mar 29, 2025
@@ -1473,7 +1473,7 @@ private static final class LabelEntry {

/**
* Another entry (and so on recursively) having the same hash code (modulo the size of {@link
* SymbolTable#labelEntries}}) as this one.
* SymbolTable#labelEntries}) as this one.
Copy link
Member

Choose a reason for hiding this comment

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

Please note that this class belongs to our fork of the ASM project.

Consequently, we do not modify the Javadoc. Changes to ASM Javadoc should be made in the upstream project.

Thus, I will revert that when merging this PR.

Copy link
Contributor

Choose a reason for hiding this comment

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

sbrannen added a commit that referenced this pull request Mar 29, 2025
This commit also reverts the change to ASM's SymbolTable class.

See gh-34679
@sbrannen sbrannen closed this in 30fcaef Mar 29, 2025
@sbrannen
Copy link
Member

This has been merged in 30fcaef and revised in 9fd1d0c.

Thanks

@ngocnhan-tran1996 ngocnhan-tran1996 deleted the remove-obsolete-braces branch March 29, 2025 13:10
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
type: documentation A documentation task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants