StrixOS/build_soong
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Set NoWarningsAsErrorsInTidyFlags" am: 4fd41ddb4a

Browse Source 
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2122977

Change-Id: I9c1647d979a2078cba21dbb38af51fe257e537c7
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
Chih-hung Hsieh
2022-06-14 16:57:06 +00:00
committed by Automerger Merge Worker
parent 38df4aeeb8 4fd41ddb4a
commit 3850c88cbc
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
cc/config/tidy.go
View File

@@ -20,7 +20,7 @@ import (
) )
var ( var (
// Some clang-tidy checks have bugs or not work for Android. // Some clang-tidy checks have bugs or don't work for Android.
// They are disabled here, overriding any locally selected checks. // They are disabled here, overriding any locally selected checks.
globalNoCheckList = []string{ globalNoCheckList = []string{
// https://b.corp.google.com/issues/153464409 // https://b.corp.google.com/issues/153464409

2
cc/tidy.go
View File

@@ -66,7 +66,7 @@ func (tidy *tidyFeature) props() []interface{} {
// Set this const to true when all -warnings-as-errors in tidy_flags // Set this const to true when all -warnings-as-errors in tidy_flags
// are replaced with tidy_checks_as_errors. // are replaced with tidy_checks_as_errors.
// Then, that old style usage will be obsolete and an error. // Then, that old style usage will be obsolete and an error.
const NoWarningsAsErrorsInTidyFlags = false const NoWarningsAsErrorsInTidyFlags = true
func (tidy *tidyFeature) flags(ctx ModuleContext, flags Flags) Flags { func (tidy *tidyFeature) flags(ctx ModuleContext, flags Flags) Flags {
CheckBadTidyFlags(ctx, "tidy_flags", tidy.Properties.Tidy_flags) CheckBadTidyFlags(ctx, "tidy_flags", tidy.Properties.Tidy_flags)