Move "modernize-use-using" to art_clang_tidy_disabled
Now that b/236243696 has been resolved, there are only
two files left with "modernize-use-using" clang-tidy
issues:
* openjdkjvmti/include/jvmti.h
* ../libcore/ojluni/src/main/native/jvm.h
Both files are external and coming from Oracle, so I
think it's better to not modify them and disable the
clang-tidy issue instead (at the risk of having new
code use the typedef statement instead of using, but
hopefully the chance of this happening should be
quite low).
Bug: 213953102
Test: m tidy-art
Change-Id: I6732bec8d67a4c25942e7322cfd4df70e6ca6fe4
diff --git a/build/Android.bp b/build/Android.bp
index 5f9c1fa..ccc30ef 100644
--- a/build/Android.bp
+++ b/build/Android.bp
@@ -50,7 +50,6 @@
// Many files have these warnings. Move them to art_clang_tidy_errors
// when all files are free of these warnings.
"android-cloexec-dup",
- "modernize-use-using", // TODO: move to art_clang_tidy_errors after b/236243696 is done
]
art_clang_tidy_disabled = [
@@ -67,6 +66,9 @@
"-modernize-return-braced-init-list",
"-modernize-use-default-member-init",
"-modernize-pass-by-value",
+ // The only two remaining offenders are art/openjdkjvmti/include/jvmti.h and
+ // libcore/ojluni/src/main/native/jvm.h, which are both external files by Oracle
+ "-modernize-use-using",
]
soong_config_module_type_import {
