Part of "Static code analysis of OPAL" (!363) · Merge requests · OPAL / src · GitLab

Code indexing in gitaly is broken and leads to code not being visible to the user. We work on the issue with highest priority.

Snippets Groups Projects

Please migrate repositories to https://gitea.psi.ch - if you need help, contact gitea@psi.ch

Merged ext-calvo_p requested to merge 62-static-code-analysis-of-opal into master 4 years ago

Cleaning, minor fixes and spacing according to coding style

Some of the fixes were found with the static code analysis of #62.

Edited 4 years ago by ext-calvo_p

Activity

ext-calvo_p added Enhancement label 4 years ago

added Enhancement label
ext-calvo_p changed the description 4 years ago

changed the description
ext-calvo_p added Cleanup label 4 years ago

added Cleanup label
ext-calvo_p assigned to @ext-calvo_p 4 years ago

assigned to @ext-calvo_p
ext-calvo_p added 1 commit 4 years ago
added 1 commit

0f909dc2 - cppcheck clean BeamStripping class

Compare with previous version
ext-calvo_p added 1 commit 4 years ago
added 1 commit

4895ccd6 - cppcheck clean AbsBeamline

Compare with previous version
ext-calvo_p added 2 commits 4 years ago
added 2 commits

bdc5ccde - cppcheck clean Classic

d08ded26 - cppcheck clean SecondaryEmissionPhysics class

Compare with previous version
ext-calvo_p unmarked as a Work In Progress 4 years ago

unmarked as a Work In Progress
ext-calvo_p changed the description 4 years ago

changed the description
ext-calvo_p changed title from Resolve "Static code analysis of OPAL" to Part of "Static code analysis of OPAL" 4 years ago

changed title from Resolve "Static code analysis of OPAL" to Part of "Static code analysis of OPAL"
snuverink_j @snuverink_j started a thread on an old version of the diff 4 years ago

Resolved 4 years ago by ext-calvo_p
Last reply by snuverink_j 4 years ago

snuverink_j @snuverink_j started a thread on an old version of the diff 4 years ago

Resolved 4 years ago by ext-calvo_p

snuverink_j @snuverink_j started a thread on an old version of the diff 4 years ago

Resolved 4 years ago by ext-calvo_p

ext-calvo_p added 1 commit 4 years ago

added 1 commit

f5fe70db - indentation and delete comments

Compare with previous version

ext-calvo_p resolved all discussions 4 years ago

resolved all discussions

snuverink_j approved this merge request 4 years ago

approved this merge request

ext-calvo_p added 1 commit 4 years ago

added 1 commit

4333418b - clean and fix more classes

Compare with previous version

snuverink_j approved this merge request 4 years ago

approved this merge request

frey_m @frey_m started a thread on an old version of the diff 4 years ago

Resolved 4 years ago by ext-calvo_p

frey_m @frey_m started a thread on an old version of the diff 4 years ago

Resolved 4 years ago by ext-calvo_p

Please register or sign in to reply