On Thu, 9 Apr 2026 10:36:44 GMT, Alan Bateman <[email protected]> wrote:
> > The new methods should be added only when we start to use them in the
> > classes extended ThreadWrapper. I hope that new tests will use more
> > TestThreadFactory (or at least new Thread(Runnable runnable)) and not
> > extending Thread class.
>
> Do you foresee usage beyond the proposed usages in the vmTestbase/nsk tests?
The
`rg 'extends Thread' | wc`
returns 600 classes for vmTestbase only.
and
rg 'extends Thread' | wc
715 for all tests
So it is quite common pattern (anti-pattern) in vmtestbase and less common in
the jtreg tests.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/30591#issuecomment-4216321663