Skip to content

Guard requestBlankLine in writeTableOpen, writePreOpen, and writeBlockquoteOpenOrClose of JavadocWriter.#1409

Merged
copybara-service[bot] merged 1 commit into
masterfrom
test_945714394
Jul 10, 2026
Merged

Guard requestBlankLine in writeTableOpen, writePreOpen, and writeBlockquoteOpenOrClose of JavadocWriter.#1409
copybara-service[bot] merged 1 commit into
masterfrom
test_945714394

Conversation

@copybara-service

Copy link
Copy Markdown

Guard requestBlankLine in writeTableOpen, writePreOpen, and writeBlockquoteOpenOrClose of JavadocWriter.

  • Prevent inserting leading blank lines (///) before <table>, <pre>, and <blockquote> when they appear as the very first token in a Javadoc comment by checking if (wroteAnythingSignificant) before requesting a blank line.
  • Add tableAtStartOfComment(), preAtStartOfComment(), and blockquoteAtStartOfComment() unit tests in JavadocFormattingTest.
  • Add tableInHtmlListItem() and tableInsideMarkdownListItemAfterText() unit tests in JavadocFormattingTest to document existing list-item table spacing behavior (Thread 4 discussion).

…kquoteOpenOrClose of JavadocWriter.

* Prevent inserting leading blank lines (`///`) before `<table>`, `<pre>`, and `<blockquote>` when they appear as the very first token in a Javadoc comment by checking `if (wroteAnythingSignificant)` before requesting a blank line.
* Add `tableAtStartOfComment()`, `preAtStartOfComment()`, and `blockquoteAtStartOfComment()` unit tests in `JavadocFormattingTest`.
* Add `tableInHtmlListItem()` and `tableInsideMarkdownListItemAfterText()` unit tests in `JavadocFormattingTest` to document existing list-item table spacing behavior (Thread 4 discussion).

PiperOrigin-RevId: 945745386
@copybara-service copybara-service Bot merged commit 77bee2d into master Jul 10, 2026
@copybara-service copybara-service Bot deleted the test_945714394 branch July 10, 2026 16:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant