-
Notifications
You must be signed in to change notification settings - Fork 960
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
rename methods for command buffer to methods for command encoder #277
Conversation
c36698a
to
231bfeb
Compare
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 good to me, thank you! Just need to make sure we regenerate (or at least keep) the remote header too.
ffi/wgpu-remote.h
Outdated
@@ -1,72 +0,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.
Looks like this was accidentally removed, maybe it wasn't regenerated during make
? If you can't get this to be generated, we could just amend the commit to keep it
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.
Yeah, something doesn't work correctly with ffi creation using make, not sure why. I'll fix it and force push it again
231bfeb
to
8ae180a
Compare
8ae180a
to
7df34c0
Compare
Should be fixed now |
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.
Great, thank you!
bors r=groveNL,kvark
ah, didn't realize we have a conflict |
Merge conflict |
@porky11 please rebase |
Replaced by #299 |
closes #250