Skip to content

Commit 838d5f9

Browse files
committed
[lib][rr_graph] fix iterator idx
1 parent 6dfc2c3 commit 838d5f9

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

libs/librrgraph/src/base/rr_graph_storage.cpp

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -706,13 +706,13 @@ void t_rr_graph_storage::remove_nodes(std::vector<RRNodeId> nodes_to_remove) {
706706

707707
// Iterate over the nodes to be removed and adjust the IDs of nodes
708708
// that fall between them.
709-
for (size_t i = 0; i < nodes_to_remove.size(); ++i) {
710-
size_t start_rr_node_index = size_t(nodes_to_remove[i]) + 1;
711-
size_t end_rr_node_index = (i == nodes_to_remove.size() - 1) ? node_storage_.size() : size_t(nodes_to_remove[i + 1]);
712-
for (size_t j = start_rr_node_index; j < end_rr_node_index; ++j) {
713-
RRNodeId old_node = RRNodeId(j);
709+
for (size_t removal_idx = 0; removal_idx < nodes_to_remove.size(); ++removal_idx) {
710+
size_t start_rr_node_index = size_t(nodes_to_remove[removal_idx]) + 1;
711+
size_t end_rr_node_index = (removal_idx == nodes_to_remove.size() - 1) ? node_storage_.size() : size_t(nodes_to_remove[removal_idx + 1]);
712+
for (size_t node_idx = start_rr_node_index; node_idx < end_rr_node_index; ++node_idx) {
713+
RRNodeId old_node = RRNodeId(node_idx);
714714
// New node index is equal to the old nodex index minus the number of nodes being removed before it.
715-
RRNodeId new_node = RRNodeId(j-(i+1));
715+
RRNodeId new_node = RRNodeId(node_idx-(removal_idx+1));
716716
node_storage_[new_node] = node_storage_[old_node];
717717
node_ptc_[new_node] = node_ptc_[old_node];
718718
node_layer_[new_node] = node_layer_[old_node];

0 commit comments

Comments
 (0)