Skip to content

fix: preserve braces in Javadoc code tags - #1218

Open
efegokdemir wants to merge 3 commits into
apache:masterfrom
efegokdemir:codex/issue-823-curly
Open

efegokdemir wants to merge 3 commits into
apache:masterfrom
efegokdemir:codex/issue-823-curly

Conversation

@efegokdemir

Copy link
Copy Markdown

Summary

Preserve closing braces inside {@code ...} content when legacy Javadoc tags are converted to XHTML. This prevents expressions such as ${project.basedir} from prematurely terminating the inline tag.

Changes

  • Allow balanced braces in the Javadoc inline-tag matcher.
  • Add a regression test covering nested braces in a multiline code block.

Testing

  • mvn -pl maven-plugin-tools-generators -am -Dspotless.apply.skip=true -Dspotless.check.skip=true -Dcheckstyle.skip=true -Drat.skip=true test — passed (61 API tests; 18 generator tests, 1 skipped)
  • git diff --check — passed

spotless:check was attempted but is blocked by the local JDK/Palantir formatter incompatibility (NoSuchFieldError in JCTree$JCCompilationUnit).

Fixes #823

@slachiewicz slachiewicz added the bug Something isn't working label Oct 29, 2026
Track balanced braces and escaped delimiters when decoding legacy inline tags. Add a regression covering nested property expressions and adjacent Javadoc tags.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[MPLUGIN-513] Closing curly brace in code block rendered incorrectly

2 participants