Fixed find-all refs not looking at all files/projects. (dotnet#10207)

ncave · Oct 2, 2020 · 0dac9c0 · 0dac9c0
1 parent 8b361cf
commit 0dac9c0
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/vsintegration/src/FSharp.Editor/Navigation/FindUsagesService.fs b/vsintegration/src/FSharp.Editor/Navigation/FindUsagesService.fs
@@ -103,7 +103,8 @@ type internal FSharpFindUsagesService
                                         |> Option.defaultValue externalDefinitionItem
 
                                 let referenceItem = FSharpSourceReferenceItem(definitionItem, FSharpDocumentSpan(doc, textSpan))
-                                do! context.OnReferenceFoundAsync(referenceItem) |> Async.AwaitTask }
+                                // REVIEW: OnReferenceFoundAsync is throwing inside Roslyn, putting a try/with so find-all refs doesn't fail.
+                                try do! context.OnReferenceFoundAsync(referenceItem) |> Async.AwaitTask with | _ -> () }
 
             match symbolUse.GetDeclarationLocation document with
             | Some SymbolDeclarationLocation.CurrentDocument ->