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

fix callback issue #4

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

fix callback issue #4

wants to merge 1 commit into from

Conversation

thechampagne
Copy link
Owner

fix #3

@gitgit3
Copy link

gitgit3 commented Jul 16, 2024

I'm not super familiar with github to be honest. How to I pull your bugfix/callback-issue to try it out?
Edit:
It's OK, I figured it out. Just checking it out now.

@gitgit3
Copy link

gitgit3 commented Jul 16, 2024

I would have said that the bug is actually in webview c-code, but you've used the arg-pointer to resolve the actual function call. I like it.
Just tested it and it works.

@gitgit3
Copy link

gitgit3 commented Jul 27, 2024

I created the pointer to CallbackContext!(BindCallback)* ctx on the heap since I could not figure out a syntax to place it as a static, this may or may not be possible, I am a beginner with D. Maybe you might suggest something. This tripped me up later when I found out that my pointer was being GCd causing a segfault. I stuck the pointers in an array in the class. How did you envisage calling the bind method?

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.

Callback function pointers do not allow for multiple callbacks
2 participants