From: Simon Glass Date: Mon, 29 May 2017 21:31:30 +0000 (-0600) Subject: patman: Rename 'list' variable in MakeCcFile() X-Git-Tag: v2017.07-rc2~54^2~30 X-Git-Url: https://git.sur5r.net/?a=commitdiff_plain;h=a44f4fb72bcb954bf38385e2953f7320b9f25aa3;p=u-boot patman: Rename 'list' variable in MakeCcFile() This is not a good variable name in Python because 'list' is a type. It shows up highlighted in some editors. Rename it. Signed-off-by: Simon Glass Tested-by: Philipp Tomsich --- diff --git a/tools/patman/series.py b/tools/patman/series.py index 395b9ea99b..d3947a7c2a 100644 --- a/tools/patman/series.py +++ b/tools/patman/series.py @@ -223,20 +223,20 @@ class Series(dict): fd = open(fname, 'w') all_ccs = [] for commit in self.commits: - list = [] + cc = [] if process_tags: - list += gitutil.BuildEmailList(commit.tags, + cc += gitutil.BuildEmailList(commit.tags, raise_on_error=raise_on_error) - list += gitutil.BuildEmailList(commit.cc_list, + cc += gitutil.BuildEmailList(commit.cc_list, raise_on_error=raise_on_error) - if type(add_maintainers) == type(list): - list += add_maintainers + if type(add_maintainers) == type(cc): + cc += add_maintainers elif add_maintainers: - list += get_maintainer.GetMaintainer(commit.patch) - list = [m.encode('utf-8') if type(m) != str else m for m in list] - all_ccs += list - print(commit.patch, ', '.join(set(list)), file=fd) - self._generated_cc[commit.patch] = list + cc += get_maintainer.GetMaintainer(commit.patch) + cc = [m.encode('utf-8') if type(m) != str else m for m in cc] + all_ccs += cc + print(commit.patch, ', '.join(set(cc)), file=fd) + self._generated_cc[commit.patch] = cc if cover_fname: cover_cc = gitutil.BuildEmailList(self.get('cover_cc', ''))