Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Separate Options from data storage #42

Open
uliska opened this issue Mar 21, 2019 · 1 comment
Open

Separate Options from data storage #42

uliska opened this issue Mar 21, 2019 · 1 comment
Assignees

Comments

@uliska
Copy link
Contributor

uliska commented Mar 21, 2019

Also related to #2

Currently some packages (and many projects, at least my own) use oll-core's option alist to store all kinds of data. This is probably not a good idea because it makes the option tree pretty complex, and maybe it's even unefficient.

It is unclear how this should be approached, but here are some thoughts:

  • there should be an interface for createing new data storage objects, beyond directly using the Tree implementation.
  • maybe in addition to the option tree there should be one second oll-data alist where arbitrary data trees can be added as top-level entries.
  • it should be possible to use different implementations for these data tree objects, but there must be a common API to manipulate them, agnostic of the tree implementation
@uliska uliska self-assigned this Mar 21, 2019
@uliska
Copy link
Contributor Author

uliska commented Aug 18, 2020

The chamged alist-access module that doesn't store its alists in the public namespace anymore is a first step.

There might be an optional argument to the options commands specifying the tree to use.

OTOH it might be a better approach to use property sets for storing data in the first place

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant