From 51ab5b6d506e58fe862623624938f72c76ac96eb Mon Sep 17 00:00:00 2001 From: LaMont Jones Date: Wed, 26 Jun 2024 14:31:05 -0700 Subject: [PATCH] Cleanup old code This code is no longer needed. Bug: 349410287 Bug: 311155208 Bug: 308625757 Test: manual, TH Change-Id: Ib3121e5aaeb2d41de5fa89262a36d69803355c50 --- android/aconfig_providers.go | 13 ------------- 1 file changed, 13 deletions(-) diff --git a/android/aconfig_providers.go b/android/aconfig_providers.go index a47e80f03..86c17148d 100644 --- a/android/aconfig_providers.go +++ b/android/aconfig_providers.go @@ -211,7 +211,6 @@ func mergeAconfigFiles(ctx ModuleContext, container string, inputs Paths, genera } func getAconfigFilePaths(m *ModuleBase, aconfigFiles map[string]Paths) (paths Paths) { - // TODO(b/311155208): The default container here should be system. container := "system" if m.SocSpecific() { @@ -223,17 +222,5 @@ func getAconfigFilePaths(m *ModuleBase, aconfigFiles map[string]Paths) (paths Pa } paths = append(paths, aconfigFiles[container]...) - if container == "system" { - // TODO(b/311155208): Once the default container is system, we can drop this. - paths = append(paths, aconfigFiles[""]...) - } - if container != "system" { - if len(aconfigFiles[container]) == 0 && len(aconfigFiles[""]) > 0 { - // TODO(b/308625757): Either we guessed the container wrong, or the flag is misdeclared. - // For now, just include the system (aka "") container if we get here. - //fmt.Printf("container_mismatch: module=%v container=%v files=%v\n", m, container, aconfigFiles) - } - paths = append(paths, aconfigFiles[""]...) - } return }