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

@Setter/@Getter and @Deprecated create Eclipse AST problems #481

Closed
lombokissues opened this issue Jul 14, 2015 · 8 comments
Closed

@Setter/@Getter and @Deprecated create Eclipse AST problems #481

lombokissues opened this issue Jul 14, 2015 · 8 comments
Milestone

Comments

@lombokissues
Copy link

Migrated from Google Code (issue 408)

@lombokissues
Copy link
Author

👤 andrePankraz   🕗 Aug 16, 2012 at 08:00 UTC

What steps will reproduce the problem?
1.field with @ Setter/@ Getter
2.add @ Deprecated to field
3.

What is the expected output? What do you see instead?

expected are no Eclipse Errors for Save Actions and Organize Imports etc.
I see many Errors in the Error Console and Error Dialogs.

What version of the product are you using? On what operating system?

Eclipse 4.2 with Lombok 0.11.4, Windows 7 64, JDK 7 u6

Please provide any additional information below.

!ENTRY org.eclipse.jdt.ui 4 2 2012-08-16 09:57:12.189
!MESSAGE Problems occurred when invoking code from plug-in: "org.eclipse.jdt.ui".
!STACK 0
java.lang.IllegalArgumentException
at org.eclipse.jdt.core.dom.ASTNode.setSourceRange(ASTNode.java:2648)
at org.eclipse.jdt.core.dom.ASTConverter.convert(ASTConverter.java:2035)
at org.eclipse.jdt.core.dom.ASTConverter.convert(ASTConverter.java:824)
at org.eclipse.jdt.core.dom.ASTConverter.setModifiers(ASTConverter.java:4402)
at org.eclipse.jdt.core.dom.ASTConverter.setModifiers(ASTConverter.java:4491)
at org.eclipse.jdt.core.dom.ASTConverter.convert(ASTConverter.java:427)
at org.eclipse.jdt.core.dom.ASTConverter.buildBodyDeclarations(ASTConverter.java:183)
at org.eclipse.jdt.core.dom.ASTConverter.convert(ASTConverter.java:2656)
at org.eclipse.jdt.core.dom.ASTConverter.convert(ASTConverter.java:1219)
at org.eclipse.jdt.core.dom.CompilationUnitResolver.convert(CompilationUnitResolver.java:291)
at org.eclipse.jdt.core.dom.ASTParser.internalCreateAST(ASTParser.java:1207)
at org.eclipse.jdt.core.dom.ASTParser.createAST(ASTParser.java:807)
at org.eclipse.jdt.internal.ui.javaeditor.ASTProvider$1.run(ASTProvider.java:544)
at org.eclipse.core.runtime.SafeRunner.run(SafeRunner.java:42)
at org.eclipse.jdt.internal.ui.javaeditor.ASTProvider.createAST(ASTProvider.java:537)
at org.eclipse.jdt.internal.ui.javaeditor.ASTProvider.getAST(ASTProvider.java:480)
at org.eclipse.jdt.ui.SharedASTProvider.getAST(SharedASTProvider.java:128)
at org.eclipse.jdt.ui.OverrideIndicatorLabelDecorator.getOverrideIndicators(OverrideIndicatorLabelDecorator.java:161)
at org.eclipse.jdt.ui.OverrideIndicatorLabelDecorator.computeAdornmentFlags(OverrideIndicatorLabelDecorator.java:136)
at org.eclipse.jdt.ui.OverrideIndicatorLabelDecorator.decorate(OverrideIndicatorLabelDecorator.java:273)
at org.eclipse.ui.internal.decorators.LightweightDecoratorDefinition.decorate(LightweightDecoratorDefinition.java:269)
at org.eclipse.ui.internal.decorators.LightweightDecoratorManager$LightweightRunnable.run(LightweightDecoratorManager.java:81)
at org.eclipse.core.runtime.SafeRunner.run(SafeRunner.java:42)
at org.eclipse.ui.internal.decorators.LightweightDecoratorManager.decorate(LightweightDecoratorManager.java:365)
at org.eclipse.ui.internal.decorators.LightweightDecoratorManager.getDecorations(LightweightDecoratorManager.java:347)
at org.eclipse.ui.internal.decorators.DecorationScheduler$1.ensureResultCached(DecorationScheduler.java:371)
at org.eclipse.ui.internal.decorators.DecorationScheduler$1.run(DecorationScheduler.java:331)
at org.eclipse.core.internal.jobs.Worker.run(Worker.java:54)

@lombokissues
Copy link
Author

👤 r.spilker   🕗 Aug 20, 2012 at 08:18 UTC

This is one we need to add to the automated eclipse tests.

@lombokissues
Copy link
Author

👤 reinierz   🕗 Oct 21, 2012 at 23:33 UTC

See also thread

https://groups.google.com/forum/?fromgroups=﹟!topic/project-lombok/-iAnwlEzyd0

where another user has run into this issue. As running into this (rare) issue is quite confusing to the user we should fix this with high priority.

@lombokissues lombokissues added the accepted The issue/enhancement is valid, sensible, and explained in sufficient detail label Jul 14, 2015
@lombokissues
Copy link
Author

👤 reinierz   🕗 Oct 29, 2012 at 23:25 UTC

Fixed with voodoo magic. Should be in 0.11.6.

@lombokissues lombokissues removed the accepted The issue/enhancement is valid, sensible, and explained in sufficient detail label Jul 14, 2015
@lombokissues lombokissues added this to the 0.11.5 milestone Jul 14, 2015
@lombokissues
Copy link
Author

👤 reinierz   🕗 Oct 30, 2012 at 00:09 UTC

Now officially released in lombok v0.11.6.

@lombokissues lombokissues modified the milestones: 0.11.5, 0.11.6 Jul 14, 2015
@lombokissues
Copy link
Author

End of migration

@goelrohi
Copy link

Hi,

I'm getting the same error while using eclipse Version: Oxygen Release (4.7.0) with Lombok v1.16.16 "Dancing Elephant"

@omega09
Copy link

omega09 commented Sep 18, 2017

@goelrohi Can't reproduce. Can you post a small code snippet?

lianhaijun pushed a commit to lianhaijun/lombok that referenced this issue May 8, 2020
lianhaijun pushed a commit to lianhaijun/lombok that referenced this issue May 8, 2020
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

No branches or pull requests

3 participants