Rename the "RedundantParentheses" rule to "ControlFlowParentheses"

[DanilaFe]

Per @bradcray's suggestion.

The RedundantParentheses rule (on main) refers to the rule that rejects parentheses around conditionals of control flow. Thus:

// bad
if (expr) {

}

// good
if expr {

}

However, Brad points out that the name "redundant parentheses" brings forth cases like ((expr)), where there are two sets of parentheses. Brad's thought -- with which I agree -- is that we should leave "redundant parentheses" for the hypothetical ((expr)) rule, and change the current rule for parentheses in ifs and loops to be "control flow parentheses".

Reviewed by @jabraham17 -- thanks!

Testing

• test/chplcheck

[jabraham17]

Looks good, but please add more context to the PR message before merging