search

FWDekker / intellij-randomness

IntelliJ plugin for inserting random numbers, UUIDs, names, IP addresses, and much more
https://plugins.jetbrains.com/plugin/9836-randomness
MIT License
45 stars 7 forks source link

Check usage of `var` vs `val` in all schemes #476

Closed FWDekker closed 1 year ago

FWDekker commented 1 year ago

For non-primitive types, anyway. Check that they are vals wherever possible. I think the serialisation doesn't like it when a list is a val instead of a var, but honestly, tough luck! I really prefer using a val there. I really dislike mutability. But do a sanity check anyway whether my dislike is justified there.

FWDekker commented 1 year ago

The following Lists are notable and should definitely be vals rather than vars:

Class Field Annotation
Settings templateList @OptionTag
TemplateList templates @MapAnnotation
Template schemes @XCollection