Merge #802

802: fix obsolete comment r=matklad a=matklad



Co-authored-by: Aleksey Kladov <aleksey.kladov@gmail.com>

Author: bors[bot]

crates/ra_hir/src/ty/method_resolution.rs

@@ -136,8 +136,8 @@ impl Ty {
         })
     }
 
-    // This would be nicer if it just returned an iterator, but that's really
-    // complicated with all the cancelable operations
+    // This would be nicer if it just returned an iterator, but that runs into
+    // lifetime problems, because we need to borrow temp `CrateImplBlocks`.
     pub fn iterate_methods<T>(
         self,
         db: &impl HirDatabase,
@@ -163,7 +163,7 @@ impl Ty {
                 for item in impl_block.items() {
                     match item {
                         ImplItem::Method(f) => {
-                            if let Some(result) = callback(f.clone()) {
+                            if let Some(result) = callback(*f) {
                                 return Some(result);
                             }
                         }