Don't add a byte order mark to editor contents.

This was causing #1137; it was obviously added for a reason in 6b81f3f but I can no longer reproduce VS erroneously signalling a file as changed if we don't add the BOM. Maybe we handle this elsewhere now.

Fixes #1137.

Author: grokys

src/GitHub.InlineReviews/Tags/InlineCommentTagger.cs

@@ -157,7 +157,9 @@ public IEnumerable<ITagSpan<InlineCommentTag>> GetTags(NormalizedSnapshotSpanCol
 
         Task<byte[]> IEditorContentSource.GetContent()
         {
-            return Task.FromResult(GetContents(buffer.CurrentSnapshot));
+            var text = buffer.CurrentSnapshot.GetText();
+            var bytes = document.Encoding.GetBytes(text);
+            return Task.FromResult(bytes);
         }
 
         void Initialize()
@@ -279,22 +281,6 @@ void Buffer_Changed(object sender, TextContentChangedEventArgs e)
             }
         }
 
-        byte[] GetContents(ITextSnapshot snapshot)
-        {
-            var currentText = snapshot.GetText();
-
-            var content = document.Encoding.GetBytes(currentText);
-
-            var preamble = document.Encoding.GetPreamble();
-            if (preamble.Length == 0) return content;
-
-            var completeContent = new byte[preamble.Length + content.Length];
-            Buffer.BlockCopy(preamble, 0, completeContent, 0, preamble.Length);
-            Buffer.BlockCopy(content, 0, completeContent, preamble.Length, content.Length);
-
-            return completeContent;
-        }
-
         async Task Rebuild(ITextSnapshot snapshot)
         {
             if (buffer.CurrentSnapshot == snapshot && session != null)