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

Refactoring: Agent Base Architecture #376

Closed
volodya-lombrozo opened this issue Aug 9, 2024 · 2 comments · Fixed by #379
Closed

Refactoring: Agent Base Architecture #376

volodya-lombrozo opened this issue Aug 9, 2024 · 2 comments · Fixed by #379
Assignees
Labels
enhancement New feature or request soon This issue will be solved soon

Comments

@volodya-lombrozo
Copy link
Member

We decided to modify the architecture slightly. Currently, we process instructions one by one. We search for an appropriate handler for each instruction, which is fast but not easily extendable. Right now, we have relatively high coupling between classes.

We can try implementing a different agent-based architecture. Instead of instructing handlers on which instruction to handle, we can implement a set of agents that would scan the entire instruction stack and independently decide how to decompile different parts of it.
This might make the architecture much more flexible.

In the beginning, we should create a sketch: just a few classes that would partially describe the entire solution.

@volodya-lombrozo volodya-lombrozo added enhancement New feature or request soon This issue will be solved soon labels Aug 9, 2024
@volodya-lombrozo volodya-lombrozo self-assigned this Aug 9, 2024
volodya-lombrozo added a commit to volodya-lombrozo/opeo-maven-plugin that referenced this issue Aug 12, 2024
volodya-lombrozo added a commit to volodya-lombrozo/opeo-maven-plugin that referenced this issue Aug 12, 2024
volodya-lombrozo added a commit to volodya-lombrozo/opeo-maven-plugin that referenced this issue Aug 12, 2024
volodya-lombrozo added a commit to volodya-lombrozo/opeo-maven-plugin that referenced this issue Aug 12, 2024
volodya-lombrozo added a commit to volodya-lombrozo/opeo-maven-plugin that referenced this issue Aug 12, 2024
volodya-lombrozo added a commit to volodya-lombrozo/opeo-maven-plugin that referenced this issue Aug 12, 2024
volodya-lombrozo added a commit to volodya-lombrozo/opeo-maven-plugin that referenced this issue Aug 12, 2024
volodya-lombrozo added a commit to volodya-lombrozo/opeo-maven-plugin that referenced this issue Aug 12, 2024
volodya-lombrozo added a commit to volodya-lombrozo/opeo-maven-plugin that referenced this issue Aug 12, 2024
volodya-lombrozo added a commit to volodya-lombrozo/opeo-maven-plugin that referenced this issue Aug 12, 2024
volodya-lombrozo added a commit to volodya-lombrozo/opeo-maven-plugin that referenced this issue Aug 12, 2024
@0pdd
Copy link

0pdd commented Aug 13, 2024

@volodya-lombrozo 2 puzzles #383, #384 are still not solved.

@0pdd
Copy link

0pdd commented Aug 16, 2024

@volodya-lombrozo the puzzle #383 is still not solved; solved: #384.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request soon This issue will be solved soon
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants