<div dir="ltr">In the switch / case handler for eVarSetOperationRemove, there is the following code:<div><br></div><div><div>      if (num_remove_indexes) {</div><div>        // Sort and then erase in reverse so indexes are always valid</div><div>        std::sort(remove_indexes.begin(), remove_indexes.end());</div><div>        for (size_t j = num_remove_indexes - 1; j < num_remove_indexes; ++j) {</div><div>          m_path_mappings.Remove(j, m_notify_changes);</div><div>        }</div><div>      }</div></div><div><br></div><div>Should the line that calls Remove() not be like this:</div><div><br></div><div>          m_path_mappings.Remove(<b>remove_indexes[j]</b>, m_notify_changes);  <br></div><div><br></div><div>?  What effect will this have on LLDB?</div></div>