From 8a296baab89cc44987e654b942f28a2cbba84088 Mon Sep 17 00:00:00 2001 From: Spandan Das Date: Wed, 12 Jun 2024 21:17:29 +0000 Subject: [PATCH] Remove unused dexpreopt install path function `odexOnSystemOther` (presumably) returns a boolean indicating whether .odex/.vdex files should be installed in system_other.img. The installation location logic is now colocated in build/soong/dexpreopt package, so we can delete this unused function. Test: go build ./java Change-Id: Ibd1a3a0b945f8d7953baad05cbaf65cb58b5a05a --- java/dexpreopt.go | 4 ---- 1 file changed, 4 deletions(-) diff --git a/java/dexpreopt.go b/java/dexpreopt.go index 832b85036..4fe3664fe 100644 --- a/java/dexpreopt.go +++ b/java/dexpreopt.go @@ -312,10 +312,6 @@ func dexpreoptToolDepsMutator(ctx android.BottomUpMutatorContext) { dexpreopt.RegisterToolDeps(ctx) } -func (d *dexpreopter) odexOnSystemOther(ctx android.ModuleContext, libName string, installPath android.InstallPath) bool { - return dexpreopt.OdexOnSystemOtherByName(libName, android.InstallPathToOnDevicePath(ctx, installPath), dexpreopt.GetGlobalConfig(ctx)) -} - // Returns the install path of the dex jar of a module. // // Do not rely on `ApexInfo.ApexVariationName` because it can be something like "apex1000", rather