-
Notifications
You must be signed in to change notification settings - Fork 25
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: Fixup and add Layer Dispatch Table headers #69
util: Fixup and add Layer Dispatch Table headers #69
Conversation
d268a94
to
8936d32
Compare
ee2ad3f
to
39fb8f7
Compare
39fb8f7
to
7ccda3a
Compare
7ccda3a
to
c8b671c
Compare
b15c42c
to
1f6389a
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.
Aside from function naming convention nitpick
1f6389a
to
a14da03
Compare
Fixes the vk_layer_dispatch_table.h header file so that they can be used in other projects. The contents of this header and vk_dispatch_table_helper.h have been moved into a new header vul_dispatch_table.h. The structs VulDeviceDispatchTable and VulInstanceDispatchTable struct contain function pointers for the device and instance, respectively. The functions vul_init_device_dispatch_table and vul_init_instance_dispatch_table fill out the aforementioned structs, making the task of setting up the disptach table in a layer much simpler.
a14da03
to
50657e6
Compare
(Code generation does **not** happen automatically at build time.) |
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.
(Code generation does **not** happen automatically at build time.) | |
Code generation does **not** happen automatically at build time. To make the library more consumable. |
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.
I think theres been enough force pushes for one PR 😛
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.
Yeah that's a FFTI no worries
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.
To make the library more consumable.
Also thats a sentence fragment. what is making the library more consumable? Should just drop the period.
typedef PFN_vkVoidFunction(VKAPI_PTR *PFN_GetPhysicalDeviceProcAddr)(VkInstance instance, const char *pName); | ||
|
||
// Instance function pointer dispatch table | ||
typedef struct VulInstanceDispatchTable_ { |
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.
VulInstanceDispatchTable_
? why the underscore
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.
Thats how C types are defined - notice the VulInstanceDispatchTable
after the }
.
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.
I am error
Fixes the vk_layer_dispatch_table.h and vk_dispatch_table_helper.h header files so that they can be used in other projects.
The vk_layer_dispatch_table.h header contains the VkLayerDispatchTable struct containing device function pointers and the VkLayerInstanceDispatchTable struct containing instance function pointers. These structs simplify the storage of Vulkan function pointers.
The vk_dispatch_table_helper.h header contains functions which fill out the structs defined in vk_layer_dispatch_table.h with function pointers. These functions automate acquiring the function pointers to the next layer in the chain.