clippy fixes

This commit is contained in:
Robin Appelman 2024-12-14 15:03:59 +01:00
commit 3175e0840c
4 changed files with 16 additions and 18 deletions

View file

@ -83,7 +83,7 @@ pub fn load_material(
let bump_map = material.bump_map().and_then(|path| { let bump_map = material.bump_map().and_then(|path| {
Some(TextureData { Some(TextureData {
image: load_texture(&path, loader).ok()?, image: load_texture(path, loader).ok()?,
name: path.into(), name: path.into(),
}) })
}); });
@ -107,10 +107,10 @@ fn load_texture(name: &str, loader: &Loader) -> Result<DynamicImage, Error> {
"materials/{}.vtf", "materials/{}.vtf",
name.trim_end_matches(".vtf").trim_start_matches('/') name.trim_end_matches(".vtf").trim_start_matches('/')
); );
let mut raw = loader let raw = loader
.load(&path)? .load(&path)?
.ok_or(Error::Other(format!("Can't find file {}", path)))?; .ok_or(Error::Other(format!("Can't find file {}", path)))?;
let vtf = VTF::read(&mut raw)?; let vtf = VTF::read(&raw)?;
let image = vtf.highres_image.decode(0)?; let image = vtf.highres_image.decode(0)?;
Ok(image) Ok(image)
} }

View file

@ -25,7 +25,7 @@ impl From<&vmdl::vvd::Vertex> for Vertex {
fn from(vertex: &vmdl::vvd::Vertex) -> Self { fn from(vertex: &vmdl::vvd::Vertex) -> Self {
Vertex { Vertex {
position: vertex.position.into(), position: vertex.position.into(),
uv: vertex.texture_coordinates.into(), uv: vertex.texture_coordinates,
normal: vertex.normal.into(), normal: vertex.normal.into(),
} }
} }

View file

@ -115,12 +115,10 @@ fn main() -> Result<(), Error> {
); );
directional[0].intensity = directional_intensity; directional[0].intensity = directional_intensity;
directional[1].intensity = directional_intensity; directional[1].intensity = directional_intensity;
if ui.checkbox(&mut shadows_enabled, "Shadows").clicked() { if ui.checkbox(&mut shadows_enabled, "Shadows").clicked() && !shadows_enabled {
if !shadows_enabled {
directional[0].clear_shadow_map(); directional[0].clear_shadow_map();
directional[1].clear_shadow_map(); directional[1].clear_shadow_map();
} }
}
ui.label("Debug options"); ui.label("Debug options");
ui.radio_value(&mut debug_type, DebugType::NONE, "None"); ui.radio_value(&mut debug_type, DebugType::NONE, "None");
@ -175,8 +173,10 @@ fn main() -> Result<(), Error> {
lights, lights,
), ),
DebugType::DEPTH => { DebugType::DEPTH => {
let mut depth_material = DepthMaterial::default(); let depth_material = DepthMaterial {
depth_material.max_distance = Some(depth_max); max_distance: Some(depth_max),
..DepthMaterial::default()
};
target.render_with_material( target.render_with_material(
&depth_material, &depth_material,
&camera, &camera,

View file

@ -214,7 +214,7 @@ struct FrameValues<'a> {
data: &'a [u8], // data starting at self.header data: &'a [u8], // data starting at self.header
} }
impl<'a> FrameValues<'a> { impl FrameValues<'_> {
pub fn get(&self, index: u8) -> Result<u16, ModelError> { pub fn get(&self, index: u8) -> Result<u16, ModelError> {
if self.header.total <= index { if self.header.total <= index {
let offset_count = self.header.valid + 1; let offset_count = self.header.valid + 1;
@ -368,9 +368,7 @@ fn read_animation(
header_offset: usize, header_offset: usize,
frames: usize, frames: usize,
) -> Result<(Animation, usize), ModelError> { ) -> Result<(Animation, usize), ModelError> {
let data = data let data = data.get(header_offset..).ok_or(ModelError::OutOfBounds {
.get(header_offset..)
.ok_or_else(|| ModelError::OutOfBounds {
data: "animation data", data: "animation data",
offset: header_offset, offset: header_offset,
})?; })?;