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

feat: decouple current hook implementations #13

Open
gottyduke opened this issue Sep 23, 2023 · 0 comments
Open

feat: decouple current hook implementations #13

gottyduke opened this issue Sep 23, 2023 · 0 comments
Labels
enhancement New feature or request

Comments

@gottyduke
Copy link
Owner

Current hook implementations are all designated to specific tasks, and there're reasonably fair amount of duplicate code. I've somewhat remedied that by separating small internal functions to use, but the major ones like AddASMPatch, AddCaveHook are narrowly specialized.

The feature would be to fully modularize each step and transform the current API into the form of composition instead of specialization. This would also expose the bespoke "modularized functions" to developers.

@gottyduke gottyduke added the enhancement New feature or request label Sep 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant