On Thu, 19 May 2022 13:10:50 GMT, Alan Bateman <al...@openjdk.org> wrote:
>> As part of the loom integration, jdb added -trackvthreads and the debug >> agent added "enumeratevthreads". The help text for these options should call >> out that they are Preview Features. > > src/jdk.jdwp.agent/share/native/libjdwp/debugInit.c line 880: > >> 878: "onuncaught=y|n debug on any uncaught? >> n\n" >> 879: "timeout=<timeout value> for listen/attach in milliseconds >> n\n" >> 880: "enumeratevthreads=y|n (Preview Feature) thread lists >> include all vthreads\n" > > I think the description needs to say "virtual threads" rather than > "vthreads". Also I think "(Preview" needs to go after "virtual threads" to > make it clearer that it's virtual threads that are preview,. I was really trying to emphasize that the flag is preview, and therefore may change or go away. If I change as you suggest, the reader could imply that as long as virtual threads support is in place, then the flag will be also (and in its current form). ------------- PR: https://git.openjdk.java.net/jdk/pull/8780