Virtual threads debug view update + JdiThread new API + unit tests

[SougandhS]

What it does

Enhancement for #296
Provides better view for Virtual threads in debug view
before -

After -

How to test

Run the test or create an ibuild with it and try (Wont be able to see in child eclipse - existing issue)

Author checklist

• I have thoroughly tested my changes
• The change is following the coding conventions
• I have signed the Eclipse Contributor Agreement (ECA)

[SougandhS]

Hi @jukzi
Could you please check on this ?

[jukzi]

I am afk 2w. Please wait or ask some1 else. Jörg Kubitz

…

Am 14.10.2024 um 11:21 schrieb Sougandh S ***@***.***>:  Hi @jukzi Could you please check on this ? — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.

[SougandhS]

Hi @iloveeclipse
If you are available, could you pls review this PR ?

[fdummert]

As commented in #345, this PR appears to solve the debug issues we had with virtual threads (and obviously improves the presentation of virtual threads significantly).

[SougandhS]

Thank you @jukzi & @fdummert

[iloveeclipse]

@SougandhS : please next time provide appropriate commit message that also contains a link to the original issue.
See https://github.com/eclipse-jdt/.github/blob/main/CONTRIBUTING.md#commit-message-recommendations

[SougandhS]

@SougandhS : please next time provide appropriate commit message that also contains a link to the original issue. See https://github.com/eclipse-jdt/.github/blob/main/CONTRIBUTING.md#commit-message-recommendations

Yes @iloveeclipse, will provide as per the contribution guide from now on. Thank you for pointing it out

[stephan-herrmann]

Repeating my question from #598: the test added here uses a Java project with preview enabled (originally 23, then updated to 24). But at this time virtual threads are no longer preview. Hence shouldn't the test use a Java project without --enable-preview?

[jukzi]

sounds ok to me. @SougandhS WDYT?

[SougandhS]

@stephan-herrmann Yes it wont need preview.

[SougandhS]

sounds ok to me. @SougandhS WDYT?

Agreeing with you