Browse Source

fix clippy

Evan Almloff 1 year ago
parent
commit
0ab7a02777
3 changed files with 4 additions and 7 deletions
  1. 1 2
      packages/core/src/diff.rs
  2. 1 3
      packages/core/tests/lifecycle.rs
  3. 2 2
      packages/html/src/transit.rs

+ 1 - 2
packages/core/src/diff.rs

@@ -577,8 +577,7 @@ impl<'b> VirtualDom {
         }
 
         // 4. Compute the LIS of this list
-        let mut lis_sequence = Vec::default();
-        lis_sequence.reserve(new_index_to_old_index.len());
+        let mut lis_sequence = Vec::with_capacity(new_index_to_old_index.len());
 
         let mut predecessors = vec![0; new_index_to_old_index.len()];
         let mut starts = vec![0; new_index_to_old_index.len()];

+ 1 - 3
packages/core/tests/lifecycle.rs

@@ -60,9 +60,7 @@ fn events_generate() {
 
     dom.handle_event(
         "click",
-        Rc::new(PlatformEventData::new(Box::new(
-            SerializedMouseData::default(),
-        ))),
+        Rc::new(PlatformEventData::new(Box::<SerializedMouseData>::default())),
         ElementId(1),
         true,
     );

+ 2 - 2
packages/html/src/transit.rs

@@ -337,8 +337,8 @@ impl HtmlEventConverter for SerializedHtmlEventConverter {
             .into()
     }
 
-    fn convert_mounted_data(&self, event: &PlatformEventData) -> MountedData {
-        event.downcast::<()>().cloned().unwrap().into()
+    fn convert_mounted_data(&self, _: &PlatformEventData) -> MountedData {
+        MountedData::from(())
     }
 
     fn convert_mouse_data(&self, event: &PlatformEventData) -> MouseData {