Skip to content

Remove unnecessary closing curly brackets in Javadoc#34679

Closed
ngocnhan-tran1996 wants to merge 1 commit into
spring-projects:mainfrom
ngocnhan-tran1996:remove-obsolete-braces
Closed

Remove unnecessary closing curly brackets in Javadoc#34679
ngocnhan-tran1996 wants to merge 1 commit into
spring-projects:mainfrom
ngocnhan-tran1996:remove-obsolete-braces

Conversation

@ngocnhan-tran1996

Copy link
Copy Markdown
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
/**
* 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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: documentation A documentation task

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants