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

Immutable list #184

Open
jaumard opened this issue Sep 8, 2019 · 2 comments
Open

Immutable list #184

jaumard opened this issue Sep 8, 2019 · 2 comments

Comments

@jaumard
Copy link
Contributor

jaumard commented Sep 8, 2019

When deserialization is done and model contains list, they should be immutable (meaning use growable:false) to prevent miss usage add allow add/remove items into those list.

@keshava19
Copy link

What about the cases when you want to work on top of the list ?

@jaumard
Copy link
Contributor Author

jaumard commented Oct 2, 2019

If models are annotated with @immutable it means we want immutable models so everything should be immutable not only final fields. People who doesn't want immutable would be able to work on top of the list like this. Or you can still create a list from the immutable one if you want to

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

2 participants