diff --git a/src/indexer/merger.rs b/src/indexer/merger.rs index 1dda7d580..248a59baa 100644 --- a/src/indexer/merger.rs +++ b/src/indexer/merger.rs @@ -236,7 +236,7 @@ impl IndexMerger { // We can remove the term if all documents which // contained it have been deleted. - if segment_postings.len() > 0 { + if !segment_postings.is_empty() { // We can now serialize this postings, by pushing each document to the // postings serializer. diff --git a/src/query/query_parser/query_parser.rs b/src/query/query_parser/query_parser.rs index 5bb25ffcb..0b6b43efe 100644 --- a/src/query/query_parser/query_parser.rs +++ b/src/query/query_parser/query_parser.rs @@ -222,7 +222,7 @@ impl QueryParser { vec![(field, literal.phrase.clone())] } None => { - if self.default_fields.len() == 0 { + if self.default_fields.is_empty() { return Err(QueryParserError::NoDefaultFieldDeclared); } else { self.default_fields @@ -238,7 +238,7 @@ impl QueryParser { asts.push(LogicalAST::Leaf(box ast)); } } - let result_ast = if asts.len() == 0 { + let result_ast = if asts.is_empty() { // this should never happen return Err(QueryParserError::SyntaxError); } else if asts.len() == 1 {