[Webkit-unassigned] [Bug 27172] bugzilla-tool: create CommitMessage class

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=27172


Eric Seidel <eric at webkit.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #32604|review?                     |review+
               Flag|                            |




--- Comment #1 from Eric Seidel <eric at webkit.org>  2009-07-10 23:04:13 PDT ---
(From update of attachment 32604)
It's difficult for me to tell where the divide between a CommitMessage class
and ChangeLog* classes should be.

 114     return CommitMessage(''.join(changelog_messages).splitlines())
Seems a little odd to take an array of lines, but OK.

Maybe CommitMessage should take an array of ChangeLog paths instead?
(eventually probably an array of ChangeLog entries?)

Should we name this bug_id and have it lazily call parse_bug_id?
 426             bug_id = options.bug_id or commit_message.parse_bug_id()

I don't understand this code:
 72         if strip_url:
 73             line = re.sub("^(\s*)<.+> ", "\1", line)

Seems it at least needs a comment.  I assume this is some new prepare-ChangeLog
thing which I'm not used to yet?

Some of this code really seems to belong in a ChangeLogEntry class.

Seems odd that these lines don't match:
 81             match = re.search("http\://webkit\.org/b/(?P<bug_id>\d+)",
line)
 84             match = re.search(Bugzilla.bug_server_regex +
"show_bug\.cgi\?id=(?P<bug_id>\d+)", line)

I don't think this is needed:
 87         return None

I think this is OK.  But you might want to try some of the comments above.  I
think much of this will end up in a ChangeLogEntry class instead of in
CommitMessage.  CommitMessage seems useful for doing all the commit-log-editor
stuff dealing with ChangeLogEntry (or similar) classes.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.