Cargo clippy

topiary-tree-sitter-facade/src/node.rs

@@ -266,9 +266,7 @@ mod native {
 
     impl<'a> PartialOrd for Node<'a> {
         fn partial_cmp(&self, that: &Node<'a>) -> Option<std::cmp::Ordering> {
-            let this = self.id();
-            let that = that.id();
-            this.partial_cmp(&that)
+            Some(self.cmp(that))
         }
     }
 

topiary-tree-sitter-facade/src/query.rs

@@ -51,7 +51,7 @@ mod native {
         }
 
         #[inline]
-        pub fn disable_pattern(&mut self, index: usize) -> () {
+        pub fn disable_pattern(&mut self, index: usize) {
             self.inner.disable_pattern(index)
         }
 

topiary-web-tree-sitter-sys/src/lib.rs

@@ -1,5 +1,3 @@
-#![allow(clippy::unused_unit)]
-
 // FIXME: Double check we construct object properties in order.
 
 use core::cell::RefCell;
@@ -77,7 +75,7 @@ impl TreeSitter {
 
         let scope = &web_sys::window().unwrap_throw();
 
-        let tree_sitter = Reflect::get(&scope, &"TreeSitter".into()).and_then(|property| {
+        let tree_sitter = Reflect::get(scope, &"TreeSitter".into()).and_then(|property| {
             if property.is_undefined() {
                 let message = "window.TreeSitter is not defined; load the tree-sitter javascript module first";
                 Err(JsError::new(message).into())
@@ -374,9 +372,7 @@ impl PartialEq for Point {
 
 impl PartialOrd for Point {
     fn partial_cmp(&self, that: &Point) -> Option<std::cmp::Ordering> {
-        let this = self.spread();
-        let that = that.spread();
-        this.partial_cmp(&that)
+        Some(self.cmp(that))
     }
 }
 
@@ -652,9 +648,7 @@ impl PartialEq<Range> for Range {
 
 impl PartialOrd<Range> for Range {
     fn partial_cmp(&self, that: &Self) -> Option<std::cmp::Ordering> {
-        let this = self.spread();
-        let that = that.spread();
-        this.partial_cmp(&that)
+        Some(self.cmp(that))
     }
 }
 

topiary/Cargo.toml

@@ -35,6 +35,7 @@ env_logger = { workspace = true }
 test-log = { workspace = true }
 tokio-test = { workspace = true }
 tree-sitter-json.workspace = true
+tree-sitter-ocaml.workspace = true
 
 [[bench]]
 name = "benchmark"

topiary/src/atom_collection.rs

@@ -630,7 +630,7 @@ impl AtomCollection {
             {
                 opened_scopes
                     .entry(scope_id)
-                    .or_insert_with(Vec::new)
+                    .or_default()
                     .push((*line_start, Vec::new()));
             } else if let Atom::ScopeEnd(ScopeInformation {
                 line_number: line_end,