-
Notifications
You must be signed in to change notification settings - Fork 382
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Java 9, 10, 11 JRE Emulation #9547
Comments
As this seems to be the only jre+java9 issue so far, adding this too: http://openjdk.java.net/jeps/269 |
Additions so far: java.util.List
java.util.Map
java.util.Set
java.util.stream.Stream
java.util.stream.Collectors
Anything I'm missing? |
java.util.Enumeration
java.util.Optional
|
I'm going to try to help with this missing emulation. I have made a pretty trivial PR just as starting point, and also, hehe it is an interface that I'm interested in 😉a reactive interface in the JRE. |
Awesome! I've got the ones in my comment all written (and tests for about half of them), will try to get that batch up for review soon. The hard part at the moment is getting the tests to actually compile, as that requires using a java10+ jdk, which our current set of build scripts can't run on yet. I've got this fork which refactors everything into modules (for use in maven, but gradle would be just as easy), maybe its time to pick that back up and finish it off... |
Bug: gwtproject#9547 Bug-Link: gwtproject#9547 Change-Id: I0e01ee3737ebee118db2b2e023cbe02beb9f2347
Bug: #9547 Bug-Link: gwtproject/gwt#9547 Change-Id: I0e01ee3737ebee118db2b2e023cbe02beb9f2347
Missed API from Java 11 java.util.Optional
|
There is my PR https://gwt-review.googlesource.com/c/gwt/+/22740 for Optional.isEmpty |
@axls do I need something more to merge it into master branch? |
@abashev PR should contains tests at least. But, IIUC, currently there is no final decision how to execute tests for Java 9+ (https://groups.google.com/g/google-web-toolkit-contributors/c/eyBpbrtYgyY). |
See earlier review at https://gwt-review.googlesource.com/c/gwt/+/21501. Fixes gwtproject#9547
See earlier review at https://gwt-review.googlesource.com/c/gwt/+/21501. Fixes gwtproject#9547
See earlier review at https://gwt-review.googlesource.com/c/gwt/+/21501. Fixes gwtproject#9547
See earlier review at https://gwt-review.googlesource.com/c/gwt/+/21501. Fixes gwtproject#9547
See earlier review at https://gwt-review.googlesource.com/c/gwt/+/21501. Fixes gwtproject#9547
See earlier review at https://gwt-review.googlesource.com/c/gwt/+/21501. Build changes get us ready for missing Java 10 and 11 emulation as well. Partial #9547
Reopening, only the first chunk of this is merged. |
Partial #9547 Co-authored-by: Alexey Abashev <[email protected]>
With the last PRs submitted for review, I think everything here is covered? |
|
Thanks @natros, I don't think that made any of the lists I'd seen so far. Want to propose a patch? |
I can try it over the weekend. |
@niloc132 How far do you want to go? All reasonable to emulate method additions of classes we already emulate? |
@jnehlmeier when we started, it was from a list you had made as I recall, and were picking out existing classes that were missing methods/etc. My goal for 2.11 was to at least finish off bringing those existing types up to Java 11 compatibility, so that 2.12 can drop Java 8 support (at least for compiling to JS) and target Java 17 features, then backfill from there on emulation. From my perspective, with these closed, we should finalize any PRs that are going to get more testing, and then cut a release, and then bump JDT and Jetty to whatever we can reasonably manage. But I'd be happy closing the door on this particular issue, and following up with specific other emulation later. |
@niloc132 Ok, then we are still missing some stuff in existing classes. But curating a list manually is always pretty time consuming. Maybe we should develop a tool that automates it. I once did a diff using Jdiff but because that tool is so old it was pretty annoying to get it running. Also the output is far from being instantly usable. Based on that diff I started jnehlmeier#15 but it is far from complete and also outdated right now. |
Add missing `not` method to Predicate to be java 11 compatible. Partial #9547
With completion of #9860, which as far as I can tell covers the remaining items mentioned in this ticket, I'm going to close this. We can open other tickets to list additional missing APIs, but there are tons of PRs already tied to this, and all closed for the moment. |
Java 10 added copyOf static factory methods to produce immutable List/Set/Map instances that contain no nulls. Follow-up #9547
Just for the future java 9 support in GWT.
https://blog.codefx.org/java/java-9-stream/
The text was updated successfully, but these errors were encountered: