Skip to content

Conversation

@snjeza
Copy link

@snjeza snjeza commented Dec 22, 2025

Fixes #2795

return new ByteBuddy()
.subclass(Object.class)
.name(name)
.defineMethod("augment", name.equals("lombok.core.FieldAugment") ? Object.class : Object.class, Visibility.PUBLIC, Ownership.STATIC)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is the point of name.equals("lombok.core.FieldAugment") ? Object.class : Object.class given that both branches return the same value?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have fixed it.

@nedtwigg
Copy link
Member

i'm trying to fix this up to merge, please don't force-push

}

dependencies {
implementation 'net.bytebuddy:byte-buddy:1.18.3'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we have a lib and lib-extra division so that lib has no dependencies, and lib-extra gets stuck with the dependency hell.

byte-buddy cannot be a dependency here. I don't fully understand what you're doing here, but you'll have to do it without adding dependencies into lib.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@nedtwigg I will try.

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

Successfully merging this pull request may close these issues.

The Spotless Maven plugin doesn’t work with lombok in Eclipse/VS Code/Cursor

3 participants