-
-
Notifications
You must be signed in to change notification settings - Fork 42
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
Surface 'array' from SPIRType in spirv::Type #58
Conversation
Note that this is a breaking change. The unit tests have been updated, but there should be at least one new unit test to cover the case where there is an array.
Codecov Report
@@ Coverage Diff @@
## master #58 +/- ##
=========================================
- Coverage 71.23% 70.33% -0.9%
=========================================
Files 5 5
Lines 584 600 +16
=========================================
+ Hits 416 422 +6
- Misses 168 178 +10
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, thanks! I hope the FFI parts weren't too confusing to modify 😄
bors r+
|
||
check!(sc_internal_free_pointer(raw.member_types as *mut c_void)); | ||
if raw.member_types_size > 0 { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Thanks! I managed not to tie either the commits or pull request to issue #43, but I'll try and correct that now. |
Also adds a unit test for a simple array example.