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.

Sign up for GitHub

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

Jump to bottom

Decouple render::mesh from GlDevice #589

Merged
merged 5 commits into from
Feb 22, 2015
Merged

Decouple render::mesh from GlDevice #589

merged 5 commits into from
Feb 22, 2015

Conversation

brendanzab
Copy link
Contributor

Working towards #416

@kvark
Copy link
Member

kvark commented Feb 22, 2015

I trust you on this. Just one question:
why do generalize some stuff completely (with R: Resources bound), while blantly putting GlResources into other spots?

kvark added a commit that referenced this pull request Feb 22, 2015
@kvark
Merge pull request #589 from bjz/associated-resources
303cdd0 
Decouple render::mesh from GlDevice
@kvark kvark merged commit 303cdd0 into gfx-rs:master Feb 22, 2015
@brendanzab
Copy link
Contributor Author

I've tried doing the whole lot at once and it becomes unmanageable. So starting at the leaves of the API, I gradually work my way up, pushing the GlResources dependency higher an higher until I've reached the root of the API.

@brendanzab brendanzab deleted the associated-resources branch February 22, 2015 19:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@brendanzab @kvark