Advertisement
Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- void G4VTwistSurface::GetBoundaryLimit(G4int areacode,
- G4double limit[]) const
- {
- ....
- if (areacode & sC0Min1Max) {
- limit[0] = fAxisMin[0];
- limit[1] = fAxisMin[1];
- } else if (areacode & sC0Max1Min) {
- limit[0] = fAxisMax[0];
- limit[1] = fAxisMin[1];
- } else if (areacode & sC0Max1Max) {
- limit[0] = fAxisMax[0];
- limit[1] = fAxisMax[1];
- } else if (areacode & sC0Min1Max) {
- limit[0] = fAxisMin[0];
- limit[1] = fAxisMax[1];
- }
- ....
- }
- Most likely this is what should be written here: if (areacode & sC0Min1Min) { limit[0] = fAxisMin[0]; limit[1] = fAxisMin[1]; }
- This suspicious code was found in Geant4 project by PVS-Studio static code analyzer.
- Warning message is:
- V517 The use of 'if (A) {...} else if (A) {...}' pattern was detected. There is a probability of logical error presence. Check lines: 793, 802. G4specsolids g4vtwistsurface.cc 793
- PVS-Studio is a static analyzer for detecting bugs in the source code of applications written in C, C++, C++11, C++/CX. Site: http://www.viva64.com/en/pvs-studio/
Advertisement
Add Comment
Please, Sign In to add comment
Advertisement