From 4ef6eb8a8535e9c6d8339497ab7c6f31dc47a115 Mon Sep 17 00:00:00 2001 From: Carter Anderson Date: Mon, 9 Nov 2020 14:12:42 -0800 Subject: [PATCH] adjust to new clippy lints (#826) --- crates/bevy_core/src/task_pool_options.rs | 10 ++++----- crates/bevy_gltf/src/loader.rs | 26 +++++++++++++---------- crates/bevy_render/src/texture/texture.rs | 6 ++++-- crates/bevy_scene/src/serde.rs | 11 +++++----- 4 files changed, 29 insertions(+), 24 deletions(-) diff --git a/crates/bevy_core/src/task_pool_options.rs b/crates/bevy_core/src/task_pool_options.rs index 902e8bf7e1..7f850f0ab3 100644 --- a/crates/bevy_core/src/task_pool_options.rs +++ b/crates/bevy_core/src/task_pool_options.rs @@ -82,11 +82,11 @@ impl Default for DefaultTaskPoolOptions { impl DefaultTaskPoolOptions { /// Create a configuration that forces using the given number of threads. pub fn with_num_threads(thread_count: usize) -> Self { - let mut options = Self::default(); - options.min_total_threads = thread_count; - options.max_total_threads = thread_count; - - options + DefaultTaskPoolOptions { + min_total_threads: thread_count, + max_total_threads: thread_count, + ..Default::default() + } } /// Inserts the default thread pools into the given resource map based on the configured values diff --git a/crates/bevy_gltf/src/loader.rs b/crates/bevy_gltf/src/loader.rs index 419a3b1051..5131ab4dc0 100644 --- a/crates/bevy_gltf/src/loader.rs +++ b/crates/bevy_gltf/src/loader.rs @@ -230,14 +230,16 @@ fn load_node( gltf::camera::Projection::Orthographic(orthographic) => { let xmag = orthographic.xmag(); let ymag = orthographic.ymag(); - let mut orthographic_projection: OrthographicProjection = Default::default(); - orthographic_projection.left = -xmag; - orthographic_projection.right = xmag; - orthographic_projection.top = ymag; - orthographic_projection.bottom = -ymag; - orthographic_projection.far = orthographic.zfar(); - orthographic_projection.near = orthographic.znear(); - orthographic_projection.get_projection_matrix(); + let orthographic_projection: OrthographicProjection = OrthographicProjection { + left: -xmag, + right: xmag, + top: ymag, + bottom: -ymag, + far: orthographic.zfar(), + near: orthographic.znear(), + ..Default::default() + }; + node.with(Camera { name: Some(base::camera::CAMERA2D.to_owned()), projection_matrix: orthographic_projection.get_projection_matrix(), @@ -246,9 +248,11 @@ fn load_node( node.with(orthographic_projection); } gltf::camera::Projection::Perspective(perspective) => { - let mut perspective_projection: PerspectiveProjection = Default::default(); - perspective_projection.fov = perspective.yfov(); - perspective_projection.near = perspective.znear(); + let mut perspective_projection: PerspectiveProjection = PerspectiveProjection { + fov: perspective.yfov(), + near: perspective.znear(), + ..Default::default() + }; if let Some(zfar) = perspective.zfar() { perspective_projection.far = zfar; } diff --git a/crates/bevy_render/src/texture/texture.rs b/crates/bevy_render/src/texture/texture.rs index 9f1ff403f9..61d25284af 100644 --- a/crates/bevy_render/src/texture/texture.rs +++ b/crates/bevy_render/src/texture/texture.rs @@ -48,8 +48,10 @@ impl Texture { } pub fn new_fill(size: Vec2, pixel: &[u8], format: TextureFormat) -> Self { - let mut value = Self::default(); - value.format = format; + let mut value = Texture { + format, + ..Default::default() + }; value.resize(size); debug_assert_eq!( diff --git a/crates/bevy_scene/src/serde.rs b/crates/bevy_scene/src/serde.rs index 0fe51b08db..8a9a47c3bf 100644 --- a/crates/bevy_scene/src/serde.rs +++ b/crates/bevy_scene/src/serde.rs @@ -92,12 +92,11 @@ impl<'a, 'de> DeserializeSeed<'de> for SceneDeserializer<'a> { where D: serde::Deserializer<'de>, { - let mut scene = DynamicScene::default(); - scene.entities = deserializer.deserialize_seq(SceneEntitySeqVisitor { - property_type_registry: self.property_type_registry, - })?; - - Ok(scene) + Ok(DynamicScene { + entities: deserializer.deserialize_seq(SceneEntitySeqVisitor { + property_type_registry: self.property_type_registry, + })?, + }) } }