Skip to content

Commit 2fc33cc

Browse files
committed
post-rebase shenaniganeries
1 parent 53f563c commit 2fc33cc

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

crates/viewer/re_viewer_context/src/viewer_context.rs

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -182,21 +182,22 @@ impl<'a> ViewerContext<'a> {
182182
if let Some(placeholder) = reflection.custom_placeholder.as_ref() {
183183
return placeholder.clone();
184184
}
185-
&reflection.datatype
185+
reflection.datatype.clone()
186186
} else {
187187
self.recording_engine()
188188
.store()
189189
.lookup_datatype(&component)
190-
.or_else(|| self.blueprint_store().lookup_datatype(&component))
190+
.cloned()
191+
.or_else(|| self.blueprint_engine().store().lookup_datatype(&component).cloned())
191192
.unwrap_or_else(|| {
192193
re_log::error_once!("Could not find datatype for component {component}. Using null array as placeholder.");
193-
&re_chunk::external::arrow2::datatypes::DataType::Null
194+
re_chunk::external::arrow2::datatypes::DataType::Null
194195
})
195196
};
196197

197198
// TODO(andreas): Is this operation common enough to cache the result? If so, here or in the reflection data?
198199
// The nice thing about this would be that we could always give out references (but updating said cache wouldn't be easy in that case).
199-
re_types::reflection::generic_placeholder_for_datatype(datatype)
200+
re_types::reflection::generic_placeholder_for_datatype(&datatype)
200201
}
201202
}
202203

0 commit comments

Comments
 (0)