Convert some Args index-based iteration to range-style iteration.

This is better for a number of reasons.  Mostly style, but also:

1) Signed-unsigned comparison warnings disappear since there is
   no loop index.
2) Iterating with the range-for style gives you back an entry
   that has more than just a const char*, so it's more efficient
   and more useful.
3) Makes code safter since the type system enforces that it's
   impossible to index out of bounds.

llvm-svn: 283413

1 files changed, 2 insertions, 3 deletions

diff --git a/lldb/source/Commands/CommandObjectCommands.cpp b/lldb/source/Commands/CommandObjectCommands.cpp
index abf2c745490..3df7fb75627 100644
--- a/lldb/source/Commands/CommandObjectCommands.cpp
+++ b/lldb/source/Commands/CommandObjectCommands.cpp
@@ -1096,10 +1096,9 @@ protected:
         result.SetStatus(eReturnStatusSuccessFinishNoResult);
       }
     } else {
-      for (size_t arg_idx = 1; arg_idx < argc; ++arg_idx) {
-        llvm::StringRef arg_strref(command.GetArgumentAtIndex(arg_idx));
+      for (auto &entry : command.entries().drop_front()) {
         bool check_only = false;
-        error = AppendRegexSubstitution(arg_strref, check_only);
+        error = AppendRegexSubstitution(entry.ref, check_only);
         if (error.Fail())
           break;
       }