Skip to content

Commit

Permalink
error lint
Browse files Browse the repository at this point in the history
  • Loading branch information
Wumpf committed Mar 31, 2023
1 parent ae961a2 commit 36a065e
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 8 deletions.
8 changes: 4 additions & 4 deletions crates/re_viewer/src/ui/view_spatial/ui_2d.rs
Original file line number Diff line number Diff line change
Expand Up @@ -326,8 +326,8 @@ fn view_2d_scrollable(

// TODO(andreas): separate setup for viewbuilder doesn't make sense.
let mut view_builder = ViewBuilder::default();
if let Err(error) = view_builder.setup_view(ctx.render_ctx, target_config) {
re_log::error!("Failed to setup view: {}", error);
if let Err(err) = view_builder.setup_view(ctx.render_ctx, target_config) {
re_log::error!("Failed to setup view: {}", err);
return response;
}

Expand Down Expand Up @@ -489,8 +489,8 @@ fn view_2d_scrollable(
&ScreenBackground::ClearColor(parent_ui.visuals().extreme_bg_color.into()),
) {
Ok(command_buffer) => command_buffer,
Err(error) => {
re_log::error!("Failed to fill view builder: {}", error);
Err(err) => {
re_log::error!("Failed to fill view builder: {}", err);
return response;
}
};
Expand Down
8 changes: 4 additions & 4 deletions crates/re_viewer/src/ui/view_spatial/ui_3d.rs
Original file line number Diff line number Diff line change
Expand Up @@ -339,8 +339,8 @@ pub fn view_3d(

let mut view_builder = ViewBuilder::default();
// TODO(andreas): separate setup_view doesn't make sense, add a `new` method instead.
if let Err(error) = view_builder.setup_view(ctx.render_ctx, target_config) {
re_log::error!("Failed to setup view: {}", error);
if let Err(err) = view_builder.setup_view(ctx.render_ctx, target_config) {
re_log::error!("Failed to setup view: {}", err);
return;
}

Expand Down Expand Up @@ -570,8 +570,8 @@ pub fn view_3d(
&ScreenBackground::GenericSkybox,
) {
Ok(command_buffer) => command_buffer,
Err(error) => {
re_log::error!("Failed to fill view builder: {}", error);
Err(err) => {
re_log::error!("Failed to fill view builder: {}", err);
return;
}
};
Expand Down

0 comments on commit 36a065e

Please sign in to comment.