Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- int AFC::Initialize()
- {
- int ret = DEVICE_OK;
- ....
- if (ret != DEVICE_OK)
- return ret;
- AddAllowedValue("DichroicMirrorIn", "0", 0);
- AddAllowedValue("DichroicMirrorIn", "1", 1);
- if (ret != DEVICE_OK)
- return ret;
- ....
- }
- Not terrible, but second if() does not make sense.
- This suspicious code was found in Micro-Manager project by PVS-Studio static code analyzer.
- Warning message is:
- V649 There are two 'if' statements with identical conditional expressions. The first 'if' statement contains function return. This means that the second 'if' statement is senseless. Check lines: 2918, 2922. LeicaDMI.cpp 2922
- 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