patman: Use defaultdict directly

Import defaultdict to avoid needing to specify its module in multiple
places. Fix an extra blank line while here.

Signed-off-by: Simon Glass <sjg@chromium.org>
This commit is contained in:
Simon Glass 2025-04-29 07:22:23 -06:00
parent 0fb0b46200
commit 39f258494f

View File

@ -8,7 +8,7 @@ Allows creation of a new branch based on the old but with the review tags
collected from patchwork. collected from patchwork.
""" """
import collections from collections import defaultdict
import concurrent.futures import concurrent.futures
from itertools import repeat from itertools import repeat
@ -55,7 +55,7 @@ def process_reviews(content, comment_data, base_rtags):
list of patchwork.Review: reviews received on the patch list of patchwork.Review: reviews received on the patch
""" """
pstrm = patchstream.PatchStream.process_text(content, True) pstrm = patchstream.PatchStream.process_text(content, True)
rtags = collections.defaultdict(set) rtags = defaultdict(set)
for response, people in pstrm.commit.rtags.items(): for response, people in pstrm.commit.rtags.items():
rtags[response].update(people) rtags[response].update(people)
@ -70,7 +70,7 @@ def process_reviews(content, comment_data, base_rtags):
rtags[response].update(people) rtags[response].update(people)
# Find the tags that are not in the commit # Find the tags that are not in the commit
new_rtags = collections.defaultdict(set) new_rtags = defaultdict(set)
for tag, people in rtags.items(): for tag, people in rtags.items():
for who in people: for who in people:
is_new = (tag not in base_rtags or is_new = (tag not in base_rtags or
@ -115,7 +115,6 @@ def compare_with_series(series, patches):
warnings.append("Cannot find patch for commit %d ('%s')" % warnings.append("Cannot find patch for commit %d ('%s')" %
(seq + 1, cmt.subject)) (seq + 1, cmt.subject))
# Check the names match # Check the names match
commit_for_patch = {} commit_for_patch = {}
all_commits = set(series.commits) all_commits = set(series.commits)