Skip to content
This repository was archived by the owner on Nov 4, 2024. It is now read-only.

Conversation

@pprados
Copy link

@pprados pprados commented Jul 19, 2017

  • I use K or V in place of Object in containers
  • It's not acceptable to have a System.exit(). I convert the code to throw a HeapCorruptedError exception.
  • I change the version type in header from int to long, and set with the serialVersionUID, after add a Serializable in the generator and generated classes
  • I add the annotation @PersistentObject in each persistent object ;-)
  • I add a cache in typeForClass() and add a strategy for Scala code
  • I change the constructor of PersistentObject to be compatible with Scala
  • I remove some unused import
  • I add some @OverRide

This change is Reviewable

Philippe PRADOS added 4 commits July 19, 2017 09:22
- Add comments
- Update the code to be compatible with Scala
- Add serialVersionUID
- Add some caches
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant