Convert defaults to a top down mutator

am: 1317701114

Change-Id: I3736f583432d56a7dd1889ba948b9253158c9543
This commit is contained in:
Colin Cross 2016-08-09 23:23:01 +00:00 committed by android-build-merger
commit adbf48c3df
2 changed files with 30 additions and 26 deletions

View File

@ -26,7 +26,7 @@ import (
func init() {
RegisterBottomUpMutator("defaults_deps", defaultsDepsMutator)
RegisterBottomUpMutator("defaults", defaultsMutator)
RegisterTopDownMutator("defaults", defaultsMutator)
RegisterBottomUpMutator("arch", ArchMutator)
}

View File

@ -45,7 +45,7 @@ func (d *DefaultableModule) setProperties(props []interface{}) {
type Defaultable interface {
defaults() *defaultsProperties
setProperties([]interface{})
applyDefaults(BottomUpMutatorContext, Defaults)
applyDefaults(TopDownMutatorContext, []Defaults)
}
var _ Defaultable = (*DefaultableModule)(nil)
@ -85,18 +85,20 @@ func InitDefaultsModule(module Module, d Defaults, props ...interface{}) (bluepr
var _ Defaults = (*DefaultsModule)(nil)
func (defaultable *DefaultableModule) applyDefaults(ctx BottomUpMutatorContext,
defaults Defaults) {
func (defaultable *DefaultableModule) applyDefaults(ctx TopDownMutatorContext,
defaultsList []Defaults) {
for _, prop := range defaultable.defaultableProperties {
for _, def := range defaults.properties() {
if proptools.TypeEqual(prop, def) {
err := proptools.PrependProperties(prop, def, nil)
if err != nil {
if propertyErr, ok := err.(*proptools.ExtendPropertyError); ok {
ctx.PropertyErrorf(propertyErr.Property, "%s", propertyErr.Err.Error())
} else {
panic(err)
for _, defaults := range defaultsList {
for _, prop := range defaultable.defaultableProperties {
for _, def := range defaults.properties() {
if proptools.TypeEqual(prop, def) {
err := proptools.PrependProperties(prop, def, nil)
if err != nil {
if propertyErr, ok := err.(*proptools.ExtendPropertyError); ok {
ctx.PropertyErrorf(propertyErr.Property, "%s", propertyErr.Err.Error())
} else {
panic(err)
}
}
}
}
@ -110,19 +112,21 @@ func defaultsDepsMutator(ctx BottomUpMutatorContext) {
}
}
func defaultsMutator(ctx BottomUpMutatorContext) {
if defaultable, ok := ctx.Module().(Defaultable); ok {
for _, defaultsDep := range defaultable.defaults().Defaults {
ctx.VisitDirectDeps(func(m blueprint.Module) {
if ctx.OtherModuleName(m) == defaultsDep {
if defaultsModule, ok := m.(Defaults); ok {
defaultable.applyDefaults(ctx, defaultsModule)
} else {
ctx.PropertyErrorf("defaults", "module %s is not an defaults module",
ctx.OtherModuleName(m))
}
func defaultsMutator(ctx TopDownMutatorContext) {
if defaultable, ok := ctx.Module().(Defaultable); ok && len(defaultable.defaults().Defaults) > 0 {
var defaultsList []Defaults
ctx.WalkDeps(func(module, parent blueprint.Module) bool {
if ctx.OtherModuleDependencyTag(module) == DefaultsDepTag {
if defaults, ok := module.(Defaults); ok {
defaultsList = append(defaultsList, defaults)
return len(defaults.defaults().Defaults) > 0
} else {
ctx.PropertyErrorf("defaults", "module %s is not an defaults module",
ctx.OtherModuleName(module))
}
})
}
}
return false
})
defaultable.applyDefaults(ctx, defaultsList)
}
}