Skip to content

Validate ext_debug_utils extension when calling set_debug_utils_object_name#2762

Merged
marc0246 merged 2 commits intovulkano-rs:masterfrom
Matusson:set-debug-utils-name-validation
Jan 7, 2026
Merged

Validate ext_debug_utils extension when calling set_debug_utils_object_name#2762
marc0246 merged 2 commits intovulkano-rs:masterfrom
Matusson:set-debug-utils-name-validation

Conversation

@Matusson
Copy link
Copy Markdown
Contributor

  1. Update documentation to reflect any user-facing changes - in this repository.

  2. Make sure that the changes are covered by unit-tests.

  3. Run cargo clippy on the changes.

  4. Run cargo +nightly fmt on the changes.

  5. Please put changelog entries in the description of this Pull Request
    if knowledge of this change could be valuable to users. No need to put the
    entries to the changelog directly, they will be transferred to the changelog
    file by maintainers right after the Pull Request merge.

    Please remove any items from the template below that are not applicable.

  6. Describe in common words what is the purpose of this change, related
    GitHub Issues, and highlight important implementation aspects.

This change adds validation for Device::set_debug_utils_object_name that checks if the required extension is enabled on the instance. Previously calling it without the extension enabled would panic in ash code, this hopefully makes the issue easier to understand.

Changelog:

### Breaking changes
Changes to `devices`:
- `Device::set_debug_utils_object_name` now validates that the `ext_debug_utils` extension is enabled on the instance.
- `Device::set_debug_utils_object_name` is now marked unsafe.

@marc0246 marc0246 added this pull request to the merge queue Jan 7, 2026
Merged via the queue into vulkano-rs:master with commit 9457f4c Jan 7, 2026
8 checks passed
marc0246 added a commit that referenced this pull request Jan 7, 2026
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