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

Lombok annotations are not showed properly #37

Closed
Ravisankar-Challa opened this issue Jun 6, 2018 · 5 comments
Closed

Lombok annotations are not showed properly #37

Ravisankar-Challa opened this issue Jun 6, 2018 · 5 comments

Comments

@Ravisankar-Challa
Copy link

image
Adding a sample project for testing.
https://github.com/Ravisankar-Challa/meecrowave_test/blob/master/src/main/java/meecrowave/model/MiscellaneousInfo.java

Note: Lombok needs to be configured in the eclipse ide
https://projectlombok.org/downloads/lombok.jar
place the lombok file in the same folder of eclipse.exe file and update the eclipse.ini file

-vmargs
-javaagent:lombok.jar
-Dosgi.requiredJavaVersion=1.8

@Ravisankar-Challa Ravisankar-Challa changed the title Lombok annotations are not showed propely Lombok annotations are not showed properly Jun 6, 2018
@angelozerr
Copy link
Owner

I have tried to follow your instruction, but I have not the same thing than you:

image

Perhaps it requires to install other plugin like STS?

@Ravisankar-Challa
Copy link
Author

Ravisankar-Challa commented Jun 20, 2018

I am using jdk 1.8.*.
Those things show up only when the below properties are enabled

image

@angelozerr
Copy link
Owner

Ok my problem was about config of lombok.jar. I have the same problem than you.

After debugging, it seems that lombok agent update JDT ASTNode to generate getter, and change @log annotation with some code.

It causes the problem that you see. I think I have a solution but it will be linked to lombok and it breaks a lot of code -(

I don't know if it's a feature that JDT wishes to support because it will be linked to lombok.

@vogella
Copy link
Contributor

vogella commented Jun 22, 2018

One thing to consider is that Lombok violates the APT spec as it modifies the class directly instead of generating new ones. So I assume we will see additional code minings for the generate code once this works.

@angelozerr
Copy link
Owner

@Ravisankar-Challa QI'm sorry, but I think I will not fix this issue. It's to specific to lombok

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

No branches or pull requests

3 participants