Skip to content

Commit 22429e0

Browse files
authored
Merge pull request #1257 from PowerGridModel/feature/sonar-reliability
Sonar issues: fix reliability issues
2 parents c6c9221 + f8cc1f4 commit 22429e0

1 file changed

Lines changed: 7 additions & 5 deletions

File tree

  • power_grid_model_c/power_grid_model/include/power_grid_model

power_grid_model_c/power_grid_model/include/power_grid_model/topology.hpp

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -353,8 +353,9 @@ class Topology {
353353
return math_idx.pos;
354354
};
355355
// k as branch number for 2-way branch
356-
for (auto const& [idx, branch_node_idx, branch_connected] :
357-
std::views::zip(std::views::iota(0), comp_topo_.branch_node_idx, comp_conn_.branch_connected)) {
356+
for (auto&& [idx, branch_node_idx, branch_connected] :
357+
std::views::zip(std::views::iota(0), std::as_const(comp_topo_.branch_node_idx),
358+
std::as_const(comp_conn_.branch_connected))) {
358359
assert(std::ssize(branch_connected) == 2); // NOSONAR(R354)
359360

360361
auto const [i, j] = branch_node_idx;
@@ -387,9 +388,10 @@ class Topology {
387388
comp_coup_.branch[idx] = Idx2D{.group = math_group, .pos = branch_pos};
388389
}
389390
// k as branch number for 3-way branch
390-
for (auto const& [idx, i, i_status, j_math] :
391-
std::views::zip(std::views::iota(0), comp_topo_.branch3_node_idx, comp_conn_.branch3_connected,
392-
std::views::drop(comp_coup_.node, comp_topo_.n_node))) {
391+
for (auto&& [idx, i, i_status, j_math] :
392+
std::views::zip(std::views::iota(0), std::as_const(comp_topo_.branch3_node_idx),
393+
std::as_const(comp_conn_.branch3_connected),
394+
std::views::drop(std::as_const(comp_coup_.node), std::as_const(comp_topo_.n_node)))) {
393395
std::array<Idx2D, 3> const i_math{
394396
comp_coup_.node[i[0]],
395397
comp_coup_.node[i[1]],

0 commit comments

Comments
 (0)