Fixed CLFuncFn and broken macros in cyber.h #99
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Noticed differences between parameter types expressed in the docs and in use in cyber.h
Changed
to
In addition I also updated the type macros
CL_CUSTOM_TYPE
andCL_CUSTOM_PRE_TYPE
to use the proper union and enum.This change made module loading somewhat functional on Windows. Still am dealing with errors regarding pointer casting in the
getOrBufPrintValueStr
function in vm.zig when trying to run a c-standard compliant version of the bind_module.c example. I'm not sure what could be causing this but it looks like a nasty issue.Errors:
Here's the code I've been using for testing purposes. The only real changes are for non-static initialization (zig cc didn't like setting members of structs via functions outside of the call stack) and I also added code ensuring all printed strings are null-terminated.