rustdoc: Avoid using `Iterator::count()` where possible

`count()` iterates over the whole collection. Using `len()` instead, or
`.next().is_none()` when comparing to zero, should be faster.

diff --git a/src/librustdoc/html/markdown.rs b/src/librustdoc/html/markdown.rs
index 47772651bf9b9e92defd25a9ea1d0b8e32504228..d811c85ea58d1df2ac9ba658241d0431d1f9c3b5 100644 (file)
--- a/src/librustdoc/html/markdown.rs
+++ b/src/librustdoc/html/markdown.rs
@@ -656,7 +656,7 @@ fn new(iter: I) -> Self {
     }
 
     fn get_entry(&mut self, key: &str) -> &mut (Vec<Event<'a>>, u16) {
-        let new_id = self.footnotes.keys().count() + 1;
+        let new_id = self.footnotes.len() + 1;
         let key = key.to_owned();
         self.footnotes.entry(key).or_insert((Vec::new(), new_id as u16))
     }

diff --git a/src/librustdoc/passes/unindent_comments.rs b/src/librustdoc/passes/unindent_comments.rs
index da2eda73641228e737f6f70cab177bbc48511940..f42e7a97eb785f0eb44a6b030c359097da2ea312 100644 (file)
--- a/src/librustdoc/passes/unindent_comments.rs
+++ b/src/librustdoc/passes/unindent_comments.rs
@@ -87,7 +87,7 @@ fn unindent_fragments(docs: &mut Vec<DocFragment>) {
     };
 
     for fragment in docs {
-        if fragment.doc.as_str().lines().count() == 0 {
+        if fragment.doc.as_str().lines().next().is_none() {
             continue;
         }