https://issues.apache.org/ooo/show_bug.cgi?id=124066

            Bug ID: 124066
        Issue Type: DEFECT
           Summary: get rid of an extraneous parentheses in "if((A==B))"
           Product: General
           Version: 4.1.0-dev
          Hardware: All
                OS: All
            Status: CONFIRMED
          Severity: normal
          Priority: P4
         Component: code
          Assignee: issues@openoffice.apache.org
          Reporter: h...@apache.org
                CC: issues@openoffice.apache.org

When there is an assignment within an "if" condition then there is a dubious
convention to add an extra pair of parentheses to silence an eventual compiler
warning about a suspicious assignment vs. equality ambiguity.

If there is an equality check then that extra pair of parentheses is much more
dubious and Clang is right to warn about it. The extra parentheses should be
removed.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.
You are watching all bug changes.

Reply via email to