From 79474288d05a2cde17030c6632ef2d045bcc2e76 Mon Sep 17 00:00:00 2001 From: Paul Masurel Date: Mon, 9 Dec 2019 13:40:04 +0900 Subject: [PATCH] Some clippy minor fixes (#722) --- src/collector/facet_collector.rs | 8 ++++---- src/indexer/delete_queue.rs | 4 ++-- src/macros.rs | 6 +++--- 3 files changed, 9 insertions(+), 9 deletions(-) diff --git a/src/collector/facet_collector.rs b/src/collector/facet_collector.rs index bb2d09028..132f35e6c 100644 --- a/src/collector/facet_collector.rs +++ b/src/collector/facet_collector.rs @@ -126,7 +126,7 @@ fn facet_depth(facet_bytes: &[u8]) -> usize { /// let searcher = reader.searcher(); /// /// { -/// let mut facet_collector = FacetCollector::for_field(facet); +/// let mut facet_collector = FacetCollector::for_field(facet); /// facet_collector.add_facet("/lang"); /// facet_collector.add_facet("/category"); /// let facet_counts = searcher.search(&AllQuery, &facet_collector)?; @@ -142,7 +142,7 @@ fn facet_depth(facet_bytes: &[u8]) -> usize { /// } /// /// { -/// let mut facet_collector = FacetCollector::for_field(facet); +/// let mut facet_collector = FacetCollector::for_field(facet); /// facet_collector.add_facet("/category/fiction"); /// let facet_counts = searcher.search(&AllQuery, &facet_collector)?; /// @@ -157,8 +157,8 @@ fn facet_depth(facet_bytes: &[u8]) -> usize { /// ]); /// } /// -/// { -/// let mut facet_collector = FacetCollector::for_field(facet); +/// { +/// let mut facet_collector = FacetCollector::for_field(facet); /// facet_collector.add_facet("/category/fiction"); /// let facet_counts = searcher.search(&AllQuery, &facet_collector)?; /// diff --git a/src/indexer/delete_queue.rs b/src/indexer/delete_queue.rs index 00203aaef..c2b3e21c9 100644 --- a/src/indexer/delete_queue.rs +++ b/src/indexer/delete_queue.rs @@ -43,14 +43,14 @@ impl DeleteQueue { // try get the last block with simply acquiring the read lock. let rlock = self.inner.read().unwrap(); if let Some(block) = rlock.last_block.upgrade() { - return block.clone(); + return block; } } // It failed. Let's double check after acquiring the write, as someone could have called // `get_last_block` right after we released the rlock. let mut wlock = self.inner.write().unwrap(); if let Some(block) = wlock.last_block.upgrade() { - return block.clone(); + return block; } let block = Arc::new(Block { operations: Arc::default(), diff --git a/src/macros.rs b/src/macros.rs index 973111b37..303e79a3e 100644 --- a/src/macros.rs +++ b/src/macros.rs @@ -35,9 +35,9 @@ /// let likes = schema_builder.add_u64_field("num_u64", FAST); /// let schema = schema_builder.build(); /// let doc = doc!( -/// title => "Life Aquatic", -/// author => "Wes Anderson", -/// likes => 4u64 +/// title => "Life Aquatic", +/// author => "Wes Anderson", +/// likes => 4u64 /// ); /// # } /// ```