BRAINSTools icon indicating copy to clipboard operation
BRAINSTools copied to clipboard

ENH: Combined the logic of identical `if/else` and `switch` statements into one.

Open kian-weimer opened this issue 5 years ago • 0 comments

There are code locations where the value of a switch or logic for an if/else is different, but the contents are the same. These have now been combined so the body is only written once. This reduces redundant code but may negatively affect readability. This implements the bugprone-branch-clone clang-tidy check. -”Repeated branch in conditional chain"

kian-weimer avatar Mar 15 '21 17:03 kian-weimer