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

added more callbacks #141

Merged
merged 6 commits into from
Feb 14, 2025
Merged

added more callbacks #141

merged 6 commits into from
Feb 14, 2025

Conversation

0xeb
Copy link
Collaborator

@0xeb 0xeb commented Feb 11, 2025

No description provided.

@0xeb 0xeb requested a review from momo5502 February 11, 2025 00:33
@0xeb
Copy link
Collaborator Author

0xeb commented Feb 11, 2025

sorry, the formatting checks is schizophrenic. the syscalls.cpp is formatted with clang formatter and it keeps complaining.
@momo5502

@robert-yates
Copy link
Collaborator

sorry, the formatting checks is schizophrenic. the syscalls.cpp is formatted with clang formatter and it keeps complaining. @momo5502

i have this problem at work also sometimes. different clang format binaries can have slight changes. in this case it appears the workflow is using clang-format v19, check you have the same version

@0xeb
Copy link
Collaborator Author

0xeb commented Feb 11, 2025

@robert-yates , I use Ctrl-K, Ctrl-D shortcut from within visual studio 2022 (latest update).

what else do I need to use for that? any detailed pointers?

@robert-yates
Copy link
Collaborator

@robert-yates , I use Ctrl-K, Ctrl-D shortcut from within visual studio 2022 (latest update).

what else do I need to use for that? any detailed pointers?

hmm im not familiar with the visual studio implementation, however i would download llvm 19.1.0 from llvm.org and add the clang-format binary at the start of your global path env and restart VS and see it it picks up that instead

@0xeb
Copy link
Collaborator Author

0xeb commented Feb 11, 2025

@robert-yates thanks, let's see

image

@momo5502
Copy link
Owner

@robert-yates , I use Ctrl-K, Ctrl-D shortcut from within visual studio 2022 (latest update).

what else do I need to use for that? any detailed pointers?

that uses the integrated visual studio formatter which won't use clang-format.
I use this plugin: https://marketplace.visualstudio.com/items?itemName=caphyon.ClangPowerTools
Then you only have to click this button:

image

In the settings you can also configure format-on-save, which is nice.

@0xeb
Copy link
Collaborator Author

0xeb commented Feb 12, 2025

Thanks for the plugin hint @momo5502. I think that’s the one you told me about before. Anyway, now the formatter works and I could tell it which tool version to use. The PR should be ready now.

@0xeb 0xeb requested a review from momo5502 February 12, 2025 06:16
@momo5502
Copy link
Owner

Thanks for the plugin hint @momo5502. I think that’s the one you told me about before. Anyway, now the formatter works and I could tell it which tool version to use. The PR should be ready now.

yep. directx install hung up. sadly that happens sometimes :( need to investigate why. i retriggered the workflow. let's see if it's green :D

@0xeb 0xeb dismissed momo5502’s stale review February 12, 2025 17:43

I am not sure how to resolve this change request, but I am pretty sure I finished all the requested changes

@0xeb
Copy link
Collaborator Author

0xeb commented Feb 13, 2025

Ping @momo5502 ?

@momo5502 momo5502 merged commit 6b4e860 into momo5502:main Feb 14, 2025
58 checks passed
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.

3 participants