-
-
Notifications
You must be signed in to change notification settings - Fork 61
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
added more callbacks #141
Conversation
sorry, the formatting checks is schizophrenic. the syscalls.cpp is formatted with clang formatter and it keeps complaining. |
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 |
@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 |
@robert-yates thanks, let's see |
that uses the integrated visual studio formatter which won't use clang-format. In the settings you can also configure format-on-save, which is nice. |
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 |
I am not sure how to resolve this change request, but I am pretty sure I finished all the requested changes
Ping @momo5502 ? |
No description provided.