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

darwinkit: Handle functions #151

Open
tmc opened this issue Jul 20, 2023 · 8 comments
Open

darwinkit: Handle functions #151

tmc opened this issue Jul 20, 2023 · 8 comments
Labels
generation Related to generation toolchain help wanted Extra attention is needed
Milestone

Comments

@tmc
Copy link
Collaborator

tmc commented Jul 20, 2023

Our code generator doesn't currently handle functions, and Core Graphics in particular has many exposed that we can't get at until we make the code generator more capable.

@tmc tmc changed the title Handle functions codegen: Handle functions Jul 20, 2023
@programmingkidx
Copy link
Contributor

I have been thinking about this for a while. If we could use Go functions as Objective-C callbacks that would make things very interesting. I want to use NSControl's SetAction() method to be able to accept a Go function.

One way I thought on how we could do this is to use a hash table to store Go functions. When a function like SetAction() is called with a Go function, the Go function is stored in the hash table on the Go side. A new Objective-C class instance is made that connects to the Objective-C side. The class will have a method called doAction: and an instance variable called goFuncName. doAction will be called when an action like a button being pushed happens. goFuncName will store the value of the Go function's name as a string. When the Objective-C system calls doAction:, it will call a Go function called goCallFunc(goFuncName). goCallFunc() will turn goFuncName into a hash value and get the Go function. Then the Go function will be called.

This might work but there might be issues. Where do we keep track of the Objective-C class instance? When do we release it?

Another idea I had involved trying to find a Go function in the mach-o file format. Rather than keeping track of methods in a hash table we try to find the Go method in the binary by somehow parsing the mach-o format. I do know symbols like functions are kept in the binary. I don't know if something like this could work.

@tmc
Copy link
Collaborator Author

tmc commented Jul 23, 2023

That all sounds well and good, I was moreso wanting to call this from go: https://developer.apple.com/documentation/coregraphics/1455691-cgdisplaycreateimage?language=objc

@programmingkidx
Copy link
Contributor

programmingkidx commented Jul 23, 2023

I see now. I missed what you wanted. You wanted to call C functions from CoreGraphics. Doesn't sound hard to do. I am guessing the mapType() function in gen/lookup.go would have to be made to handle C functions. But then again is a C function a data type? I'm thinking a new system that wraps C function needs to be made.

@programmingkidx
Copy link
Contributor

I guess we should decide how to start. Do we use MacSchema to download a schema for Core Graphics? If so we need to enhance MacSchema to handle this case - C functions as oppose to Objective-C methods. Are you able to download a json file for Core Graphics without using MacSchema?

After we have a json file for Core Graphics we need to have the generation system generate a new package for Core Graphics. My guess is we would need to change how gen/cmd/loader.go loads this json file. The generatePackage() function is what we would probably have to change.

@tmc
Copy link
Collaborator Author

tmc commented Jul 23, 2023

I have a bit of a start at https://github.com/progrium/macschema/pull/20 but it needs to be rebased.

@tmc
Copy link
Collaborator Author

tmc commented Jul 23, 2023

@progrium
Copy link
Owner

I mentioned in Discord maybe we should use FFI, but we could instead model functions after that example where we would generate functions that call cgo functions defined in Objective-C that as far as I can tell are needed to cast raw pointers to typed pointers. We'd generate a functions.gen.go and a functions.gen.m.

@progrium progrium added this to the 0.5.0 milestone Aug 19, 2023
@progrium progrium added help wanted Extra attention is needed generation Related to generation toolchain labels Aug 19, 2023
@progrium progrium changed the title codegen: Handle functions darwinkit: Handle functions Aug 19, 2023
@progrium
Copy link
Owner

It would be helpful if somebody could look at the functions in that coregraphics code (display and window) with the docs for each function to start putting together some rules for types. Then I could pretty quickly get generation in for functions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
generation Related to generation toolchain help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants