diff options
| author | 2017-11-29 00:27:14 -0800 | |
|---|---|---|
| committer | 2017-11-30 00:44:11 +0000 | |
| commit | aabf67968c4abc98d94bd8ccd6d612246b2fbbdc (patch) | |
| tree | fa6ef71b0fd9ce7746e8a9f6b7f02e353cace36b /android/variable.go | |
| parent | 178d5fefc0cea9d0f031c0bdee125b9d960f32c3 (diff) | |
Make *Context.Config return a Config instead of a interface{}
In Soong, a Config() method will always return a Config. Make
ModuleContext, SingletonContext, TopDownMutatorContext and
BottomUpMutatorContext's Config() methods explictly return
a Config to avoid having to type-assert everywhere. Overriding
the Config method requires duplicating the list of methods in
blueprint.BaseModuleContext and blueprint.BottomUpMutatorContext,
following the same pattern used by the other *Contexts.
Config() obsoletes the AConfig() method used in some places, which
will be cleaned up in the next patch.
Test: m checkbuild
Change-Id: Ibe21efde933959811d52443496967ab8ce71215e
Diffstat (limited to 'android/variable.go')
| -rw-r--r-- | android/variable.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/android/variable.go b/android/variable.go index 13b5abf26..c8a672e31 100644 --- a/android/variable.go +++ b/android/variable.go @@ -258,7 +258,7 @@ func variableMutator(mctx BottomUpMutatorContext) { property := "product_variables." + proptools.PropertyNameForField(name) // Check that the variable was set for the product - val := reflect.ValueOf(mctx.Config().(Config).ProductVariables).FieldByName(name) + val := reflect.ValueOf(mctx.Config().ProductVariables).FieldByName(name) if !val.IsValid() || val.Kind() != reflect.Ptr || val.IsNil() { continue } |