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

Small arm64 compat header fixes #2563

Merged
merged 2 commits into from
Dec 5, 2024

Conversation

satk0
Copy link
Contributor

@satk0 satk0 commented Nov 29, 2024

Your checklist for this pull request

  • I've documented or updated the documentation of every API function and struct this PR changes.
  • I've added tests that prove my fix is effective or that my feature works (if possible)

Detailed description

...

Test plan

...

Closing issues

...

First of all, thanks a lot for making those changes to arm64 compat header :)

I've tried to use the newest csnext in r2 and found some things missing so I've made this PR implementing it, hope it helps 😅

@github-actions github-actions bot added the AArch64 Arch label Nov 29, 2024
@Rot127
Copy link
Collaborator

Rot127 commented Nov 29, 2024

Thanks! Please also fix it in the header generator script and add these two to the test c file.

@satk0
Copy link
Contributor Author

satk0 commented Nov 29, 2024

Allright, thanks for such a quick response

@github-actions github-actions bot added the Auto-Sync-files Auto-Sync label Nov 30, 2024
@satk0
Copy link
Contributor Author

satk0 commented Nov 30, 2024

Is that it? I couldn't find this test c file you were talking about to add it here ;/

Copy link
Collaborator

@Rot127 Rot127 left a comment

Choose a reason for hiding this comment

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

All good. Thanks a lot!

@XVilka
Copy link
Contributor

XVilka commented Dec 3, 2024

@kabeor please merge this one

Copy link
Member

@kabeor kabeor left a comment

Choose a reason for hiding this comment

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

Thank you!

@Rot127
Copy link
Collaborator

Rot127 commented Dec 5, 2024

@satk0 Can you rebase please?

For whatever reason Github doesn't want to merge this one.

@satk0
Copy link
Contributor Author

satk0 commented Dec 5, 2024

Allright, give me 5 minutes

@satk0 satk0 force-pushed the compat-header-fixes branch from e77acdf to 628f947 Compare December 5, 2024 16:26
@satk0
Copy link
Contributor Author

satk0 commented Dec 5, 2024

@Rot127 Done

@Rot127 Rot127 merged commit 0a29bf8 into capstone-engine:next Dec 5, 2024
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants