summaryrefslogtreecommitdiff
path: root/tools
diff options
context:
space:
mode:
authorSimon Glass <sjg@chromium.org>2020-10-29 21:46:20 -0600
committerSimon Glass <sjg@chromium.org>2020-11-05 09:11:31 -0700
commitdd147eda10d24a04ed3378bfc8679ac25160f96c (patch)
tree1c2e84175e0de3ad931e9c06d6678acf949fab69 /tools
parentd93720e13800423d3d6c9f8318d5a75ce7362639 (diff)
patman: Rename variables in patchstream
Some variables are too short or shadow other variables or types. Fix these to keep pylint3 happy. Signed-off-by: Simon Glass <sjg@chromium.org>
Diffstat (limited to 'tools')
-rw-r--r--tools/patman/patchstream.py52
1 files changed, 26 insertions, 26 deletions
diff --git a/tools/patman/patchstream.py b/tools/patman/patchstream.py
index 5487799bbe..3a057fbd6c 100644
--- a/tools/patman/patchstream.py
+++ b/tools/patman/patchstream.py
@@ -149,7 +149,7 @@ class PatchStream:
"""
try:
return int(value)
- except ValueError as str:
+ except ValueError:
raise ValueError("%s: Cannot decode version info '%s'" %
(self.commit.hash, line))
@@ -385,10 +385,10 @@ class PatchStream:
# Well that means this is an ordinary line
else:
# Look for space before tab
- m = RE_SPACE_BEFORE_TAB.match(line)
- if m:
+ mat = RE_SPACE_BEFORE_TAB.match(line)
+ if mat:
self.warn.append('Line %d/%d has space before tab' %
- (self.linenum, m.start()))
+ (self.linenum, mat.start()))
# OK, we have a valid non-blank line
out = [line]
@@ -527,10 +527,10 @@ def get_metadata_for_list(commit_range, git_dir=None, count=None,
params = gitutil.LogCmd(commit_range, reverse=True, count=count,
git_dir=git_dir)
stdout = command.RunPipe([params], capture=True).stdout
- ps = PatchStream(series, is_log=True)
+ pst = PatchStream(series, is_log=True)
for line in stdout.splitlines():
- ps.process_line(line)
- ps.finalise()
+ pst.process_line(line)
+ pst.finalise()
return series
def get_metadata(branch, start, count):
@@ -554,13 +554,13 @@ def get_metadata_for_test(text):
text:
"""
series = Series()
- ps = PatchStream(series, is_log=True)
+ pst = PatchStream(series, is_log=True)
for line in text.splitlines():
- ps.process_line(line)
- ps.finalise()
+ pst.process_line(line)
+ pst.finalise()
return series
-def fix_patch(backup_dir, fname, series, commit):
+def fix_patch(backup_dir, fname, series, cmt):
"""Fix up a patch file, by adding/removing as required.
We remove our tags from the patch file, insert changes lists, etc.
@@ -571,16 +571,16 @@ def fix_patch(backup_dir, fname, series, commit):
Args:
fname: Filename to patch file to process
series: Series information about this patch set
- commit: Commit object for this patch file
+ cmt: Commit object for this patch file
Return:
A list of errors, or [] if all ok.
"""
handle, tmpname = tempfile.mkstemp()
outfd = os.fdopen(handle, 'w', encoding='utf-8')
infd = open(fname, 'r', encoding='utf-8')
- ps = PatchStream(series)
- ps.commit = commit
- ps.process_stream(infd, outfd)
+ pst = PatchStream(series)
+ pst.commit = cmt
+ pst.process_stream(infd, outfd)
infd.close()
outfd.close()
@@ -588,7 +588,7 @@ def fix_patch(backup_dir, fname, series, commit):
if backup_dir:
shutil.copy(fname, os.path.join(backup_dir, os.path.basename(fname)))
shutil.move(tmpname, fname)
- return ps.warn
+ return pst.warn
def fix_patches(series, fnames):
"""Fix up a list of patches identified by filenames
@@ -603,10 +603,10 @@ def fix_patches(series, fnames):
backup_dir = None #tempfile.mkdtemp('clean-patch')
count = 0
for fname in fnames:
- commit = series.commits[count]
- commit.patch = fname
- commit.count = count
- result = fix_patch(backup_dir, fname, series, commit)
+ cmt = series.commits[count]
+ cmt.patch = fname
+ cmt.count = count
+ result = fix_patch(backup_dir, fname, series, cmt)
if result:
print('%d warnings for %s:' % (len(result), fname))
for warn in result:
@@ -623,11 +623,11 @@ def insert_cover_letter(fname, series, count):
series: Series object
count: Number of patches in the series
"""
- fd = open(fname, 'r')
- lines = fd.readlines()
- fd.close()
+ fil = open(fname, 'r')
+ lines = fil.readlines()
+ fil.close()
- fd = open(fname, 'w')
+ fil = open(fname, 'w')
text = series.cover
prefix = series.GetPatchPrefix()
for line in lines:
@@ -647,5 +647,5 @@ def insert_cover_letter(fname, series, count):
# Now the change list
out = series.MakeChangeLog(None)
line += '\n' + '\n'.join(out)
- fd.write(line)
- fd.close()
+ fil.write(line)
+ fil.close()