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

Create buffer mapped #84

Merged
merged 1 commit into from
Mar 1, 2019
Merged

Create buffer mapped #84

merged 1 commit into from
Mar 1, 2019

Conversation

swiftcoder
Copy link
Contributor

I haven't really taken a swing at error handling here, posting it up early for feedback on the API.

Copy link
Member

@kvark kvark left a comment

Choose a reason for hiding this comment

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

Nice!
bors r+

let vertex_buf = {
let (vertex_buf, vertex_buf_data) = device.create_buffer_mapped(&wgpu::BufferDescriptor {
size: vertex_buffer_length as u32,
usage: wgpu::BufferUsageFlags::VERTEX | wgpu::BufferUsageFlags::TRANSFER_DST | wgpu::BufferUsageFlags::MAP_WRITE,
Copy link
Member

Choose a reason for hiding this comment

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

we can/should imply MAP_WRITE here. Also, TRANSFER_DST shouldn't be here?

bors bot added a commit that referenced this pull request Mar 1, 2019
84: Create buffer mapped r=kvark a=swiftcoder

I haven't really taken a swing at error handling here, posting it up early for feedback on the API.

85: Unpin nightly date for bindings generation r=kvark a=grovesNL

rust-lang/rust#57915 allows us to run bindings generation with nightly again, so we shouldn't have to pin to an older nightly date anymore (assuming everyone has a recent nightly).

Co-authored-by: Tristam MacDonald <tristam@trist.am>
Co-authored-by: Joshua Groves <josh@joshgroves.com>
@bors bors bot merged commit d179b5a into gfx-rs:master Mar 1, 2019
@bors
Copy link
Contributor

bors bot commented Mar 1, 2019

Build succeeded

@swiftcoder swiftcoder deleted the create_buffer_mapped branch March 2, 2019 03:19
mitchmindtree pushed a commit to mitchmindtree/wgpu that referenced this pull request Feb 23, 2020
151: Add Lyon to the friends list r=kvark a=Veykril

Closes gfx-rs#84

Co-authored-by: Lukas Wirth <lukastw97@gmail.com>
kvark pushed a commit to kvark/wgpu that referenced this pull request Jun 3, 2021
151: Add Lyon to the friends list r=kvark a=Veykril

Closes gfx-rs#84

Co-authored-by: Lukas Wirth <lukastw97@gmail.com>
RandyMcMillan pushed a commit to RandyMcMillan/wgpu that referenced this pull request Jun 19, 2024
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.

2 participants