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

Add VK_KHR/EXT_buffer_device_address extension #405

Merged
merged 1 commit into from
Mar 20, 2021

Conversation

Rua
Copy link
Contributor

@Rua Rua commented Mar 16, 2021

This extension comes in both a KHR and an EXT version, so I included them both for completeness.

@MaikKlein MaikKlein merged commit 98d66c6 into ash-rs:master Mar 20, 2021
@Rua Rua deleted the VK_KHR_buffer_device_address branch April 30, 2021 15:18
MarijnS95 added a commit that referenced this pull request May 8, 2021
Fixes: 98d66c6 ("Add VK_KHR/EXT_buffer_device_address extension (#405)")
MaikKlein pushed a commit that referenced this pull request May 8, 2021
…ublic (#430)

* GetPhysicalDeviceProperties2: Make API functions public

Fixes: 5eb39fe ("Add VK_KHR_get_physical_device_properties2 extension (#400)")

* BufferDeviceAddress: Make API functions public

Fixes: 98d66c6 ("Add VK_KHR/EXT_buffer_device_address extension (#405)")

* GetMemoryRequirements2: Make API functions public

Fixes: d8d7423 ("Add VK_KHR_get_memory_requirements2 extension (#401)")

* Maintenance1/Maintenance3: Make API functions public

Fixes: a0a1f5d ("Add VK_KHR_maintenance extensions (#406)")

* Globally remove all `allow(dead_code)` exceptions

This is hiding the fact that some extension functions are inadvertently
not public and hence unusable by crate users, nor does it enforce clean
coding practices.

In addition remove `ash/src/allocator.rs` which does not appear to be
used (anymore?) and isn't in a working state anyway.
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