ekolis/FrEee

Convert simple classes that mainly just store data to records

Opened this issue · 1 comments

What needs to be cleaned up?

There are a lot of simple classes that mainly just store some data and don't have much in the way of references to other classes that need to maintain referential integrity. Convert them to data classes. Umm, I mean, records, this is not Kotlin!

How will this benefit us?

Records are supposed to have better performance than classes I think? And the syntax is simpler since properties can be converted to constructor parameters and equals/hashcode/etc are auto-implemented.

What potential drawbacks are there to making this change?

Just the time consumed doing it and any potential bugs introduced

Perhaps this could be a first step toward implementing #312?