Improve symbol re-lookup implementation (#107)

* Simplify symbol re-lookup implementation

* Use the address offset and IDiaSession::findLinesByAddr to avoid an
  expensive loop through the entire symbol address range

* Directly invoke ProcessFrameModuleOffset on effective RVA to avoid
  having to again parse the simulated frame, thereby gaining speed and
  streamlining code.

* Fix some corner cases for single-line input detection

* Add specific handling for single-line callstacks in relookup case

* Consolidate `else` case processing

* Restore method signature to avoid breaking change

* Minor refactor

* Add test for relookup of single-line stacks /w frame nums

* Address CodeQL suggestion

Author: arvindsh

Engine/StackDetails.cs

@@ -8,11 +8,13 @@ public class StackDetails {
         private string _annotation, _callStack, _resolvedStack;
         private readonly string _stackKey;
         private readonly bool _framesOnSingleLine;
+        private readonly bool _relookupSource;
 
-        public StackDetails(string callStack, bool framesOnSingleLine, string annotation = null, string stackKey = null) {
+        public StackDetails(string callStack, bool framesOnSingleLine, string annotation = null, string stackKey = null, bool relookupSource = false) {
             this._annotation = annotation;
             this._stackKey = stackKey;
             this._framesOnSingleLine = framesOnSingleLine;
+            this._relookupSource = relookupSource;
             this._callStack = System.Net.WebUtility.HtmlDecode(framesOnSingleLine ? Regex.Replace(callStack, @"\s{2,}", " ") : callStack);
             _stackKey = stackKey;
         }
@@ -27,7 +29,8 @@ public string[] CallstackFrames {
                 // sometimes we see call stacks which are arranged horizontally (this typically is seen when copy-pasting directly
                 // from the SSMS XEvent window (copying the callstack field without opening it in its own viewer)
                 // in that case, space is a valid delimiter, and we need to support that as an option
-                var delims = this._framesOnSingleLine ? new char[3] { ' ', '\t', '\n' } : new char[1] { '\n' };
+                var delims = this._framesOnSingleLine ? new char[] { '\t', '\n' } : new char[] { '\n' };
+                if (!this._relookupSource && this._framesOnSingleLine) delims = delims.Append(' ').ToArray();
                 return this._callStack.Replace("\r", string.Empty).Split(delims);
             }
         }