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.

Leave a Reply

Fill in your details below or click an icon to log in: Logo

You are commenting using your account. Log Out /  Change )

Google photo

You are commenting using your Google account. Log Out /  Change )

Twitter picture

You are commenting using your Twitter account. Log Out /  Change )

Facebook photo

You are commenting using your Facebook account. Log Out /  Change )

Connecting to %s

This site uses Akismet to reduce spam. Learn how your comment data is processed.