patman: Convert camel case in commit.py

Convert this file to snake case and update all files which use it.

Signed-off-by: Simon Glass <sjg@chromium.org>
This commit is contained in:
Simon Glass 2022-01-29 14:14:07 -07:00
parent ae5e926550
commit a3eeadfeb9
3 changed files with 10 additions and 10 deletions

View file

@ -44,7 +44,7 @@ class Commit:
def __str__(self): def __str__(self):
return self.subject return self.subject
def AddChange(self, version, info): def add_change(self, version, info):
"""Add a new change line to the change list for a version. """Add a new change line to the change list for a version.
Args: Args:
@ -55,7 +55,7 @@ class Commit:
self.changes[version] = [] self.changes[version] = []
self.changes[version].append(info) self.changes[version].append(info)
def CheckTags(self): def check_tags(self):
"""Create a list of subject tags in the commit """Create a list of subject tags in the commit
Subject tags look like this: Subject tags look like this:
@ -78,7 +78,7 @@ class Commit:
str = m.group(2) str = m.group(2)
return None return None
def AddCc(self, cc_list): def add_cc(self, cc_list):
"""Add a list of people to Cc when we send this patch. """Add a list of people to Cc when we send this patch.
Args: Args:
@ -86,7 +86,7 @@ class Commit:
""" """
self.cc_list += cc_list self.cc_list += cc_list
def CheckDuplicateSignoff(self, signoff): def check_duplicate_signoff(self, signoff):
"""Check a list of signoffs we have send for this patch """Check a list of signoffs we have send for this patch
Args: Args:
@ -99,7 +99,7 @@ class Commit:
self.signoff_set.add(signoff) self.signoff_set.add(signoff)
return True return True
def AddRtag(self, rtag_type, who): def add_rtag(self, rtag_type, who):
"""Add a response tag to a commit """Add a response tag to a commit
Args: Args:

View file

@ -180,7 +180,7 @@ class PatchStream:
who (str): Person who gave that rtag, e.g. who (str): Person who gave that rtag, e.g.
'Fred Bloggs <fred@bloggs.org>' 'Fred Bloggs <fred@bloggs.org>'
""" """
self.commit.AddRtag(rtag_type, who) self.commit.add_rtag(rtag_type, who)
def _close_commit(self): def _close_commit(self):
"""Save the current commit into our commit list, and reset our state""" """Save the current commit into our commit list, and reset our state"""
@ -230,7 +230,7 @@ class PatchStream:
elif self.in_change == 'Cover': elif self.in_change == 'Cover':
self.series.AddChange(self.change_version, None, change) self.series.AddChange(self.change_version, None, change)
elif self.in_change == 'Commit': elif self.in_change == 'Commit':
self.commit.AddChange(self.change_version, change) self.commit.add_change(self.change_version, change)
self.change_lines = [] self.change_lines = []
def _finalise_snippet(self): def _finalise_snippet(self):
@ -494,14 +494,14 @@ class PatchStream:
who.find(os.getenv('USER') + '@') != -1): who.find(os.getenv('USER') + '@') != -1):
self._add_warn("Ignoring '%s'" % line) self._add_warn("Ignoring '%s'" % line)
elif rtag_type == 'Patch-cc': elif rtag_type == 'Patch-cc':
self.commit.AddCc(who.split(',')) self.commit.add_cc(who.split(','))
else: else:
out = [line] out = [line]
# Suppress duplicate signoffs # Suppress duplicate signoffs
elif signoff_match: elif signoff_match:
if (self.is_log or not self.commit or if (self.is_log or not self.commit or
self.commit.CheckDuplicateSignoff(signoff_match.group(1))): self.commit.check_duplicate_signoff(signoff_match.group(1))):
out = [line] out = [line]
# Well that means this is an ordinary line # Well that means this is an ordinary line

View file

@ -94,7 +94,7 @@ class Series(dict):
Args: Args:
commit: Commit object to add commit: Commit object to add
""" """
commit.CheckTags() commit.check_tags()
self.commits.append(commit) self.commits.append(commit)
def ShowActions(self, args, cmd, process_tags): def ShowActions(self, args, cmd, process_tags):