Skip to content

Commit

Permalink
fixes
Browse files Browse the repository at this point in the history
- vulkaninfo already had a #VK_NO_PROTOTYPES somewhere
- macOS DemoViewController includes of mvk_vulkan.h are confusing Volk and don't seem to be necessary,
  since cube.c and cube.cpp #include vulkan.h and vulkan.hpp, respectively
  • Loading branch information
lunarpapillo committed May 16, 2024
1 parent 808aae2 commit 27d7d76
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 7 deletions.
2 changes: 0 additions & 2 deletions cube/macOS/cube/DemoViewController.m
Original file line number Diff line number Diff line change
Expand Up @@ -19,8 +19,6 @@
#import "DemoViewController.h"
#import <QuartzCore/CAMetalLayer.h>

#include <MoltenVK/mvk_vulkan.h>

#include "cube.c"

#pragma mark -
Expand Down
2 changes: 0 additions & 2 deletions cube/macOS/cubepp/DemoViewController.mm
Original file line number Diff line number Diff line change
Expand Up @@ -19,8 +19,6 @@
#import "DemoViewController.h"
#import <QuartzCore/CAMetalLayer.h>

#include <MoltenVK/mvk_vulkan.h>

#include "cube.cpp"

#pragma mark -
Expand Down
3 changes: 0 additions & 3 deletions vulkaninfo/vulkaninfo.h
Original file line number Diff line number Diff line change
Expand Up @@ -86,9 +86,6 @@
#include <wayland-client.h>
#endif

// Volk requires VK_NO_PROTOTYPES before including vulkan.h
#define VK_NO_PROTOTYPES

#include <vulkan/vulkan.h>

#define VOLK_IMPLEMENTATION
Expand Down

0 comments on commit 27d7d76

Please sign in to comment.