diff options
author | Colin Cross <ccross@android.com> | 2015-04-07 17:11:30 -0700 |
---|---|---|
committer | Colin Cross <ccross@android.com> | 2015-04-08 15:19:24 -0700 |
commit | 1332b0035caf416d4d9b275e6bfead48fe0b9cac (patch) | |
tree | 588b0698a2d6288e383bf314aae5282f89bbb69c /java | |
parent | 8df14ac3f097ae09e536a58b9e8f7505dd2b5550 (diff) | |
download | build_soong-1332b0035caf416d4d9b275e6bfead48fe0b9cac.tar.gz build_soong-1332b0035caf416d4d9b275e6bfead48fe0b9cac.tar.bz2 build_soong-1332b0035caf416d4d9b275e6bfead48fe0b9cac.zip |
Move config into common and provide helper
Using ctx.Config().(Config) everywhere is a mouthful, and it is
inefficient to do the type assertion. Put the Config interface into
the context, and provide an AConfig() to return the Config already
converted to the right type.
Change-Id: I301a1fd7d2a005580aabca7866a37c5d42ad8c69
Diffstat (limited to 'java')
-rw-r--r-- | java/builder.go | 4 | ||||
-rw-r--r-- | java/java.go | 12 |
2 files changed, 4 insertions, 12 deletions
diff --git a/java/builder.go b/java/builder.go index f0cfeaea..4eb03c25 100644 --- a/java/builder.go +++ b/java/builder.go @@ -85,10 +85,10 @@ func init() { pctx.StaticVariable("javacCmd", "javac -J-Xmx1024M $commonJdkFlags") pctx.StaticVariable("jarCmd", filepath.Join(bootstrap.BinDir, "soong_jar")) pctx.VariableFunc("dxCmd", func(c interface{}) (string, error) { - return c.(Config).HostBinTool("dx") + return c.(common.Config).HostBinTool("dx") }) pctx.VariableFunc("jarjarCmd", func(c interface{}) (string, error) { - return c.(Config).HostJavaTool("jarjar.jar") + return c.(common.Config).HostJavaTool("jarjar.jar") }) } diff --git a/java/java.go b/java/java.go index f9347218..8750dbf8 100644 --- a/java/java.go +++ b/java/java.go @@ -29,14 +29,6 @@ import ( "android/soong/common" ) -type Config interface { - SrcDir() string - PrebuiltOS() string - HostBinTool(string) (string, error) - HostJavaTool(string) (string, error) - Getenv(string) string -} - // TODO: // Autogenerated files: // AIDL @@ -266,11 +258,11 @@ func (j *javaBase) GenerateJavaBuildActions(ctx common.AndroidModuleContext) { dxFlags = append(dxFlags, "--no-locals") } - if ctx.Config().(Config).Getenv("NO_OPTIMIZE_DX") != "" { + if ctx.AConfig().Getenv("NO_OPTIMIZE_DX") != "" { dxFlags = append(dxFlags, "--no-optimize") } - if ctx.Config().(Config).Getenv("GENERATE_DEX_DEBUG") != "" { + if ctx.AConfig().Getenv("GENERATE_DEX_DEBUG") != "" { dxFlags = append(dxFlags, "--debug", "--verbose", |