Skip to content
This repository has been archived by the owner on Jun 23, 2023. It is now read-only.

Ensure all primitive types are mapped properly #4

Closed
bengreenier opened this issue Jan 4, 2022 · 0 comments · Fixed by #13, #17, #41, #49 or #51
Closed

Ensure all primitive types are mapped properly #4

bengreenier opened this issue Jan 4, 2022 · 0 comments · Fixed by #13, #17, #41, #49 or #51
Labels
question Further information is requested

Comments

@bengreenier
Copy link
Contributor

Currently, a best-effort set of primitives is mapped in tsgen/resolve.ts. However, we should ensure that the list covers all types that ref.types knows how to represent, which should, by extension, be all valid primitive types.

If you're reading this issue, it might be because a type is missing. To debug, use set DEBUG=clangffi:tsgen:resolve and look for your type. The output should make it clear if it was resolved as a "builtin" or "unknown" type. If you believe it to be a primitive but it's showing up as "unknown", we need to add it to these lookup tables.

@bengreenier bengreenier added bug Something isn't working question Further information is requested and removed bug Something isn't working labels Jan 4, 2022
bengreenier added a commit that referenced this issue Jan 5, 2022
bengreenier added a commit that referenced this issue Jan 5, 2022
This was referenced Jan 5, 2022
This was referenced Mar 24, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.