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

[var] since majority of 'var' is used, used the rest #858

Merged
merged 1 commit into from
Mar 17, 2024

Conversation

hazendaz
Copy link
Member

@hazendaz hazendaz commented Mar 17, 2024

'var' is not hard to understand. Its already stating directly what it is by the right hand side. This is still type safe. Further its just the opposite of diamond operation. Java provided this for a valid reason, we end up with less imports, cleaner code, and lose nothing using it.

This was already done in mass a long time back because code base moved above java 10. Being consistent at this point as very little is not done.

'var' is not hard to understand.  Its already stating directly what it is by the right hand side.  This is still type safe.  Further its just the opposite of diamond operation.  Java provided this for a valid reason, we end up with less imports, cleaner code, and lose nothing using it.
@hazendaz hazendaz self-assigned this Mar 17, 2024
@hazendaz hazendaz merged commit e17ce9c into revelc:main Mar 17, 2024
3 checks passed
@hazendaz hazendaz added this to the 2.24.0 milestone Mar 17, 2024
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.

1 participant