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

Reply via email to