[Concept,06/29] buildman: Rename variables that shadow builtins

Message ID 20260106220403.2915630-7-sjg@u-boot.org
State New
Headers
Series buildman: Clean up builder.py |

Commit Message

Simon Glass Jan. 6, 2026, 10:03 p.m. UTC
  From: Simon Glass <simon.glass@canonical.com>

Rename 'type' to 'sym_type' in _get_func_sizes_for_elf() and 'str' to
'text' in add_target_list() to avoid shadowing Python built-in names.

Co-developed-by: Claude <noreply@anthropic.com>
Signed-off-by: Simon Glass <simon.glass@canonical.com>
---

 tools/buildman/builder.py | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)
  

Patch

diff --git a/tools/buildman/builder.py b/tools/buildman/builder.py
index 5a31f9aba19..1c6a38b232e 100644
--- a/tools/buildman/builder.py
+++ b/tools/buildman/builder.py
@@ -736,8 +736,8 @@  class Builder:
             line = line.strip()
             parts = line.split()
             if line and len(parts) == 3:
-                size, type, name = line.split()
-                if type in NM_SYMBOL_TYPES:
+                size, sym_type, name = line.split()
+                if sym_type in NM_SYMBOL_TYPES:
                     # function names begin with '.' on 64-bit powerpc
                     if '.' in name[1:]:
                         name = 'static.' + name.split('.')[0]
@@ -1003,14 +1003,14 @@  class Builder:
                 arch = board_dict[target].arch
             else:
                 arch = 'unknown'
-            str = self.col.build(color, ' ' + target)
+            text = self.col.build(color, ' ' + target)
             if not arch in done_arch:
-                str = ' %s  %s' % (self.col.build(color, char), str)
+                text = ' %s  %s' % (self.col.build(color, char), text)
                 done_arch[arch] = True
             if not arch in arch_list:
-                arch_list[arch] = str
+                arch_list[arch] = text
             else:
-                arch_list[arch] += str
+                arch_list[arch] += text
 
 
     def colour_num(self, num):