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.

Sign up for GitHub

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

Jump to bottom

Build a hierarchy for RsmState class #12

Open

bachish opened this issue Jan 19, 2024 · 0 comments

Collaborator

bachish commented Jan 19, 2024

Technically, the RsmState class contains from 3 entities:

simple state of Rsm
state with error recovery information
state with string view of path in Rsm

Some things are unnecessary and can be omitted under the right conditions (e.g. adding recovery information if recovery mode if OF).

Therefore, it is worth dividing them into different entities (classes, interfaces) and calling the appropriate instance in each case.

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