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

Convert analysis a (cmd_analysis.c) commands to rzshell #1442

Closed
20 of 21 tasks
XVilka opened this issue Aug 13, 2021 · 1 comment
Closed
20 of 21 tasks

Convert analysis a (cmd_analysis.c) commands to rzshell #1442

XVilka opened this issue Aug 13, 2021 · 1 comment
Labels
cmd-api Switch from cmd calls to API high-priority refactor Refactoring requests RzAnalysis shell
Milestone

Comments

@XVilka
Copy link
Member

XVilka commented Aug 13, 2021

Some of the commands are already converted, we should finish the complete migration. Moreover, some analysis logic is inside the librz/core/cmd_analysis.c. We should move as much as possible inside either librz/analysis/* or librz/core/canalysis.c from librz/core/cmd_analysis.c.

The task implies also possible API changes and some refactorings, see #1223, #1117, #992, and #489

See these files:

  • librz/core/cmd_analysis.c
  • librz/core/cmd_descs/cmd_analysis.yaml

The task is important for further porting of different architectures to the global variables, new IL.

@XVilka XVilka added this to the 0.4.0 milestone Aug 13, 2021
@ret2libc ret2libc added the cmd-api Switch from cmd calls to API label Sep 3, 2021
@ret2libc ret2libc changed the title Convert analysis a (cmd_analysis.c) commands to the newshell Convert analysis a (cmd_analysis.c) commands to rzshell Sep 3, 2021
@pelijah
Copy link
Contributor

pelijah commented Dec 7, 2021

ax commands have already been ported

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cmd-api Switch from cmd calls to API high-priority refactor Refactoring requests RzAnalysis shell
Development

No branches or pull requests

3 participants