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

Manifold properties combined with Lombok @RequiredArgsConstructor #598

Open
EotT123 opened this issue Jun 2, 2024 · 3 comments
Open

Manifold properties combined with Lombok @RequiredArgsConstructor #598

EotT123 opened this issue Jun 2, 2024 · 3 comments

Comments

@EotT123
Copy link
Contributor

EotT123 commented Jun 2, 2024

Lombok's RequiredArgsConstructor is a convenient tool for generating constructors without the need to manually write boilerplate code. However, this functionality encounters issues when used with Manifold properties. Fields annotated with @val are treated as final by the compiler, but Lombok does not recognize them as such.
I'm not sure if anything can be done about this, or if changes to Lombok's codebase are required to resolve this incompatibility?

@rsmckinney
Copy link
Member

It suspect it's a chicken/egg problem that can't be rectified, but I'll take a deeper look.

On the brighter side @data classes are next in the lineup, which will similarly support a required arguments constructor etc.

@EotT123
Copy link
Contributor Author

EotT123 commented Jun 4, 2024

I'm looking forward to it! Is there already some information available about the (possible) features?

@rsmckinney
Copy link
Member

See #257

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

2 participants