[webkit-reviews] review granted: [Bug 57008] prepare-ChangeLog doesn't find names of modified classes/methods in Python source files : [Attachment 116227] Teach prepare-ChangeLog how to find changed classes/methods/functions in Python files
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Tue Nov 22 08:36:09 PST 2011
Daniel Bates <dbates at webkit.org> has granted Adam Roben (:aroben)
<aroben at apple.com>'s request for review:
Bug 57008: prepare-ChangeLog doesn't find names of modified classes/methods in
Python source files
https://bugs.webkit.org/show_bug.cgi?id=57008
Attachment 116227: Teach prepare-ChangeLog how to find changed
classes/methods/functions in Python files
https://bugs.webkit.org/attachment.cgi?id=116227&action=review
------- Additional Comments from Daniel Bates <dbates at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=116227&action=review
This looks good to me. I have some minor nits.
> Tools/ChangeLog:12
> + or that use python as their interpreter. Modified the code that
extracts the interpreter to
Nit: python => Python
> Tools/ChangeLog:13
> + ignore an leading "/usr/bin/env " in the interpreter line, as is
common for our Python
Nit: an => a
> Tools/Scripts/prepare-ChangeLog:1274
> + for (; $i < scalar(@scopeStack); ++$i) {
Nit: scalar(@scopeStack) => @scopeStack
It's unnecessary to call scalar() in this context.
> Tools/Scripts/prepare-ChangeLog:1282
> + for ($i = 0; $i < scalar(@poppedScopes); ++$i) {
Nit: scalar(@poppedScopes) => @poppedScopes
> Tools/Scripts/prepare-ChangeLog:1283
> + my $lineAfterEnd = $i + 1 == scalar(@poppedScopes) ? $. :
$poppedScopes[$i + 1]->{line};
Ditto.
> Tools/Scripts/prepare-ChangeLog:1286
> + scalar(@scopeStack) or warn "Popped off last scope at
$fileName:$.\n";
Ditto.
More information about the webkit-reviews
mailing list