Skip to content

Commit 2fb1fd7

Browse files
authored
Merge pull request #1 from DasEtwas/patch-1
Improve error messages
2 parents 4af6e74 + 2af5c2e commit 2fb1fd7

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

spirv_struct_layout/Cargo.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,4 +12,4 @@ readme = "../README.md"
1212

1313
[dependencies]
1414
spirv-struct-layout-derive = { version = "0.1", path = "../spirv_struct_layout_derive" }
15-
spirq = "0.2"
15+
spirq = "0.4"

spirv_struct_layout_derive/src/lib.rs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -23,9 +23,9 @@ pub fn spirv_layout_derive(input: proc_macro::TokenStream) -> proc_macro::TokenS
2323
#repr_check
2424

2525
let spv: spirq::SpirvBinary = spirv.into();
26-
let entries = spv.reflect().unwrap();
26+
let entries = spv.reflect().expect("Failed to parse SPIR-V");
2727

28-
let buffer_desc = entries[0].resolve_desc(spirq::sym::Sym::new(name)).unwrap();
28+
let buffer_desc = entries[0].resolve_desc(spirq::sym::Sym::new(name)).expect(format!("Failed to find symbol with name \"{}\"", name).as_str());
2929

3030
let mut _rust_offset = 0;
3131

@@ -64,7 +64,7 @@ fn build_function_body(data: &Data) -> TokenStream {
6464

6565
if let Some(desc) = buffer_desc.desc_ty.resolve(spirq::sym::Sym::new(&symbol)) {
6666
let spirv_offset = desc.offset;
67-
let spirv_size = desc.ty.nbyte().unwrap();
67+
let spirv_size = desc.ty.nbyte().expect(format!("Rust struct field named \"{}\" does not have a basic data type (float, vec, mat, array, struct) as a SPIR-V counterpart", &symbol).as_str());
6868

6969
assert_eq!(
7070
spirv_size, rust_size,

0 commit comments

Comments
 (0)