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

Decouple specialised memories from generic memories #763

Merged
merged 6 commits into from
May 11, 2021

Conversation

nolar
Copy link
Owner

@nolar nolar commented May 11, 2021

As preparation for the layered layout of the codebase (#765), decouple specialised memories from the generic memories. As a result, the generic memories can be shifted to the lower levels of the module-/class-dependency tree.

As of now, memories are unified, so they have to depend on specialised aspects (e.g. on daemons to keep running daemons, on indexing to keep the intermediate indexing state, etc). Those specialised modules, in turn, have to depend on the memories to access their specialised parts.

The separation is done by splitting the implementations from the specialised interfaces.

The are no changes in the external behaviour or public interfaces.

nolar added 6 commits May 10, 2021 23:42
Signed-off-by: Sergey Vasilyev <nolar@nolar.info>
Signed-off-by: Sergey Vasilyev <nolar@nolar.info>
Signed-off-by: Sergey Vasilyev <nolar@nolar.info>
Signed-off-by: Sergey Vasilyev <nolar@nolar.info>
Signed-off-by: Sergey Vasilyev <nolar@nolar.info>
Signed-off-by: Sergey Vasilyev <nolar@nolar.info>
@nolar nolar added the refactoring Code cleanup without new features added label May 11, 2021
@nolar nolar enabled auto-merge May 11, 2021 15:44
@nolar nolar merged commit 2cae641 into main May 11, 2021
@nolar nolar deleted the refactor-decouple-memories branch May 11, 2021 15:49
@nolar nolar changed the title Refactor decouple memories Decouple specialised memories from generic memories May 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactoring Code cleanup without new features added
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant