buildman: Tidy up common code in parse_file()
Use a function to add to the maintainers database, to avoid duplicating
the same code twice.
Signed-off-by: Simon Glass <sjg@chromium.org>
diff --git a/tools/buildman/boards.py b/tools/buildman/boards.py
index 541c82f..df57f27 100644
--- a/tools/buildman/boards.py
+++ b/tools/buildman/boards.py
@@ -345,6 +345,12 @@
srcdir (str): Directory containing source code (Kconfig files)
fname (str): MAINTAINERS file to be parsed
"""
+ def add_targets():
+ """Add any new targets"""
+ if targets:
+ for target in targets:
+ self.database[target] = (status, maintainers)
+
targets = []
maintainers = []
status = '-'
@@ -382,14 +388,11 @@
if match and not rear:
targets.append(front)
elif line == '\n':
- for target in targets:
- self.database[target] = (status, maintainers)
+ add_targets()
targets = []
maintainers = []
status = '-'
- if targets:
- for target in targets:
- self.database[target] = (status, maintainers)
+ add_targets()
class Boards: