summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPatrick Georgi <pgeorgi@google.com>2018-11-12 17:09:40 +0100
committerPhilipp Deppenwiese <zaolin.daisuki@gmail.com>2018-11-12 21:13:32 +0000
commit92332635bf6b36c0db771aa9bc775e338507702b (patch)
tree44faf5a1a206132077d9b74d572d873ad725eaa2
parent670cd7016460e98627014e011cee34a92a8b4c5d (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>
-rw-r--r--util/scripts/maintainers.go39
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)
}
}
}