BUG OF THE MONTH | A typo
V501 There are identical sub-expressions ‘(!this->m_Rev1.IsEmpty())’ to the left and to the right of the ‘||’ operator. gitstatuslistctrl.cpp 1560
class CGitStatusListCtrl : public CListCtrl { .... CString m_Rev1; CString m_Rev2; .... }; void CGitStatusListCtrl::OnContextMenuList(....) { .... if( (!this->m_Rev1.IsEmpty()) || (!this->m_Rev1.IsEmpty()) ) .... }
There are two members in the class: m_Rev1 and m_Rev2. It is these members that should have been most likely used in the expression. Then the code should look as follows:
if( (!this->m_Rev1.IsEmpty()) || (!this->m_Rev2.IsEmpty()) )
Please click here to see more bugs from this project.