allemang.d/vulkan-zig
1
0
Fork 0
forked from mirror/vulkan-zig
Code Pull Requests Activity

Merge pull request #141 from slimsag/master

Browse Source 
always provide default opaque{} external type implementations
This commit is contained in:
Robin Voetter
2024-06-02 23:25:51 +02:00
committed by GitHub
parent ac5ad34a95 cfaf58a4de
commit f2c2e0ff80
2 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
README.md
View File

@@ -349,9 +349,9 @@ Note that this information is not everywhere as useful in the registry, leading
Defaults with the same ABI layout are generated for most platform-defined types. These can either by bitcasted to, or overridden by defining them in the project root:
```zig
pub const xcb_connection_t = if (@hasDecl(root, "xcb_connection_t")) root.xcb_connection_t else @Type(.Opaque);
pub const xcb_connection_t = if (@hasDecl(root, "xcb_connection_t")) root.xcb_connection_t else opaque{};
```
For some times (such as those from Google Games Platform) no default is known. Usage of these without providing a concrete type in the project root generates a compile error.
For some times (such as those from Google Games Platform) no default is known, but an `opaque{}` will be used by default. Usage of these without providing a concrete type in the project root is likely an error.
### Shader compilation

2
src/vulkan/render.zig
View File

@@ -1031,7 +1031,7 @@ fn Renderer(comptime WriterType: type) type {
try self.writer.writeAll(default);
try self.writer.writeAll(";\n");
} else {
try self.writer.print("@compileError(\"Missing type definition of '{s}'\");\n", .{name});
try self.writer.print("opaque {{}};\n", .{});
}
}