V3003 The use of ‘if (A) {…} else if (A) {…}’ pattern was detected. There is a probability of logical error presence. Check lines: 120, 125. CCTrayLib CCTrayProject.cs 120

public override bool Equals(object obj)
  if ((buildServer != null) && 
      (objToCompare.buildServer != null))
    // If both instances have a build server then compare the build
    // server settings
    isSame = string.Equals(buildServer.Url,
  else if ((buildServer != null) && 
           (objToCompare.buildServer != null))
    // If neither instance has a build server then they are the same
    isSame = true;

Both if blocks contain the same condition.

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.