nordic-dev.net/wgpu
nordic-dev.net/wgpu
2
0
Fork 0
mirror of https://github.com/gfx-rs/wgpu.git synced 2025-02-18 09:52:35 +00:00
Code Issues Packages Projects Releases Wiki Activity

Decrease error variant code verbosity, apply reviews

Browse Source
This commit is contained in:
DasEtwas 2021-01-11 23:46:26 +01:00
parent 151d521ad9
commit 93f68add27
1 changed files with 5 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
wgpu-core/src/command/render.rs
View File

@ -389,11 +389,7 @@ pub enum RenderPassErrorInner {
InvalidAttachment(id::TextureViewId),
#[error("necessary attachments are missing")]
MissingAttachments,
#[error("color and/or depth attachments have differing sizes: a {mismatching_attachment_type_name} attachment of size {mismatching_dimensions_width}x{mismatching_dimensions_height} is not compatible with a {previous_attachment_type_name} attachment's size ({previous_dimensions_height}x{previous_dimensions_width})",
previous_dimensions_width = previous_dimensions.0,
previous_dimensions_height = previous_dimensions.1,
mismatching_dimensions_width = mismatching_dimensions.0,
mismatching_dimensions_height = mismatching_dimensions.1,
#[error("color and/or depth attachments have differing sizes: a {mismatching_attachment_type_name} attachment of size {mismatching_dimensions:?} is not compatible with a {previous_attachment_type_name} attachment's size {previous_dimensions:?}",
)]
AttachmentsDimensionMismatch {
/// depth or color
@ -538,10 +534,9 @@ impl<'a, B: GfxBackend> RenderPassInfo<'a, B> {
let mut render_attachments = AttachmentDataVec::<RenderAttachment>::new();
// need to be initialized to None!, check `AttachmentsDimensionMismatch` error
let mut attachment_width = None;
let mut attachment_height = None;
let mut attachment_type_name = None;
let mut attachment_type_name = "";
let mut mismatching_dimensions = None;
@ -598,7 +593,7 @@ impl<'a, B: GfxBackend> RenderPassInfo<'a, B> {
));
} else {
// this attachment's size was successfully inserted into `attachment_width` and `attachment_height`
attachment_type_name = Some("depth");
attachment_type_name = "depth";
}
depth_stencil_aspects = view.aspects;
@ -673,7 +668,7 @@ impl<'a, B: GfxBackend> RenderPassInfo<'a, B> {
));
} else {
// this attachment's size was successfully inserted into `attachment_width` and `attachment_height`
attachment_type_name = Some("color");
attachment_type_name = "color";
}
let layouts = match view.inner {
@ -741,7 +736,7 @@ impl<'a, B: GfxBackend> RenderPassInfo<'a, B> {
return Err(RenderPassErrorInner::AttachmentsDimensionMismatch {
// okay to unwrap: for `mismatching_dimensions` to be set, `attachment_{width/height}` must have been
// set, which includes `attachment_type_name` being set in a later `else` block
previous_attachment_type_name: attachment_type_name.unwrap(),
previous_attachment_type_name: attachment_type_name,
mismatching_attachment_type_name,
previous_dimensions,
mismatching_dimensions,