aboutsummaryrefslogtreecommitdiffstats
path: root/java
diff options
context:
space:
mode:
authorColin Cross <ccross@android.com>2015-04-07 17:11:30 -0700
committerColin Cross <ccross@android.com>2015-04-08 15:19:24 -0700
commit1332b0035caf416d4d9b275e6bfead48fe0b9cac (patch)
tree588b0698a2d6288e383bf314aae5282f89bbb69c /java
parent8df14ac3f097ae09e536a58b9e8f7505dd2b5550 (diff)
downloadbuild_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.go4
-rw-r--r--java/java.go12
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",