eic / EICrecon

EIC Reconstruction - JANA based
https://eic.github.io/EICrecon
GNU Lesser General Public License v3.0
6 stars 27 forks source link

ImagingTopoCluster: fix Init printouts by adding missing break statements #1533

Closed sebouh137 closed 1 month ago

sebouh137 commented 1 month ago

Fixed an issue where "break" is not included in a case/break statement.

Briefly, what does this PR introduce?

When initializing the ImagingTopoCluster algorithm, there is a switch statement wherein different things are printed to inform the user what the parameters are (depending on the layer mode). This lacked breaks for the cases, causing the other print statements to be executed. This patch fixes it.

What kind of change does this PR introduce?

Please check if this PR fulfills the following:

Does this PR introduce breaking changes? What changes might users need to make to their code?

no. no

Does this PR change default behavior?

yes. only prints out the information about the cuts for defining neighbors that is relevent based on the layer mode.

github-actions[bot] commented 1 month ago

Capybara summary for PR 1533

sonarcloud[bot] commented 1 month ago

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud