Intellisense

[gpx1000]

Included vulkan_hpp_intellisense.h across all attachment files to resolve C++20 module intellisense issues in IDEs. This is a blind test to help fix: #75

[SaschaWillems]

Sadly doesn't do anything for me with VS2022. Still no intellisense.

Only way I can get intellisense to work is remove the VulkanCppModule project and explicitly add the vulkan-hpp module to each project. Might be an option.

[SaschaWillems]

Would love to do a PR myself, but my CMake knowledge simply isn't goot enough. I can't get it to properly work :/

[gpx1000]

I tried a version with direct module linking, try turning that option on and see if it does the trick for you. If it works, maybe we make it a default build method.

[SaschaWillems]

Sorry for the stupid question, but how do I actually set/enable that option?

[gpx1000]

No question is stupid; if you're using the command line just pass it in to cmake like this:
cmake -d<OPTION_NAME>= ..
or:
mkdir build && cd build
cmake -dUSE_DIRECT_MODULE_IMPORT=ON ..

[gpx1000]

I also left comments in the cmake to detail how to set the option.

[SaschaWillems]

Didn't seem to change anything for me on VS2022. Will try a few different things first and get back to you.

[gpx1000]

okay, this version I tried to directly tell intellisense that it needs to include vulkan.hpp directly.
#ifdef INTELLISENSE
#include <vulkan/vulkan.hpp>
#else
import vulkan_hpp;
#endif
https://stackoverflow.com/questions/26724244/is-there-a-macro-defined-when-intellisense-is-parsing that macro should hopefully be the key to getting intellisense to behave.

[SaschaWillems]

That does give me Intellisense, but only partially:

Stuff inside the RAII namespace (?) still doesn't get resolved.

But does that still use the module then, or just the normal header?

[gpx1000]

Yep, it still uses the module, only when INTELLISENSE is running (i.e. the static analysis step of "compiling" the source file) will it include the whole vulkan hpp file. Rest of the time it should be the module. Think I need to include vulkan_raii.hpp to get the right stuff instead of vulkan hpp by itself. I hate doing this, macros for a static analysis platform are silly, detract from understanding the code and don't precisely give what should exist from the compiler which is the whole reason static analysis exists. But I'll get off my soapbox as I'm sure Intellisense will eventually work with modules sooner or later.

[gpx1000]

Okay, all of the tutorial should now resolve correctly under intellisense. Lemme know if I missed anything.

[SaschaWillems]

That did it. Intellisense is now properly working for me 👍🏻