RFR: 8286983: Add mention of "Preview Feature" for new loom related jdb and debug agent options

Chris Plummer cjplummer at openjdk.java.net
Thu May 19 15:32:58 UTC 2022


On Thu, 19 May 2022 13:10:50 GMT, Alan Bateman <alanb at 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


More information about the serviceability-dev mailing list