clippy: Fix semicolon_if_nothing_returned lints. (#184)

crates/xilem_web/src/app.rs

@@ -51,7 +51,7 @@ impl<T: 'static, V: View<T> + 'static, F: FnMut(&mut T) -> V + 'static> App<T, V
     pub fn run(self, root: &web_sys::HtmlElement) {
         self.0.borrow_mut().ensure_app(root);
         // Latter may not be necessary, we have an rc loop.
-        std::mem::forget(self)
+        std::mem::forget(self);
     }
 }
 

crates/xilem_web/src/context.rs

@@ -21,10 +21,10 @@ fn set_attribute(element: &web_sys::Element, name: &str, value: &str) {
     // doesn't work after the value has been changed.
     if name == "value" {
         let element: &web_sys::HtmlInputElement = element.dyn_ref().unwrap_throw();
-        element.set_value(value)
+        element.set_value(value);
     } else if name == "checked" {
         let element: &web_sys::HtmlInputElement = element.dyn_ref().unwrap_throw();
-        element.set_checked(true)
+        element.set_checked(true);
     } else {
         element.set_attribute(name, value).unwrap_throw();
     }
@@ -35,7 +35,7 @@ fn remove_attribute(element: &web_sys::Element, name: &str) {
     // doesn't work after the value has been changed.
     if name == "checked" {
         let element: &web_sys::HtmlInputElement = element.dyn_ref().unwrap_throw();
-        element.set_checked(false)
+        element.set_checked(false);
     } else {
         element.remove_attribute(name).unwrap_throw();
     }

crates/xilem_web/src/vecmap.rs

@@ -204,7 +204,7 @@ impl<K, V> VecMap<K, V> {
     }
 
     pub fn clear(&mut self) {
-        self.0.clear()
+        self.0.clear();
     }
 
     /// Returns `true` if the map contains no elements.

crates/xilem_web/web_examples/todomvc/src/state.rs

@@ -75,7 +75,7 @@ impl AppState {
         if let Some(ref mut todo) = self.todos.iter_mut().find(|todo| todo.id == id) {
             todo.title_editing.clear();
             todo.title_editing.push_str(&todo.title);
-            self.editing_id = Some(id)
+            self.editing_id = Some(id);
         }
     }
 
@@ -96,7 +96,7 @@ impl AppState {
     /// Save the current state to local_storage
     pub fn save(&self) {
         let raw = serde_json::to_string(self).unwrap_throw();
-        storage().set_item(KEY, &raw).unwrap_throw()
+        storage().set_item(KEY, &raw).unwrap_throw();
     }
 }
 

examples/hello.rs

@@ -26,7 +26,7 @@ fn app_logic(data: &mut AppData) -> impl View<AppData> {
                 data.count = 0;
             }),
             switch(data.is_on, |data: &mut AppData, value: bool| {
-                data.is_on = value
+                data.is_on = value;
             }),
         )),
     ))
@@ -55,5 +55,5 @@ fn main() {
     };
 
     let app = App::new(data, app_logic);
-    AppLauncher::new(app).run()
+    AppLauncher::new(app).run();
 }

examples/taffy.rs

@@ -126,5 +126,5 @@ fn app_logic(state: &mut AppState) -> impl View<AppState> {
 
 fn main() {
     let app = App::new(AppState::new(), app_logic);
-    AppLauncher::new(app).run()
+    AppLauncher::new(app).run();
 }

src/geometry.rs

@@ -62,7 +62,7 @@ impl Axis {
 
     /// Updates the scalar of value on the orthogonal axis.
     pub fn set_minor<T: Dim2>(self, value: &mut T, major: T::Scalar) {
-        self.cross().set_major(value, major)
+        self.cross().set_major(value, major);
     }
 
     /// Maps the scalar of the value on this axis.

src/view/taffy_layout.rs

@@ -127,7 +127,7 @@ impl<T, A, VT: ViewSequence<T, A>> View<T, A> for TaffyLayout<T, A, VT> {
 
         if self.background_color != prev.background_color {
             element.background_color = self.background_color;
-            flags |= ChangeFlags::PAINT
+            flags |= ChangeFlags::PAINT;
         }
 
         if self.style != prev.style {
@@ -137,7 +137,7 @@ impl<T, A, VT: ViewSequence<T, A>> View<T, A> for TaffyLayout<T, A, VT> {
 
         // Clear layout cache if the layout ChangeFlag is set
         if flags.contains(ChangeFlags::LAYOUT) || flags.contains(ChangeFlags::TREE) {
-            element.cache.clear()
+            element.cache.clear();
         }
 
         flags

src/widget/button.rs

@@ -72,7 +72,7 @@ impl Widget for Button {
 
     fn lifecycle(&mut self, cx: &mut LifeCycleCx, event: &LifeCycle) {
         if let LifeCycle::HotChanged(_) = event {
-            cx.request_paint()
+            cx.request_paint();
         }
     }
 

src/widget/core.rs

@@ -149,7 +149,7 @@ impl WidgetState {
     }
 
     fn request(&mut self, flags: PodFlags) {
-        self.flags |= flags
+        self.flags |= flags;
     }
 }
 

src/widget/piet_scene_helpers.rs

@@ -22,7 +22,7 @@ pub fn stroke<'b>(
         brush,
         None,
         path,
-    )
+    );
 }
 
 // Note: copied from piet
@@ -77,5 +77,5 @@ pub fn fill_lin_gradient(
 }
 
 pub fn fill_color(scene: &mut Scene, path: &impl Shape, color: Color) {
-    scene.fill(Fill::NonZero, Affine::IDENTITY, color, None, path)
+    scene.fill(Fill::NonZero, Affine::IDENTITY, color, None, path);
 }

src/widget/switch.rs

@@ -74,7 +74,7 @@ impl Widget for Switch {
             Event::MouseUp(_) => {
                 if self.is_dragging {
                     if self.is_on != (self.knob_position.x > SWITCH_WIDTH / 2.0) {
-                        cx.add_message(Message::new(self.id_path.clone(), ()))
+                        cx.add_message(Message::new(self.id_path.clone(), ()));
                     }
                 } else if cx.is_active() {
                     cx.add_message(Message::new(self.id_path.clone(), ()));