Skip to content

Commit 58b933c

Browse files
committed
Format + Clippy fixes
1 parent 9d222d0 commit 58b933c

File tree

5 files changed

+29
-29
lines changed

5 files changed

+29
-29
lines changed

src/context.rs

Lines changed: 14 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -811,11 +811,11 @@ fn update_widgets(
811811
if widget_name != prev_widget_name {
812812
// It doesn't matter we always need to remove state
813813
remove_state.push(changed_entity.0);
814-
if let Some(_) = tree.parent(*changed_entity) {
814+
if tree.parent(*changed_entity).is_some() {
815815
for child in
816816
tree.down_iter_at(*changed_entity, false)
817817
{
818-
info!(
818+
trace!(
819819
"Removing AvsB children {}::{}",
820820
entity_ref
821821
.get::<WidgetName>()
@@ -827,25 +827,25 @@ fn update_widgets(
827827
if let Ok(order_tree) =
828828
order_tree.try_read()
829829
{
830-
if let Some(order_tree_parent) = order_tree
831-
.parent(*changed_entity) {
830+
if let Some(order_tree_parent) =
831+
order_tree.parent(*changed_entity)
832+
{
832833
'back_up: for sibling in order_tree
833834
.child_iter(order_tree_parent)
834835
{
835-
dbg!(sibling, changed_entity);
836836
if sibling == *changed_entity {
837837
continue 'back_up;
838838
}
839-
for child in
840-
tree.down_iter_at(sibling, true)
841-
{
839+
for child in tree
840+
.down_iter_at(sibling, true)
841+
{
842842
// Ignore self again.
843-
dbg!(child);
844843
if child == *parent {
845844
continue;
846845
}
847846
if let Some(entity_ref) =
848-
world.get_entity(child.0)
847+
world
848+
.get_entity(child.0)
849849
{
850850
if let Some(children) =
851851
entity_ref
@@ -857,7 +857,7 @@ fn update_widgets(
857857
.0,
858858
)
859859
{
860-
info!("Caught an entity that was marked as deleted but wasn't! {:?} in {:?}", changed_entity.0, child.0);
860+
trace!("Caught an entity that was marked as deleted but wasn't! {:?} in {:?}", changed_entity.0, child.0);
861861
// Don't despawn changed entity because it exists as a child passed via props
862862
should_delete =
863863
false;
@@ -885,9 +885,10 @@ fn update_widgets(
885885
}
886886
}
887887

888-
for entity in remove_state.iter() {
888+
for entity in remove_state.iter() {
889889
if let Some(state_entity) = widget_state.remove(*entity) {
890-
if let Some(mut entity_mut) = world.get_entity_mut(state_entity) {
890+
if let Some(mut entity_mut) = world.get_entity_mut(state_entity)
891+
{
891892
entity_mut.remove_parent();
892893
entity_mut.despawn_recursive();
893894
}

src/context_entities.rs

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,10 @@ impl ContextEntities {
3030
inner.insert(T::default().type_id(), context_entity);
3131
}
3232

33-
pub fn get_context_entity<T: Default + 'static>(&self, parent_id: Option<Entity>) -> Option<Entity> {
33+
pub fn get_context_entity<T: Default + 'static>(
34+
&self,
35+
parent_id: Option<Entity>,
36+
) -> Option<Entity> {
3437
if !self.ce.contains_key(&parent_id) {
3538
return None;
3639
}

src/render/ui_pass.rs

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ use bevy::ecs::prelude::*;
44
use bevy::prelude::{Color, Image};
55
use bevy::render::render_asset::RenderAssets;
66
use bevy::render::render_phase::{
7-
BatchedPhaseItem, CachedRenderPipelinePhaseItem, DrawFunctionId, PhaseItem, DrawFunctions,
7+
BatchedPhaseItem, CachedRenderPipelinePhaseItem, DrawFunctionId, DrawFunctions, PhaseItem,
88
};
99
use bevy::render::render_resource::{CachedRenderPipelineId, RenderPassColorAttachment};
1010
use bevy::render::{
@@ -233,8 +233,12 @@ impl Node for MainPassUINode {
233233

234234
let mut tracked_pass =
235235
render_context.begin_tracked_render_pass(pass_descriptor);
236-
237-
for item in transparent_opacity_phase.items.iter().filter(|i| i.opacity_layer == layer_id) {
236+
237+
for item in transparent_opacity_phase
238+
.items
239+
.iter()
240+
.filter(|i| i.opacity_layer == layer_id)
241+
{
238242
let draw_function = draw_functions.get_mut(item.draw_function()).unwrap();
239243
draw_function.draw(world, &mut tracked_pass, view_entity, item);
240244
}

src/render/unified/pipeline.rs

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -950,12 +950,7 @@ pub fn queue_quads_inner(
950950
position: final_position.into(),
951951
color,
952952
uv: [0.0; 4],
953-
pos_size: [
954-
0.0,
955-
0.0,
956-
sprite_rect.size().x,
957-
sprite_rect.size().y,
958-
],
953+
pos_size: [0.0, 0.0, sprite_rect.size().x, sprite_rect.size().y],
959954
});
960955
}
961956
*index += indices.len() as u32;

src/widget_context.rs

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -103,10 +103,7 @@ impl KayakWidgetContext {
103103
}
104104

105105
// Finally check root AKA no parent.
106-
if let Some(entity) = self
107-
.context_entities
108-
.get_context_entity::<T>(None)
109-
{
106+
if let Some(entity) = self.context_entities.get_context_entity::<T>(None) {
110107
return Some(entity);
111108
}
112109
}
@@ -200,10 +197,10 @@ impl KayakWidgetContext {
200197
/// widget_context.add_widget(None, root_entity);
201198
/// }
202199
///```
203-
pub fn spawn_widget<'a>(
200+
pub fn spawn_widget(
204201
&self,
205202
commands: &mut Commands,
206-
key: Option<&'a str>,
203+
key: Option<&str>,
207204
parent_id: Option<Entity>,
208205
) -> Entity {
209206
let mut entity = None;

0 commit comments

Comments
 (0)