Fixing todo

Author: WardenGnaw

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);
         }
 

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;
@@ -42,7 +43,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 Dictionary<int, ThreadFrameEnumInfo> m_threadFrameEnumInfos = new Dictionary<int, ThreadFrameEnumInfo>();
@@ -705,7 +708,8 @@ protected override void HandleInitializeRequestAsync(IRequestResponder<Initializ
                 SupportsReadMemoryRequest = true,
                 SupportsGotoTargetsRequest = true,
                 SupportsModulesRequest = true,
-                AdditionalModuleColumns = additionalModuleColumns
+                AdditionalModuleColumns = additionalModuleColumns,
+                SupportsDisassembleRequest = true
             };
 
             responder.SetResponse(initializeResponse);
@@ -1240,15 +1244,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)
             {
@@ -1293,11 +1299,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));
                     }
                 }