Skip to content

Conversation

@akurtakov
Copy link
Member

No description provided.

Reduce chances for surprises at runtime.
It's already transitive dependency through Lucene.
In Java 21 Thread.stop simply throws UnsupportedOperationException and
ThreadDeath was thrown only from that method in older JVM versions.
@akurtakov akurtakov self-assigned this Dec 17, 2024
@merks
Copy link
Contributor

merks commented Dec 17, 2024

This all makes good sense. Thanks. 👍

@github-actions
Copy link
Contributor

Test Results

 1 755 files  ±0   1 755 suites  ±0   1h 30m 27s ⏱️ -26s
 4 170 tests ±0   4 148 ✅ ±0   22 💤 ±0  0 ❌ ±0 
13 107 runs  ±0  12 943 ✅ ±0  164 💤 ±0  0 ❌ ±0 

Results for commit 3f20ca2. ± Comparison against base commit 72453fe.

@akurtakov
Copy link
Member Author

There are 20 new code warnings - all about deprecated URL methods/constructors usage which have been around since Java 20 but not reported due to older JVM.
Pushing despite the new warnings as this makes the build reports inline with actual state.

@akurtakov akurtakov merged commit 9636c7e into eclipse-platform:master Dec 17, 2024
14 of 17 checks passed
@merks
Copy link
Contributor

merks commented Dec 17, 2024

FYI, note that changing the URL methods being called to fix the deprecation warnings is really dangerous. The handling of error cases is different. You can create a URL with a space, but you cannot create a URI with a space...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants