Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- typedef struct SCMDIFFSTATE
- {
- ....
- bool fIgnoreTrailingWhite;
- bool fIgnoreLeadingWhite;
- ....
- } SCMDIFFSTATE;
- /* Pointer to a diff state. */
- typedef SCMDIFFSTATE *PSCMDIFFSTATE;
- /* Compare two lines */
- DECLINLINE(bool) scmDiffCompare(PSCMDIFFSTATE pState, ....)
- {
- ....
- if (pState->fIgnoreTrailingWhite //<==
- || pState->fIgnoreTrailingWhite) //<==
- return scmDiffCompareSlow(....);
- ....
- }
- This suspicious code was found in VirtualBox project by PVS-Studio static code analyzer.
- Warning message is:
- V501 There are identical sub-expressions 'pState->fIgnoreTrailingWhite' to the left and to the right of the '||' operator. scmdiff.cpp 238
- 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