Skip to content

refactor(c++): 🎨 implementation into either source or inline files - #203

Open
robertodr wants to merge 5 commits into
mainfrom
refactor-split-impls
Open

refactor(c++): 🎨 implementation into either source or inline files#203
robertodr wants to merge 5 commits into
mainfrom
refactor-split-impls

style: re-format one file

35427b3
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Aug 11, 2026 in 52s

Annotations

Check warning on line 38 in cpp/monoprop/detail/partition/CpuTopology.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use std::format instead of concatenating pieces manually.

See more on https://sonarcloud.io/project/issues?id=Algorithmiq_monoprop&issues=AZ_Q2tQ5cpe6GFsppuR1&open=AZ_Q2tQ5cpe6GFsppuR1&pullRequest=203

Check warning on line 44 in cpp/monoprop/detail/partition/CpuTopology.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace with the version of "std::ranges::min_element" that takes a range.

See more on https://sonarcloud.io/project/issues?id=Algorithmiq_monoprop&issues=AZ_Q2tQ5cpe6GFsppuR2&open=AZ_Q2tQ5cpe6GFsppuR2&pullRequest=203

Check failure on line 24 in cpp/monoprop/detail/partition/CpuTopology.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 34 to the 25 allowed.

See more on https://sonarcloud.io/project/issues?id=Algorithmiq_monoprop&issues=AZ_Q2tQ5cpe6GFsppuRz&open=AZ_Q2tQ5cpe6GFsppuRz&pullRequest=203

Check warning on line 69 in cpp/monoprop/detail/partition/CpuTopology.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace with the version of "std::ranges::find" that takes a range.

See more on https://sonarcloud.io/project/issues?id=Algorithmiq_monoprop&issues=AZ_Q2tQ5cpe6GFsppuR3&open=AZ_Q2tQ5cpe6GFsppuR3&pullRequest=203

Check failure on line 56 in cpp/monoprop/detail/partition/CpuTopology.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this code to not nest more than 3 if|for|do|while|switch statements.

See more on https://sonarcloud.io/project/issues?id=Algorithmiq_monoprop&issues=AZ_Q2tQ5cpe6GFsppuR0&open=AZ_Q2tQ5cpe6GFsppuR0&pullRequest=203

Check warning on line 179 in cpp/monoprop/detail/graph_encoding/MPGraphEncoding.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

The result of "std::move" should not be passed as a const reference.

See more on https://sonarcloud.io/project/issues?id=Algorithmiq_monoprop&issues=AZ_WcRtePHirgAHj0Jx5&open=AZ_WcRtePHirgAHj0Jx5&pullRequest=203