PiotrZSL marked 8 inline comments as done. PiotrZSL added inline comments.
================ Comment at: clang-tools-extra/clang-tidy/readability/OperatorsRepresentationCheck.cpp:220 + if (!isAnyOperatorEnabled(OverloadedOperators, OperatorsRepresentation) && + isAnyOperatorEnabled(OverloadedOperators, UnaryRepresentation)) + return; ---------------- carlosgalvezp wrote: > Is this condition intended to be negated too? Also, would it make sense to > move this early return to the beginning of the function? Good catch. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D144522/new/ https://reviews.llvm.org/D144522 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits