Fixing todo

microsoft · Dec 22, 2020 · 6f958d5 · 6f958d5
1 parent 4847179
commit 6f958d5
Show file tree

Hide file tree

Showing 2 changed files with 23 additions and 13 deletions.
diff --git a/src/MICore/CommandFactories/lldb.cs b/src/MICore/CommandFactories/lldb.cs
@@ -186,7 +186,7 @@ public override async Task ExecJump(string filename, int line)
 
         public override async Task ExecJump(ulong address)
         {
-            string command = "jump *" + string.Format("0x{0:X}", address);
+            string command = "jump *" + string.Format(CultureInfo.InvariantCulture, "0x{0:X}", address);
             await _debugger.CmdAsync(command, ResultClass.running);
         }
 

diff --git a/src/OpenDebugAD7/AD7DebugSession.cs b/src/OpenDebugAD7/AD7DebugSession.cs
@@ -2,6 +2,7 @@
 // Licensed under the MIT license. See LICENSE file in the project root for full license information.
 
 using System;
+using System.Collections.Concurrent;
 using System.Collections.Generic;
 using System.Diagnostics;
 using System.Globalization;
@@ -44,7 +45,9 @@ internal sealed class AD7DebugSession : DebugAdapterBase, IDebugPortNotify2, IDe
 
         private readonly DebugEventLogger m_logger;
         private readonly Dictionary<string, Dictionary<int, IDebugPendingBreakpoint2>> m_breakpoints;
-        private readonly List<IDebugCodeContext2> m_gotoCodeContexts = new List<IDebugCodeContext2>();
+
+        private readonly ConcurrentDictionary<int, IDebugCodeContext2> m_gotoCodeContexts = new ConcurrentDictionary<int, IDebugCodeContext2>();
+        private int m_nextContextId = 1;
 
         private Dictionary<string, IDebugPendingBreakpoint2> m_functionBreakpoints;
         private readonly HandleCollection<IDebugStackFrame2> m_frameHandles;
@@ -1321,15 +1324,17 @@ protected override void HandleGotoRequestAsync(IRequestResponder<GotoArguments>
             var builder = new ErrorBuilder(() => AD7Resources.Error_UnableToSetNextStatement);
             try
             {
-                var gotoTarget = m_gotoCodeContexts[responder.Arguments.TargetId];
-                IDebugThread2 thread = null;
-                lock (m_threads)
+                if (m_gotoCodeContexts.TryGetValue(responder.Arguments.TargetId, out IDebugCodeContext2 gotoTarget))
                 {
-                    if (!m_threads.TryGetValue(responder.Arguments.ThreadId, out thread))
-                        throw new AD7Exception("Unknown thread id: " + responder.Arguments.ThreadId.ToString(CultureInfo.InvariantCulture));
+                    IDebugThread2 thread = null;
+                    lock (m_threads)
+                    {
+                        if (!m_threads.TryGetValue(responder.Arguments.ThreadId, out thread))
+                            throw new AD7Exception("Unknown thread id: " + responder.Arguments.ThreadId.ToString(CultureInfo.InvariantCulture));
+                    }
+                    BeforeContinue();
+                    builder.CheckHR(thread.SetNextStatement(null, gotoTarget));
                 }
-                BeforeContinue();
-                builder.CheckHR(thread.SetNextStatement(null, gotoTarget));
             }
             catch (AD7Exception e)
             {
@@ -1374,11 +1379,16 @@ protected override void HandleGotoTargetsRequestAsync(IRequestResponder<GotoTarg
                         IDebugDocumentContext2 documentContext;
                         if (codeContext.GetDocumentContext(out documentContext) == HRConstants.S_OK)
                         {
-                            var pos = new TEXT_POSITION[1];
-                            if (documentContext.GetStatementRange(pos, null) == HRConstants.S_OK)
-                                line = m_pathConverter.ConvertDebuggerLineToClient((int)pos[0].dwLine);
+                            var startPos = new TEXT_POSITION[1];
+                            var endPos = new TEXT_POSITION[1];
+                            if (documentContext.GetStatementRange(startPos, endPos) == HRConstants.S_OK)
+                                line = m_pathConverter.ConvertDebuggerLineToClient((int)startPos[0].dwLine);
                         }
-                        targets.Add(new GotoTarget(m_gotoCodeContexts.Create(codeContext), contextName, line));
+
+                        int codeContextId = m_nextContextId++;
+                        m_gotoCodeContexts.TryAdd(codeContextId, codeContext);
+
+                        targets.Add(new GotoTarget(codeContextId, contextName, line));
                     }
                 }