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

Optimisation, fixe some bugs, add some comments #2

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

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

- 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