This is an archive of the discontinued LLVM Phabricator instance.

Refactor: Simplify boolean expressions in clang with clang-tidy
ClosedPublic

Authored by LegalizeAdulthood on Mar 8 2015, 4:42 PM.

Diff Detail

Event Timeline

LegalizeAdulthood retitled this revision from to Refactor: Simplify boolean expressions in clang with clang-tidy.
LegalizeAdulthood updated this object.
LegalizeAdulthood edited the test plan for this revision. (Show Details)
LegalizeAdulthood added a subscriber: Unknown Object (MLST).
dblaikie accepted this revision.Mar 8 2015, 4:56 PM
dblaikie added a reviewer: dblaikie.
dblaikie added a subscriber: dblaikie.

Looks good to me, please commit (after dropping parens)

lib/Driver/ToolChain.cpp
300

Drop the extra parens

lib/Lex/PreprocessingRecord.cpp
77

Drop the extra parens

lib/Sema/SemaOverload.cpp
9500

Drop extra parens

This revision is now accepted and ready to land.Mar 8 2015, 4:56 PM
lib/Driver/ToolChain.cpp
300

Fixed

lib/Lex/PreprocessingRecord.cpp
77

Fixed

lib/Sema/SemaOverload.cpp
9500

Fixed

LegalizeAdulthood edited edge metadata.

Drop unnecessary parens.

I do not have commit access, so would someone commit this for me please? Thanks.

This revision was automatically updated to reflect the committed changes.