diff options
author | Patrick Georgi <pgeorgi@google.com> | 2018-11-12 17:09:40 +0100 |
---|---|---|
committer | Philipp Deppenwiese <zaolin.daisuki@gmail.com> | 2018-11-12 21:13:32 +0000 |
commit | 92332635bf6b36c0db771aa9bc775e338507702b (patch) | |
tree | 44faf5a1a206132077d9b74d572d873ad725eaa2 /util | |
parent | 670cd7016460e98627014e011cee34a92a8b4c5d (diff) |
util/scripts/maintainers.go: Stub support for website tag
ME Cleaner's component has an entry specifying its website, which this
parser didn't know how to handle. Avoid the resulting warning.
While at it, de-C the switch statement and make it work go-style. This
also fixes "R" statements being ignored.
Change-Id: Ifc23e28daba9d85bf690557a80134accea8bed21
Signed-off-by: Patrick Georgi <pgeorgi@google.com>
Reviewed-on: https://review.coreboot.org/29601
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
Reviewed-by: Philipp Deppenwiese <zaolin.daisuki@gmail.com>
Diffstat (limited to 'util')
-rw-r--r-- | util/scripts/maintainers.go | 39 |
1 files changed, 7 insertions, 32 deletions
diff --git a/util/scripts/maintainers.go b/util/scripts/maintainers.go index 03c7709c73..a45758147d 100644 --- a/util/scripts/maintainers.go +++ b/util/scripts/maintainers.go @@ -95,40 +95,15 @@ func build_maintainers(maintainers []string) { current.name = line } else { switch line[0] { - case 'R': - case 'M': - { - /* Add subsystem maintainer */ - current.maintainer = - append(current.maintainer, - line[3:len(line)]) - break - } - case 'S': - { - break - } - case 'L': - { - break - } - case 'T': - { - break - } + case 'R', 'M': + /* Add subsystem maintainer */ + current.maintainer = append(current.maintainer, line[3:len(line)]) case 'F': - { - // add files - current.file = - append(current.file, - line[3:len(line)]) - break - } + // add files + current.file = append(current.file, line[3:len(line)]) + case 'L', 'S', 'T', 'W': // ignore default: - { - fmt.Println("No such specifier: ", line) - break - } + fmt.Println("No such specifier: ", line) } } } |