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

Fixed pipeline macro #1077

Closed
wants to merge 3 commits into from
Closed

Fixed pipeline macro #1077

wants to merge 3 commits into from

Conversation

snuk182
Copy link
Contributor

@snuk182 snuk182 commented Nov 8, 2016

An attempt to fix #1070 by adding a crate parameter to the macro.

@Bastacyclop
Copy link
Contributor

Still requires the $crt module to be visible inside the super module. This won't work: https://is.gd/y9S3xD. Maybe use use ::$crt; ? https://is.gd/ztAWxP

@Emilgardis
Copy link

Is this being worked on? @snuk182

@snuk182
Copy link
Contributor Author

snuk182 commented Nov 28, 2016

@Emilgardis
Looks like this issue is not reproducible anymore at both nightly and stable (with 0.12.2 and git)

@Emilgardis
Copy link

0.12.2 has been yanked, but master has been reported to work.

@sectopod
Copy link
Contributor

Is this still needed?

@snuk182
Copy link
Contributor Author

snuk182 commented Dec 15, 2016

Not sure. Closing as of now.

@snuk182 snuk182 closed this Dec 15, 2016
adamnemecek pushed a commit to adamnemecek/gfx that referenced this pull request Apr 1, 2021
1077: Remove extern "C" from set_index_buffer r=kvark a=DevOrc

**Connections**
Fixes an issue that is blocking gfx-rs/wgpu-native#61

**Description**
Moves the ffi definition of wgpu_render_bundle_set_index_buffer / wgpu_render_pass_set_index_buffer to wgpu-native. This is needed because wgpu-native has its own version of IndexFormat that is different than the wgpu_types version. 

**Testing**
My projects with wgpu-native work


Co-authored-by: Noah Charlton <[email protected]>
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.

GFX is broken on nightly
4 participants