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

Make device::gl_device crate fully self-contained #597

Merged
merged 2 commits into from
Feb 23, 2015

Conversation

brendanzab
Copy link
Contributor

Closes #416.

Sorry for flip-flopping on the type parameter of Renderer - I previously changed it before GlResources was a 'thing'.

@@ -49,8 +49,8 @@ fn test_vertex_format() {
use secret_lib::gfx;
use secret_lib::device;

let buf_vert = device::make_fake_buffer();
let buf_inst = device::make_fake_buffer();
let buf_vert = device::gl_device::make_dummy_buffer();
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd like these tests to be device-agnostic. We'll clean it up later.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agreed.

kvark added a commit that referenced this pull request Feb 23, 2015
Make device::gl_device crate fully self-contained
@kvark kvark merged commit bb652dd into gfx-rs:master Feb 23, 2015
@kvark
Copy link
Member

kvark commented Feb 23, 2015

👍 🎉

@brendanzab
Copy link
Contributor Author

\o/ party!

@brendanzab brendanzab deleted the associated-resources branch February 23, 2015 21:57
@bvssvni
Copy link
Contributor

bvssvni commented Feb 23, 2015

🎈

@ghost
Copy link

ghost commented Feb 23, 2015

🎈 🎉

adamnemecek pushed a commit to adamnemecek/gfx that referenced this pull request Apr 1, 2021
598: Enable READ access for texture storage r=kvark a=kvark

This is a short-term workaround until we properly implement gfx-rs#597 

Co-authored-by: Dzmitry Malyshau <[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.

Make Device completely self-contained with associated types
3 participants