Tweak whitespace around use statements and other lint fixes

editor/src/communication/dispatcher.rs

@@ -42,9 +42,10 @@ impl Dispatcher {
 
 	#[remain::check]
 	pub fn handle_message<T: Into<Message>>(&mut self, message: T) {
+		use Message::*;
+
 		self.message_queue.push_back(message.into());
 
-		use Message::*;
 		while let Some(message) = self.message_queue.pop_front() {
 			// Skip processing of this message if it will be processed later
 			if SIDE_EFFECT_FREE_MESSAGES.contains(&message.to_discriminant()) && self.message_queue.contains(&message) {
@@ -106,6 +107,7 @@ impl Dispatcher {
 
 	fn log_message(&self, message: &Message) {
 		use Message::*;
+
 		if log::max_level() == log::LevelFilter::Trace
 			&& !(matches!(
 				message,
@@ -126,6 +128,7 @@ mod test {
 	use crate::message_prelude::*;
 	use crate::misc::test_utils::EditorTestUtils;
 	use crate::Editor;
+
 	use graphene::color::Color;
 	use graphene::Operation;
 

editor/src/document/artboard_message_handler.rs

@@ -26,8 +26,9 @@ impl ArtboardMessageHandler {
 impl MessageHandler<ArtboardMessage, (&mut LayerMetadata, &GrapheneDocument, &InputPreprocessorMessageHandler)> for ArtboardMessageHandler {
 	#[remain::check]
 	fn process_action(&mut self, message: ArtboardMessage, _data: (&mut LayerMetadata, &GrapheneDocument, &InputPreprocessorMessageHandler), responses: &mut VecDeque<Message>) {
-		// let (layer_metadata, document, ipp) = data;
 		use ArtboardMessage::*;
+
+		// let (layer_metadata, document, ipp) = data;
 		#[remain::sorted]
 		match message {
 			AddArtboard { top, left, height, width } => {

editor/src/document/document_message_handler.rs

@@ -78,10 +78,10 @@ impl DocumentMessageHandler {
 		let deserialized_result: Result<Self, DocumentError> = serde_json::from_str(serialized_content).map_err(|e| DocumentError::InvalidFile(e.to_string()));
 		match deserialized_result {
 			Ok(document) => {
-				if document.version != GRAPHITE_DOCUMENT_VERSION {
-					Err(DocumentError::InvalidFile("Graphite document version mismatch".to_string()))
-				} else {
+				if document.version == GRAPHITE_DOCUMENT_VERSION {
 					Ok(document)
+				} else {
+					Err(DocumentError::InvalidFile("Graphite document version mismatch".to_string()))
 				}
 			}
 			Err(e) => Err(e),
@@ -440,6 +440,7 @@ impl MessageHandler<DocumentMessage, &InputPreprocessorMessageHandler> for Docum
 	#[remain::check]
 	fn process_action(&mut self, message: DocumentMessage, ipp: &InputPreprocessorMessageHandler, responses: &mut VecDeque<Message>) {
 		use DocumentMessage::*;
+
 		#[remain::sorted]
 		match message {
 			AbortTransaction => {
@@ -585,7 +586,7 @@ impl MessageHandler<DocumentMessage, &InputPreprocessorMessageHandler> for Docum
 			}
 			ExportDocument => {
 				// TODO(MFISH33): Add Dialog to select artboards
-				let bbox = self.document_bounds().unwrap_or([DVec2::ZERO, ipp.viewport_bounds.size()]);
+				let bbox = self.document_bounds().unwrap_or_else(|| [DVec2::ZERO, ipp.viewport_bounds.size()]);
 				let size = bbox[1] - bbox[0];
 				let name = match self.name.ends_with(FILE_SAVE_SUFFIX) {
 					true => self.name.clone().replace(FILE_SAVE_SUFFIX, FILE_EXPORT_SUFFIX),

editor/src/document/layer_panel.rs

@@ -114,6 +114,7 @@ impl fmt::Display for LayerDataTypeDiscriminant {
 impl From<&LayerDataType> for LayerDataTypeDiscriminant {
 	fn from(data: &LayerDataType) -> Self {
 		use LayerDataType::*;
+
 		match data {
 			Folder(_) => LayerDataTypeDiscriminant::Folder,
 			Shape(_) => LayerDataTypeDiscriminant::Shape,

editor/src/document/movement_message_handler.rs

@@ -113,8 +113,10 @@ impl MovementMessageHandler {
 impl MessageHandler<MovementMessage, (&Document, &InputPreprocessorMessageHandler)> for MovementMessageHandler {
 	#[remain::check]
 	fn process_action(&mut self, message: MovementMessage, data: (&Document, &InputPreprocessorMessageHandler), responses: &mut VecDeque<Message>) {
-		let (document, ipp) = data;
 		use MovementMessage::*;
+
+		let (document, ipp) = data;
+
 		#[remain::sorted]
 		match message {
 			DecreaseCanvasZoom { center_on_mouse } => {

editor/src/document/overlays_message_handler.rs

@@ -14,8 +14,9 @@ pub struct OverlaysMessageHandler {
 impl MessageHandler<OverlaysMessage, (&mut LayerMetadata, &Document, &InputPreprocessorMessageHandler)> for OverlaysMessageHandler {
 	#[remain::check]
 	fn process_action(&mut self, message: OverlaysMessage, _data: (&mut LayerMetadata, &Document, &InputPreprocessorMessageHandler), responses: &mut VecDeque<Message>) {
-		// let (layer_metadata, document, ipp) = data;
 		use OverlaysMessage::*;
+
+		// let (layer_metadata, document, ipp) = data;
 		#[remain::sorted]
 		match message {
 			ClearAllOverlays => todo!(),

editor/src/document/portfolio_message_handler.rs

@@ -127,6 +127,7 @@ impl MessageHandler<PortfolioMessage, &InputPreprocessorMessageHandler> for Port
 	fn process_action(&mut self, message: PortfolioMessage, ipp: &InputPreprocessorMessageHandler, responses: &mut VecDeque<Message>) {
 		use DocumentMessage::*;
 		use PortfolioMessage::*;
+
 		#[remain::sorted]
 		match message {
 			AutoSaveActiveDocument => responses.push_back(PortfolioMessage::AutoSaveDocument { document_id: self.active_document_id }.into()),

editor/src/document/transform_layer_message_handler.rs

@@ -39,7 +39,7 @@ impl MessageHandler<TransformLayerMessage, (&mut HashMap<Vec<LayerId>, LayerMeta
 		let mut selected = Selected::new(&mut self.original_transforms, &mut self.pivot, layer_metadata, responses, document);
 
 		let mut begin_operation = |operation: TransformOperation, typing: &mut Typing, mouse_position: &mut DVec2, start_mouse: &mut DVec2| {
-			if !(operation == TransformOperation::None) {
+			if operation != TransformOperation::None {
 				selected.revert_operation();
 				typing.clear();
 			} else {

editor/src/global/global_message_handler.rs

@@ -9,6 +9,7 @@ impl MessageHandler<GlobalMessage, ()> for GlobalMessageHandler {
 	#[remain::check]
 	fn process_action(&mut self, message: GlobalMessage, _data: (), _responses: &mut VecDeque<Message>) {
 		use GlobalMessage::*;
+
 		#[remain::sorted]
 		match message {
 			LogDebug => {

editor/src/input/input_mapper.rs

@@ -234,6 +234,7 @@ impl Default for Mapping {
 impl Mapping {
 	pub fn match_message(&self, message: InputMapperMessage, keys: &KeyStates, actions: ActionList) -> Option<Message> {
 		use InputMapperMessage::*;
+
 		let list = match message {
 			KeyDown(key) => &self.key_down[key as usize],
 			KeyUp(key) => &self.key_up[key as usize],

editor/src/viewport_tools/tools/ellipse.rs

@@ -45,6 +45,7 @@ impl<'a> MessageHandler<ToolMessage, ToolActionHandlerData<'a>> for Ellipse {
 
 	fn actions(&self) -> ActionList {
 		use EllipseToolFsmState::*;
+
 		match self.fsm_state {
 			Ready => actions!(EllipseMessageDiscriminant; DragStart),
 			Drawing => actions!(EllipseMessageDiscriminant; DragStop, Abort, Resize),
@@ -81,9 +82,11 @@ impl Fsm for EllipseToolFsmState {
 		input: &InputPreprocessorMessageHandler,
 		responses: &mut VecDeque<Message>,
 	) -> Self {
-		let mut shape_data = &mut data.data;
 		use EllipseMessage::*;
 		use EllipseToolFsmState::*;
+
+		let mut shape_data = &mut data.data;
+
 		if let ToolMessage::Ellipse(event) = event {
 			match (self, event) {
 				(Ready, DragStart) => {

editor/src/viewport_tools/tools/eyedropper.rs

@@ -73,6 +73,7 @@ impl Fsm for EyedropperToolFsmState {
 	) -> Self {
 		use EyedropperMessage::*;
 		use EyedropperToolFsmState::*;
+
 		if let ToolMessage::Eyedropper(event) = event {
 			match (self, event) {
 				(Ready, lmb_or_rmb) if lmb_or_rmb == LeftMouseDown || lmb_or_rmb == RightMouseDown => {

editor/src/viewport_tools/tools/fill.rs

@@ -73,6 +73,7 @@ impl Fsm for FillToolFsmState {
 	) -> Self {
 		use FillMessage::*;
 		use FillToolFsmState::*;
+
 		if let ToolMessage::Fill(event) = event {
 			match (self, event) {
 				(Ready, lmb_or_rmb) if lmb_or_rmb == LeftMouseDown || lmb_or_rmb == RightMouseDown => {

editor/src/viewport_tools/tools/navigate.rs

@@ -44,6 +44,7 @@ impl<'a> MessageHandler<ToolMessage, ToolActionHandlerData<'a>> for Navigate {
 
 	fn actions(&self) -> ActionList {
 		use NavigateToolFsmState::*;
+
 		match self.fsm_state {
 			Ready => actions!(NavigateMessageDiscriminant; TranslateCanvasBegin, RotateCanvasBegin, ZoomCanvasBegin),
 			_ => actions!(NavigateMessageDiscriminant; ClickZoom, MouseMove, TransformCanvasEnd),

editor/src/viewport_tools/tools/path.rs

@@ -52,6 +52,7 @@ impl<'a> MessageHandler<ToolMessage, ToolActionHandlerData<'a>> for Path {
 	// Different actions depending on state may be wanted:
 	fn actions(&self) -> ActionList {
 		use PathToolFsmState::*;
+
 		match self.fsm_state {
 			Ready => actions!(PathMessageDiscriminant; DragStart),
 			Dragging => actions!(PathMessageDiscriminant; DragStop, PointerMove),

editor/src/viewport_tools/tools/pen.rs

@@ -54,6 +54,7 @@ impl<'a> MessageHandler<ToolMessage, ToolActionHandlerData<'a>> for Pen {
 
 	fn actions(&self) -> ActionList {
 		use PenToolFsmState::*;
+
 		match self.fsm_state {
 			Ready => actions!(PenMessageDiscriminant; Undo, DragStart, DragStop, Confirm, Abort),
 			Drawing => actions!(PenMessageDiscriminant; DragStop, PointerMove, Confirm, Abort),
@@ -88,10 +89,11 @@ impl Fsm for PenToolFsmState {
 		input: &InputPreprocessorMessageHandler,
 		responses: &mut VecDeque<Message>,
 	) -> Self {
-		let transform = document.graphene_document.root.transform;
-
 		use PenMessage::*;
 		use PenToolFsmState::*;
+
+		let transform = document.graphene_document.root.transform;
+
 		if let ToolMessage::Pen(event) = event {
 			match (self, event) {
 				(Ready, DragStart) => {

editor/src/viewport_tools/tools/rectangle.rs

@@ -45,6 +45,7 @@ impl<'a> MessageHandler<ToolMessage, ToolActionHandlerData<'a>> for Rectangle {
 
 	fn actions(&self) -> ActionList {
 		use RectangleToolFsmState::*;
+
 		match self.fsm_state {
 			Ready => actions!(RectangleMessageDiscriminant; DragStart),
 			Drawing => actions!(RectangleMessageDiscriminant; DragStop, Abort, Resize),
@@ -80,9 +81,11 @@ impl Fsm for RectangleToolFsmState {
 		input: &InputPreprocessorMessageHandler,
 		responses: &mut VecDeque<Message>,
 	) -> Self {
-		let mut shape_data = &mut data.data;
 		use RectangleMessage::*;
 		use RectangleToolFsmState::*;
+
+		let mut shape_data = &mut data.data;
+
 		if let ToolMessage::Rectangle(event) = event {
 			match (self, event) {
 				(Ready, DragStart) => {

editor/src/viewport_tools/tools/select.rs

@@ -56,6 +56,7 @@ impl<'a> MessageHandler<ToolMessage, ToolActionHandlerData<'a>> for Select {
 
 	fn actions(&self) -> ActionList {
 		use SelectToolFsmState::*;
+
 		match self.fsm_state {
 			Ready => actions!(SelectMessageDiscriminant; DragStart),
 			Dragging => actions!(SelectMessageDiscriminant; DragStop, MouseMove),
@@ -229,7 +230,7 @@ impl Fsm for SelectToolFsmState {
 					data.drag_current = mouse_position + closest_move;
 					Dragging
 				}
-				(DrawingBox, MouseMove { snap_angle: _ }) => {
+				(DrawingBox, MouseMove { .. }) => {
 					data.drag_current = input.mouse.position;
 					let half_pixel_offset = DVec2::splat(0.5);
 					let start = data.drag_start + half_pixel_offset;

editor/src/viewport_tools/tools/shape.rs

@@ -46,6 +46,7 @@ impl<'a> MessageHandler<ToolMessage, ToolActionHandlerData<'a>> for Shape {
 
 	fn actions(&self) -> ActionList {
 		use ShapeToolFsmState::*;
+
 		match self.fsm_state {
 			Ready => actions!(ShapeMessageDiscriminant; DragStart),
 			Drawing => actions!(ShapeMessageDiscriminant; DragStop, Abort, Resize),
@@ -82,9 +83,11 @@ impl Fsm for ShapeToolFsmState {
 		input: &InputPreprocessorMessageHandler,
 		responses: &mut VecDeque<Message>,
 	) -> Self {
-		let mut shape_data = &mut data.data;
 		use ShapeMessage::*;
 		use ShapeToolFsmState::*;
+
+		let mut shape_data = &mut data.data;
+
 		if let ToolMessage::Shape(event) = event {
 			match (self, event) {
 				(Ready, DragStart) => {