[webkit-reviews] review granted: [Bug 74992] In prepare-ChangeLog, replace constant global variables with "use constant" : [Attachment 120141] Patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Wed Dec 21 00:09:05 PST 2011
Ryosuke Niwa <rniwa at webkit.org> has granted Kentaro Hara
<haraken at chromium.org>'s request for review:
Bug 74992: In prepare-ChangeLog, replace constant global variables with "use
constant"
https://bugs.webkit.org/show_bug.cgi?id=74992
Attachment 120141: Patch
https://bugs.webkit.org/attachment.cgi?id=120141&action=review
------- Additional Comments from Ryosuke Niwa <rniwa at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=120141&action=review
Looks sane to me.
> Tools/Scripts/prepare-ChangeLog:320
> + my $command = GIT . " log --max-count=1 --pretty=\"format:%an\"
\"$gitCommit\"";
Why don't we use single quotations here?
More information about the webkit-reviews
mailing list