From: Simon Glass <sjg@chromium.org>
Date: Wed, 19 Jul 2023 23:48:18 +0000 (-0600)
Subject: buildman: Tidy up common code in parse_file()
X-Git-Url: http://git.dujemihanovic.xyz/?a=commitdiff_plain;h=a114c6153699d0a9f3730d630a04a3c6c648a775;p=u-boot.git

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 541c82ff99..df57f278da 100644
--- a/tools/buildman/boards.py
+++ b/tools/buildman/boards.py
@@ -345,6 +345,12 @@ class MaintainersDatabase:
             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 @@ class MaintainersDatabase:
                                 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: