Change include of vulkan.h on windows platforms. about volk HOT 4 CLOSED

As an addition, since the Windows redefinitions must only be done in the .c file, there is no problem including windows.h further on the user's code.

from volk.

Is the manual definition of LoadLibraryA etc. required in volk.c, or is the main goal to reduce the include cost of volk.h?

It looks like I can get this to work with both, but the amount of manual definitions in the header is slightly smaller than the full number of definitions required to compile volk.c.

from volk.

Something like this seems to work in the header - I think this is more correct than the version proposed in the tweet.

#ifndef VULKAN_H_
#	ifdef VK_USE_PLATFORM_WIN32_KHR
#		include <vulkan/vk_platform.h>
#		include <vulkan/vulkan_core.h>

		typedef unsigned long DWORD;
		typedef const wchar_t* LPCWSTR;
		typedef void* HANDLE;
		typedef struct HINSTANCE__ *HINSTANCE;
		typedef struct HWND__* HWND;
		typedef struct HMONITOR__* HMONITOR;
		typedef struct _SECURITY_ATTRIBUTES SECURITY_ATTRIBUTES;

#		include <vulkan/vulkan_win32.h>
#	else
#		include <vulkan/vulkan.h>
#	endif
#endif

from volk.

Nice! This solution looks better indeed.

from volk.