On Tue, 23 Jan 2024 05:21:17 GMT, psoujany <d...@openjdk.org> wrote:

>> @psoujany just to make sure we're on the same page, you'll also update the 
>> tests required by this change?
>
> @hns I've modified regression tests with the code changes and see langtools 
> jtreg tests passing with the new fixes. Could you please review this PR. 
> Thank you.

@psoujany I still get two failing tests, possibly because I updated my local 
repository by merging the current master branch  before running tests. This PR 
is based on a very outdated branch, so we should definitely merge the master 
branch before integrating. The failing tests I get on my up-to-date branch are 
these:

jdk/javadoc/doclet/testPreview/TestPreview.java
jdk/javadoc/doclet/testSpecTag/TestSpecTag.java

-------------

PR Comment: https://git.openjdk.org/jdk/pull/16148#issuecomment-1917027809

Reply via email to