Did a search for statements similar to expression ? TRUE : FALSE, and just replaced it with (bool) expression for simplicity.

CommentFileSizeAuthor
#1 ternary-booleans-319666-1-D7.patch5.48 KBdave reid

Comments

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Title: Remove unnecessary true or false ternary conditionals » Remove unnecessary boolean ternary conditionals
Status: Active » Needs review
StatusFileSize
new ternary-booleans-319666-1-D7.patch5.48 KB

Patch replaces any condition ? TRUE : FALSE or condition ? FALSE : TRUE ternary conditionals. I did a search for condition ? 0 : 1 and similar, but I'm not sure if I should replace those. Having to cast a boolean result to an int seems more complex than using the ternary in those cases.

Anonymous’s picture

Anonymous (not verified) commented
Status: Needs review » Needs work

This looks good. It contains some space clean-up which is under a different issue.

dries’s picture

dries commented
Status: Needs work » Fixed

Committed to CVS HEAD. Thanks.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.