-
Notifications
You must be signed in to change notification settings - Fork 93
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
Fixdeprecations #1686
Fixdeprecations #1686
Conversation
bc7fea8
to
9e3d2a7
Compare
Hi @jjohnstn , when you find the time could you have a look? While we reduced the amount of warnings with the last patch by half we are still one of the projects with the highest amount of deprecations. If you are unsure about one or another change feel free to leave it out this time. |
Hi @carstenartur Will look at it today. |
...e.manipulation/common/org/eclipse/jdt/internal/core/manipulation/JavaManipulationPlugin.java
Outdated
Show resolved
Hide resolved
org.eclipse.jdt.ui/ui/org/eclipse/jdt/internal/ui/JavaPlugin.java
Outdated
Show resolved
Hide resolved
org.eclipse.jdt.ui/ui/org/eclipse/jdt/internal/ui/JavaPlugin.java
Outdated
Show resolved
Hide resolved
755c6cf
to
41a997c
Compare
41a997c
to
a32b2e3
Compare
a32b2e3
to
1fd8cf5
Compare
1fd8cf5
to
35c2cbf
Compare
Would you guys please the failures at check https://download.eclipse.org/eclipse/downloads/drops4/I20241020-0600/testresults/html/org.eclipse.pde.ui.tests_ep434I-unit-cen64-gtk3-java23_linux.gtk.x86_64_23.html ? They started to appear after this change and I haven't noticed any other change in this area. |
Just from looking at the code |
Looks like it has different cause as explained in eclipse-jdt/eclipse.jdt.core#3125 (comment) |
What it does
Fix some deprecations by doing what the deprecation message suggests.
How to test
Ideally nothing should change in behavior.
Author checklist