Enable readability checks
clang-tidy readability checks are overall a good thing, and help us to
write consistent and readable code, even if it doesn't change the
result.
All changes done by the robot.
Tested: Code compiles, inspection only (changes made by robot)
Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: Iee4a0c74a11eef9f158f0044eae675ebc518b549
diff --git a/.clang-tidy b/.clang-tidy
index dfca83a..d892bdd 100644
--- a/.clang-tidy
+++ b/.clang-tidy
@@ -276,10 +276,18 @@
readability-delete-null-pointer,
readability-deleted-default,
readability-else-after-return,
+readability-function-size,
+readability-identifier-naming,
readability-implicit-bool-conversion,
+readability-inconsistent-declaration-parameter-name,
+readability-isolate-declaration,
readability-make-member-function-const,
+readability-misleading-indentation,
+readability-misplaced-array-index,
readability-named-parameter,
readability-non-const-parameter,
+readability-qualified-auto,
+readability-redundant-access-specifiers,
readability-redundant-control-flow,
readability-redundant-declaration,
readability-redundant-function-ptr-dereference,
@@ -288,8 +296,13 @@
readability-redundant-smartptr-get,
readability-redundant-string-cstr,
readability-redundant-string-init,
+readability-simplify-boolean-expr,
+readability-simplify-subscript-expr,
readability-static-accessed-through-instance,
-readability-identifier-naming,
+readability-static-definition-in-anonymous-namespace,
+readability-string-compare,
+readability-suspicious-call-argument,
+readability-uniqueptr-delete-release,
readability-uppercase-literal-suffix'
WarningsAsErrors: '*'
@@ -301,4 +314,3 @@
- { key: readability-identifier-naming.ParameterCase, value: camelBack }
- { key: readability-identifier-naming.NamespaceCase, value: lower_case }
- { key: readability-identifier-naming.StructCase, value: CamelCase }
-