Skip to content

Add buffer arrays#1315

Merged
bors[bot] merged 1 commit intogfx-rs:masterfrom
csnewman:add-buffer-arrays
Apr 29, 2021
Merged

Add buffer arrays#1315
bors[bot] merged 1 commit intogfx-rs:masterfrom
csnewman:add-buffer-arrays

Conversation

@csnewman
Copy link
Copy Markdown
Contributor

@csnewman csnewman commented Apr 8, 2021

Connections
gfx-rs/wgpu-rs#850

Description
Adds buffer array bindings

Testing
TODO

})
}

fn create_buffer_descriptor<'a>(
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This method has too many arguments causing a clippy error. Not sure the best way to clean this up.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

in the worst case, just add the [allow(...)] on it

@cwfitzgerald cwfitzgerald self-requested a review April 8, 2021 20:17
Copy link
Copy Markdown
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.

Looks reasonable to me!

@kvark
Copy link
Copy Markdown
Member

kvark commented Apr 12, 2021

Let's make clippy happy and turn this into a non-draft PR

@kvark
Copy link
Copy Markdown
Member

kvark commented Apr 28, 2021

@csnewman can we finish this for 0.8?

@kvark kvark added this to the Version 0.8 milestone Apr 28, 2021
@kvark kvark force-pushed the add-buffer-arrays branch from 618cf36 to 48249e5 Compare April 29, 2021 04:35
@kvark kvark marked this pull request as ready for review April 29, 2021 04:35
@kvark kvark force-pushed the add-buffer-arrays branch from 48249e5 to 904ec7a Compare April 29, 2021 04:38
@kvark
Copy link
Copy Markdown
Member

kvark commented Apr 29, 2021

Taking this over now

@kvark
Copy link
Copy Markdown
Member

kvark commented Apr 29, 2021

bors r+

@bors
Copy link
Copy Markdown
Contributor

bors bot commented Apr 29, 2021

@bors bors bot merged commit acd21ee into gfx-rs:master Apr 29, 2021
bors bot added a commit to gfx-rs/wgpu-rs that referenced this pull request Apr 29, 2021
850: Add uniform and storage buffer arrays r=kvark a=csnewman

**Connections**
gfx-rs/wgpu#1315

**Description**
Adds uniform and storage buffer arrays

Usage:
```
wgpu::BindGroupEntry {
    binding: 0,
    resource: BindingResource::BufferArray(&[
        buffer1.as_entire_buffer_binding(),
        buffer2.as_entire_buffer_binding(),
    ]),
}
```



Co-authored-by: Chandler Newman <chandler2newman@hotmail.co.uk>
kvark pushed a commit to kvark/wgpu that referenced this pull request Jun 3, 2021
850: Add uniform and storage buffer arrays r=kvark a=csnewman

**Connections**
gfx-rs#1315

**Description**
Adds uniform and storage buffer arrays

Usage:
```
wgpu::BindGroupEntry {
    binding: 0,
    resource: BindingResource::BufferArray(&[
        buffer1.as_entire_buffer_binding(),
        buffer2.as_entire_buffer_binding(),
    ]),
}
```



Co-authored-by: Chandler Newman <chandler2newman@hotmail.co.uk>
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.

3 participants