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

layers: Warn if VkLayerSettingEXT has wrong type or duplicated #9032

Merged

Conversation

spencer-lunarg
Copy link
Contributor

@spencer-lunarg spencer-lunarg commented Dec 16, 2024

closes #9028

We ultimately need a more streamline, auto-generated way of doing this type of checks (else adding a setting only gets more complicated again)

For now, at least warn the user if they have the wrong VkLayerSettingTypeEXT or duplicated a setting

@spencer-lunarg spencer-lunarg requested a review from a team as a code owner December 16, 2024 06:31
@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 324969.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18337 running.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18337 failed.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 324991.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18338 running.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18338 passed.

layers/layer_options.cpp Outdated Show resolved Hide resolved
@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 325735.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18347 running.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18347 failed.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 325855.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18351 running.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 18351 passed.

@spencer-lunarg spencer-lunarg merged commit d659d80 into KhronosGroup:main Dec 17, 2024
21 checks passed
@spencer-lunarg spencer-lunarg deleted the spencer-lunarg-setting branch December 17, 2024 07:37
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.

Invalid settings from VK_EXT_layer_settings are not reported
4 participants