[Lldb-commits] [PATCH] D11049: [lldb-mi] size_t rather than MIuint for arg counts.

[Ilia K]

ki.stfu requested changes to this revision.
This revision now requires changes to proceed.

================
Comment at: tools/lldb-mi/MICmdArgContext.cpp:152
@@ -151,3 +151,3 @@
 bool
-CMICmdArgContext::RemoveArgAtPos(const CMIUtilString &vArg, const MIuint nArgIndex)
+CMICmdArgContext::RemoveArgAtPos(const CMIUtilString &vArg, size_t nArgIndex)
 {
----------------
but you forgot this fact here.

================
Comment at: tools/lldb-mi/MICmdArgContext.h:35
@@ -34,3 +34,3 @@
     bool RemoveArg(const CMIUtilString &vArg);
-    bool RemoveArgAtPos(const CMIUtilString &vArg, const MIuint nArgIndex);
+    bool RemoveArgAtPos(const CMIUtilString &vArg, const size_t nArgIndex);
     //
----------------
nArgIndex is _const_ size_t


http://reviews.llvm.org/D11049