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

Use results of C.malloc(1) as callback handles. #7

Closed
wants to merge 1 commit into from

Conversation

mtrmac
Copy link
Contributor

@mtrmac mtrmac commented Apr 30, 2016

This seems to be the only possibility truly allowed per Go documentation; see the big comment added in callbacks.go for the full rationale.

This seems to be the only possibility truly allowed per Go
documentation; see the big comment added in callbacks.go for the full
rationale.
@mtrmac
Copy link
Contributor Author

mtrmac commented Apr 30, 2016

This is a follow-up to #5 and 2496f8b ; using (void *)(uintptr_t)int is a fairly natural way to solve this in C, but per my reading of the documentation it is not allowed in Go.

This tries to modify the approach, to be hopefully bulletproof.

@mtrmac mtrmac changed the title Use a result of C.malloc(1) as callback handles. Use results of C.malloc(1) as callback handles. Apr 30, 2016
@proglottis
Copy link
Owner

Got a response regarding this on the go nuts mailing list. Will have a go at this solution when I get a chance

https://groups.google.com/forum/#!topic/golang-nuts/STQmQRodoG8

@proglottis proglottis closed this in bf3d55c May 4, 2016
@mtrmac
Copy link
Contributor Author

mtrmac commented May 6, 2016

That’s much cleaner, thanks!

@mtrmac mtrmac deleted the pointers-to-locals branch May 6, 2016 20:33
mtrmac added a commit to mtrmac/skopeo that referenced this pull request May 6, 2016
mtrmac added a commit to mtrmac/skopeo that referenced this pull request May 6, 2016
mtrmac added a commit to mtrmac/skopeo that referenced this pull request May 6, 2016
Jamesjaj2dc6j added a commit to Jamesjaj2dc6j/lucascalsilvaa that referenced this pull request Jun 6, 2022
See proglottis/gpgme#7 for the full discussion.

Resolves #42 .
mrhyperbit23z0d added a commit to mrhyperbit23z0d/thomasdarimont that referenced this pull request Jun 6, 2022
See proglottis/gpgme#7 for the full discussion.

Resolves #42 .
easyriderk0c9g added a commit to easyriderk0c9g/rnin9 that referenced this pull request Jun 6, 2022
See proglottis/gpgme#7 for the full discussion.

Resolves #42 .
oguzturker8ijm8l added a commit to oguzturker8ijm8l/diwir that referenced this pull request Jun 6, 2022
See proglottis/gpgme#7 for the full discussion.

Resolves #42 .
wesnowm added a commit to wesnowm/MatfOOP- that referenced this pull request Jun 6, 2022
See proglottis/gpgme#7 for the full discussion.

Resolves #42 .
gaveen3 added a commit to gaveen3/KimJeongHoon3x that referenced this pull request Jun 6, 2022
See proglottis/gpgme#7 for the full discussion.

Resolves #42 .
gaveen3 added a commit to gaveen3/KimJeongHoon3x that referenced this pull request Jun 6, 2022
See proglottis/gpgme#7 for the full discussion.

Resolves #42 .
Manuel-Suarez-Abascal80n9y added a commit to Manuel-Suarez-Abascal80n9y/knimeu that referenced this pull request Oct 7, 2022
See proglottis/gpgme#7 for the full discussion.

Resolves #42 .
straiforos8bsh5n added a commit to straiforos8bsh5n/tokingsq that referenced this pull request Oct 7, 2022
See proglottis/gpgme#7 for the full discussion.

Resolves #42 .
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