Skip to content

Fix broken Javadoc tags#26967

Closed
izeye wants to merge 1 commit intospring-projects:mainfrom
izeye:fix-javadoc-tags
Closed

Fix broken Javadoc tags#26967
izeye wants to merge 1 commit intospring-projects:mainfrom
izeye:fix-javadoc-tags

Conversation

@izeye
Copy link
Copy Markdown
Contributor

@izeye izeye commented May 22, 2021

This PR fixes broken Javadoc tags.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged or decided on label May 22, 2021
@sbrannen sbrannen self-assigned this May 22, 2021
@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 May 22, 2021
@sbrannen sbrannen added this to the 5.3.8 milestone May 22, 2021
@sbrannen
Copy link
Copy Markdown
Member

Out of curiosity, how did you notice all of those broken tags?


Regarding the Javadoc for org.springframework.asm.ClassWriter, the following is our policy on changes to the ASM code.

Please refrain from modifying classes under org.springframework.asm, org.springframework.cglib, and org.springframework.objenesis. Those include repackaged forks of the third-party libraries ASM, CGLIB, and Objenesis. Any refactoring to those classes should take place upstream in the originating repository. The Spring Framework will then pick up the changes when syncing with official updates of the forked third-party libraries.

So I'll simply revert that change when merging.

@sbrannen sbrannen closed this in ab2c78a May 23, 2021
@sbrannen
Copy link
Copy Markdown
Member

This has been merged into main.

Thanks

@izeye izeye deleted the fix-javadoc-tags branch May 23, 2021 23:16
@izeye
Copy link
Copy Markdown
Contributor Author

izeye commented May 24, 2021

@sbrannen Thanks for the review and spotting the accidental change on the ASM module.

Out of curiosity, how did you notice all of those broken tags?

I just found one and then searched to fix similar occurrences.

@izeye
Copy link
Copy Markdown
Contributor Author

izeye commented May 24, 2021

I created https://gitlab.ow2.org/asm/asm/-/merge_requests/321 to fix it in the upstream module.

lxbzmy pushed a commit to lxbzmy/spring-framework that referenced this pull request Mar 26, 2022
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.

3 participants