On Thu, 29 Apr 2021 08:00:18 GMT, Hannes Wallnöfer <hann...@openjdk.org> wrote:
> I don't know why this test was disabled because of JDK-8146022, in fact I > wasn't able to understand what JDK-8146022 was all about as I didn't find a > commit for it. However, the test runs fine when updated to the current output > format, so I guess whatever bug caused it to fail has been fixed in the > meantime. I also removed some vestiges of an old test framework nearby that > has been commented out from time immemorial. FWIW, the way I see the history here ... a bug was filed because of a resource issue, and this test was disabled because of the bug. Using `@ignore` was a precursor/alternate to using the problem list file. Separately, refactoring and cleanup fixed the resource issue, using separate independent cleanup RFEs. So, JDK-8146022 was closed (without needing a changeset), and without removing the `@ignore` here. Lines 49-51 are a reminder of life before `JavadocTester`. ------------- Marked as reviewed by jjg (Reviewer). PR: https://git.openjdk.java.net/jdk/pull/3791