From 376de68f207519786e9a455e9d137eb63fea9c15 Mon Sep 17 00:00:00 2001 From: Kevin Halverson Date: Fri, 25 Sep 2015 12:14:03 -0700 Subject: [PATCH] Add null check in SemanticModelService for #load'ed trees... --- .../SemanticModelWorkspaceServiceFactory.cs | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/src/Workspaces/Core/Portable/SemanticModelWorkspaceService/SemanticModelWorkspaceServiceFactory.cs b/src/Workspaces/Core/Portable/SemanticModelWorkspaceService/SemanticModelWorkspaceServiceFactory.cs index a914181549d..74576bc646b 100644 --- a/src/Workspaces/Core/Portable/SemanticModelWorkspaceService/SemanticModelWorkspaceServiceFactory.cs +++ b/src/Workspaces/Core/Portable/SemanticModelWorkspaceService/SemanticModelWorkspaceServiceFactory.cs @@ -485,7 +485,15 @@ private CompilationSet(VersionStamp version, ValueSource compilatio } var documentId = newProject.GetDocumentId(newTree); - Contract.Requires(documentId != null); + + // GetDocumentId will return null for #load'ed trees. + // TODO: Remove this check and add logic to fetch the #load'ed tree's + // Document once https://github.com/dotnet/roslyn/issues/5260 is fixed. + if (documentId == null) + { + Debug.Assert(newProject.Solution.Workspace.Kind == "Interactive"); + continue; + } map = map.SetItem(documentId, newTree); } -- GitLab