Bug in CommandObjectThreadUntil

CommandObjectThreadUntil::DoExecute() has the following code:

for (size_t i = 0; i < num_args; i++) {
uint32_t line_number;
line_number = StringConvert::ToUInt32(command.GetArgumentAtIndex(0),
UINT32_MAX);
if (line_number == UINT32_MAX) {
result.AppendErrorWithFormat(“invalid line number: ‘%s’.\n”,
command.GetArgumentAtIndex(0));
result.SetStatus(eReturnStatusFailed);
return false;
} else
line_numbers.push_back(line_number);
}

  1. It looks to me like this is supposed to be GetArgumentAtIndex(i) instead of GetArgumentAtIndex(0). Is this in fact a bug?

  2. The ability for this command to take multiple line numbers is not documented. Should it be?

Obviously a bug. Fixed in r287386.

Jim