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

pprados added 4 commits July 19, 2017 09:22
- Add comments
- Update the code to be compatible with Scala
- Add serialVersionUID
- Add some caches
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant