Ideas for cleanups & API simplification
vorner opened this issue · 1 comments
vorner commented
- The Lease and similar „transient“ objects could deref to Arc (or the held pointer) instead of T directly.
- This would solve the problem with Option valued things.
- This would also allow
Arc::clone
without the need for our ownupgrade
method.
- There are entirely too many load-style methods. Can we pick fewer of them and get rid of the rest?
- Is the name
load
confusing? Eg.let db = in_memory_db.load()
hints at loading the whole database somewhere from disk, not at just getting a pointer to it.
vorner commented
Decided to leave load
as it is (or, actually, migrate everything to loads).