[#RSRP-30774] '|| false' is redundant in logical expressions

ReSharper

'|| false' is redundant in logical expressions

Created: 31 Oct 06 11:26 Updated: 20 Jun 08 16:44

Component/s:

Code Analysis

Fix Version/s:

Mirabilie Futurum

Security Level:

Everybody (All jira users)

Original Estimate:

Unknown

Remaining Estimate:

Unknown

Time Spent:

Unknown

Build:

300

Description

« Hide

class Stuff
{
    void Foo(bool f)
    {
        if(f || false) // || false is reundant
        {
            
        }
        if(false || f) // false || is reundant
        {
            
        }
    }
}

All

Comments

Work Log

Change History

Sort Order:

[ Permlink | « Hide ]

Michael Hedgpeth - 20 Jun 08 16:44

Another one to fix along these lines is:

if (f == false)

changes to

if (!f)