mirror of
https://github.com/ClickHouse/ClickHouse.git
synced 2024-11-26 09:32:01 +00:00
89aa9ae00f
The check is currently *not* part of .clang-tidy. It complains about: (1) "switch has multiple consecutive identical branches" (2) "repeated branch in conditional chain" About (1): Lots of findings in switches were about redundant "[[fallthrough]]" in places where the compiler would not warn anyways. I have cleaned these up. About (2): In if-else_if-else chains, fixing the warning would usually mean concatenating multiple if-conditions. As this would reduce readability in most cases, I did not fix these places. Because of (2), I also refrained from adding "bugprone-branch-clone" to .clang-tidy. |
||
---|---|---|
.. | ||
clickhouse-client.cpp | ||
clickhouse-client.xml | ||
Client.cpp | ||
Client.h | ||
CMakeLists.txt |