<div dir="ltr">Hi folks,<div>so recently I was testing clang-tidy, and I had a problem with the check readability-else-after-return:</div><div><br></div><div>1. running git-clang-format after applying transformation doesn't format everything, because this check only removes one line (+ braces), which doesn't affect other lines, which leeds to wrong identation.</div><div>2. It only transform one else at one time so for example for code like this:</div><div><div> </div><div>     if (multilevel && isMultilevelPolicyDescendable(policy, getDepth()))</div><div>     {</div><div>         return ..<br></div><div>     }</div><div>     else if (size() > 1) // it was only able to remove this one</div><div>     {</div><div>        return ...</div><div>     }</div><div>     else </div><div>    {</div><div>         if (policy.type == ClassificationType::EXTRA_PARAMS)</div><div>             return </div><div>         else</div><div>             return </div><div>    }</div><div><br></div></div><div><br></div><div>Did you know about those problems? I was talking with Nick (cc) and he was surprised that not all formatting can be done by just applying clang-format-diff.</div><div><br></div><div>Piotr</div></div>