-
Notifications
You must be signed in to change notification settings - Fork 28
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
util: Add vk_struct_helper.hpp #110
util: Add vk_struct_helper.hpp #110
Conversation
d8062e9
to
39a7c91
Compare
|
What about |
Thats a good idea due to the inclusion of pNext chain helpers as well. |
39a7c91
to
f1932e5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You need to update the chromium build.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good overall aside from chromium build issue.
f1932e5
to
ee66bcb
Compare
build.gn has been updated. |
ee66bcb
to
6656d65
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Update PR title with correct header file
Move the vk_typemap_helper.h file in Vulkan-ValidationLayers over to this repo with a different name to reflect the use of it Changes from that include: * Rename the functions to be more obvious as to what they do * Placing them in the `vku` namespace * Adding the `InitStructHelper` class which deduces the type based on the variable that is being initialized * Compiler error if there is no corresponding sType
6656d65
to
7fd27de
Compare
FWIW I'd start with updating the extension layer instead of vvl. |
Move the vk_struct_helper.h file in Vulkan-ValidationLayers over to this repo.
Changes from that include:
vku
namespaceInitStructHelper
class which deduces the type based on the variable that is being initializedResolves #74