Skip to content
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

"deprecation" + "removal" warnings are not suppressable simultaneously #469

Open
veita opened this issue Oct 12, 2022 · 5 comments
Open
Assignees
Labels
bug Something isn't working help wanted Extra attention is needed
Milestone

Comments

@veita
Copy link

veita commented Oct 12, 2022

With Eclipse 2022-09 and Java 17 it seems not to be possible to suppress both removal and deprecation warnings on the class level.

//@SuppressWarnings("deprecation")
//@SuppressWarnings("removal")
@SuppressWarnings({ "deprecation", "removal" })
public final class Foo
{
	public void test() 
	{
		throw new AccessControlException("foo");
	}
}
@veita veita changed the title Warnings not suppressable Deprecation/removal warnings not suppressable Oct 12, 2022
@jukzi jukzi added the bug Something isn't working label Feb 14, 2024
@jukzi
Copy link
Contributor

jukzi commented Feb 14, 2024

Can reproduce:

image

i stumpled across the same problem today: the same error is now shown in the platform workspace at
org.eclipse.osgi.tests.bundles.EquinoxBundleAdaptTests.testAdapt_ProtectionDomain()
image

@jukzi jukzi changed the title Deprecation/removal warnings not suppressable "deprecation" + "removal" warnings are not suppressable simultaneously Feb 14, 2024
@jukzi
Copy link
Contributor

jukzi commented Feb 14, 2024

@mpalat who is the expert for this warnings?

@mpalat
Copy link
Contributor

mpalat commented Feb 15, 2024

@mpalat who is the expert for this warnings?

Well, one of the jdt.core committers will have to look at this. I will target this to 4.32 and open it up as helpwanted.
If anyone is interested to take this up, please take a look at the following (@jukzi - I am sure this information is evident for you since you are anyway familiar with the code - please feel free to take this up if time permits)
ProblemReporter.getIrritant() method called with problemID == IProblem.UnusedWarningToken

@mpalat mpalat added help wanted Extra attention is needed good first issue Good for newcomers labels Feb 15, 2024
@mpalat mpalat added this to the 4.32 M1 milestone Feb 15, 2024
@mpalat mpalat removed the good first issue Good for newcomers label Feb 15, 2024
@mpalat mpalat modified the milestones: 4.32 M1, MilestoneNxt Mar 1, 2024
@mpalat mpalat modified the milestones: MilestoneNxt, 4.32 M1 Mar 8, 2024
@subyssurendran666
Copy link
Contributor

@mpalat can you pls assign this ticket to me ?

@jukzi
Copy link
Contributor

jukzi commented Nov 26, 2024

still happens

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

4 participants