Merge pull request #1179 from github/fixes/1124-nre-inlinecommenttagger

jcansdale · web-flow · commit 52152f618aa7 · 2017-08-22T11:00:30.000+01:00
Add additional null check in InlineCommentTagger.GetTags.
diff --git a/src/GitHub.InlineReviews/Tags/InlineCommentTagger.cs b/src/GitHub.InlineReviews/Tags/InlineCommentTagger.cs
@@ -100,7 +100,7 @@ public IEnumerable<ITagSpan<InlineCommentTag>> GetTags(NormalizedSnapshotSpanCol
                 // Sucessful initialization will call NotifyTagsChanged, causing this method to be re-called.
                 Initialize();
             }
-            else if (file != null)
+            else if (file != null && session != null)
             {
                 foreach (var span in spans)
                 {

Original file line number	Diff line number	Diff line change
`@@ -100,7 +100,7 @@ public IEnumerable<ITagSpan<InlineCommentTag>> GetTags(NormalizedSnapshotSpanCol`
`100`	`100`	`// Sucessful initialization will call NotifyTagsChanged, causing this method to be re-called.`
`101`	`101`	`Initialize();`
`102`	`102`	`}`
`103`		`- else if (file != null)`
	`103`	`+ else if (file != null && session != null)`
`104`	`104`	`{`
`105`	`105`	`foreach (var span in spans)`
`106`	`106`	`{`