Skip to content

Fix rparse to rcore binding for afen#346

Merged
trufae merged 2 commits into
radareorg:masterfrom
satk0:fix-afen-parser
Jun 25, 2024
Merged

Fix rparse to rcore binding for afen#346
trufae merged 2 commits into
radareorg:masterfrom
satk0:fix-afen-parser

Conversation

@satk0

@satk0 satk0 commented Jun 23, 2024

Copy link
Copy Markdown
Contributor

Checklist

Description

Comment thread afen/src/main.c Outdated

// RParse plugin Definition Info
RParsePlugin r_parse_plugin_afen = {
.name = "rparse-afen",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What about naming it “afen”

Suggested change
.name = "rparse-afen",
.name = "afen",

Comment thread afen/src/main.c Outdated
static int r_cmd_fini(void *user, const char *input) {
static int r_core_fini_afen(void *user, const char *input) {
r_list_free(old_names);
r_list_free(new_names);

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Space before (

@satk0

satk0 commented Jun 25, 2024

Copy link
Copy Markdown
Contributor Author

Hopefully, now is cool 😄

@trufae
trufae merged commit 6e4ece2 into radareorg:master Jun 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants