Introduce unified stoppoint indexing

LLDB uses separate indexing for breakpoints and watchpoints unlike GDB
(and thus MI protocol too) which uses the same space of indices for
both. So, it is necessary to provide some mapping that maps possibly
overlapping breakpoint and watchpoint LLDB IDs to non-overlapping GDB
IDs.

This commit adds such mechanism to CMICmnLLDBDebugSessionInfo class.
Existing methods and commands are modified to support new indexing and
watchpoints. Also, there introduced some new methods that will be used
by further commits.

Author: ilya-nozhkin

src/MICmdCmdBreak.cpp

@@ -32,6 +32,7 @@
 
 // In-house headers:
 #include "MICmdBase.h"
+#include "MICmnLLDBDebugSessionInfo.h"
 
 //++
 //============================================================================
@@ -155,7 +156,7 @@ class CMICmdCmdBreakDisable : public CMICmdBase {
 private:
   const CMIUtilString m_constStrArgNamedBrkPt;
   bool m_bBrkPtDisabledOk;
-  MIuint m_nBrkPtId;
+  MIuint m_nMiStopPtId;
 };
 
 //++
@@ -186,7 +187,7 @@ class CMICmdCmdBreakEnable : public CMICmdBase {
 private:
   const CMIUtilString m_constStrArgNamedBrkPt;
   bool m_bBrkPtEnabledOk;
-  MIuint m_nBrkPtId;
+  MIuint m_nMiStopPtId;
 };
 
 //++
@@ -213,11 +214,30 @@ class CMICmdCmdBreakAfter : public CMICmdBase {
   // From CMICmnBase
   /* dtor */ ~CMICmdCmdBreakAfter() override;
 
+  // Methods:
+private:
+  bool UpdateStopPtInfo(CMICmnLLDBDebugSessionInfo &rSessionInfo);
+  template <class T>
+  bool SetIgnoreCount(CMICmnLLDBDebugSessionInfo &rSessionInfo, T &vrStopPt) {
+    if (!vrStopPt.IsValid()) {
+      const CMIUtilString strBrkPtId(CMIUtilString::Format(
+          "%" PRIu64, static_cast<uint64_t>(m_nMiStopPtId)));
+      SetError(CMIUtilString::Format(MIRSRC(IDS_CMD_ERR_STOPPT_INVALID),
+                                     m_cmdData.strMiCmd.c_str(),
+                                     strBrkPtId.c_str()));
+      return MIstatus::failure;
+    }
+
+    vrStopPt.SetIgnoreCount(m_nBrkPtCount);
+
+    return UpdateStopPtInfo(rSessionInfo);
+  }
+
   // Attributes:
 private:
   const CMIUtilString m_constStrArgNamedNumber;
   const CMIUtilString m_constStrArgNamedCount;
-  MIuint m_nBrkPtId;
+  MIuint m_nMiStopPtId;
   MIuint m_nBrkPtCount;
 };
 
@@ -248,6 +268,22 @@ class CMICmdCmdBreakCondition : public CMICmdBase {
   // Methods:
 private:
   CMIUtilString GetRestOfExpressionNotSurroundedInQuotes();
+  bool UpdateStopPtInfo(CMICmnLLDBDebugSessionInfo &rSessionInfo);
+  template <class T>
+  bool SetCondition(CMICmnLLDBDebugSessionInfo &rSessionInfo, T &vrStopPt) {
+    if (!vrStopPt.IsValid()) {
+      const CMIUtilString strBrkPtId(CMIUtilString::Format(
+          "%" PRIu64, static_cast<uint64_t>(m_nMiStopPtId)));
+      SetError(CMIUtilString::Format(MIRSRC(IDS_CMD_ERR_STOPPT_INVALID),
+                                     m_cmdData.strMiCmd.c_str(),
+                                     strBrkPtId.c_str()));
+      return MIstatus::failure;
+    }
+
+    vrStopPt.SetCondition(m_strBrkPtExpr.c_str());
+
+    return UpdateStopPtInfo(rSessionInfo);
+  }
 
   // Attributes:
 private:
@@ -257,6 +293,6 @@ class CMICmdCmdBreakCondition : public CMICmdBase {
                                                       // spec, we need to handle
                                                       // expressions not
                                                       // surrounded by quotes
-  MIuint m_nBrkPtId;
+  MIuint m_nMiStopPtId;
   CMIUtilString m_strBrkPtExpr;
 };