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.