RFR: JDK-8298405: Support Markdown in Documentation Comments [v38]
Jonathan Gibbons
jjg at openjdk.org
Thu Feb 22 20:52:10 UTC 2024
On Thu, 22 Feb 2024 19:17:09 GMT, Jonathan Gibbons <jjg at openjdk.org> wrote:
>>>I think in this place we also need to check for indented code blocks, since we reduced currIndent and may have 4 or more character indentation compared to the new currIndent value.
>>
>> This seems wrong. Surely, we should not reduce the indentation and then for the same line in the comment check for an extra-indented line. Or else I am missing something.
>>
>> That being said, thanks for the test case. I will investigate it.
>
> Update: I see and understand your test case.
Update: I found that more elegant solution you hinted at, by refactoring the method from nested `if` statements to a series of checks with early returns.
The test case you provided now works as expected.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/16388#discussion_r1499918026
More information about the build-dev
mailing list