prepare-ChangeLog throws false positive warnings for modified files.
authorllango.u-szeged@partner.samsung.com <llango.u-szeged@partner.samsung.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 25 Mar 2014 15:53:39 +0000 (15:53 +0000)
committerllango.u-szeged@partner.samsung.com <llango.u-szeged@partner.samsung.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 25 Mar 2014 15:53:39 +0000 (15:53 +0000)
https://bugs.webkit.org/show_bug.cgi?id=130687

Reviewed by Darin Adler.

* Scripts/prepare-ChangeLog:
(originalFile): Ignore git commit paremeter.
We don't need this to get the unmodified file.
(extractLineRangeBeforeChange):
Simplify the if statement. Follow-up after r166189.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@166235 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Tools/ChangeLog
Tools/Scripts/prepare-ChangeLog

index 57b33fd..ee09ca2 100644 (file)
@@ -1,3 +1,16 @@
+2014-03-25  László Langó  <llango.u-szeged@partner.samsung.com>
+
+        prepare-ChangeLog throws false positive warnings for modified files.
+        https://bugs.webkit.org/show_bug.cgi?id=130687
+
+        Reviewed by Darin Adler.
+
+        * Scripts/prepare-ChangeLog:
+        (originalFile): Ignore git commit paremeter.
+        We don't need this to get the unmodified file.
+        (extractLineRangeBeforeChange):
+        Simplify the if statement. Follow-up after r166189.
+
 2014-03-25  Gabor Rapcsanyi  <rgabor@webkit.org>
 
         [EFL] Add ARM64 build support
index 182f79f..c581b7e 100755 (executable)
@@ -241,8 +241,9 @@ sub originalFile($$$$)
         my $escapedPathsString = escapeSubversionPath($file);
         $command = SVN . " cat $escapedPathsString";
     } elsif (isGit()) {
-        $command = GIT . " show " . diffFromToString($gitCommit, $gitIndex, $mergeBase);
-        $command .= ":$file" unless $gitCommit or $mergeBase;
+        $command = GIT . " show ";
+        $command .= "$mergeBase" if $mergeBase;
+        $command .= ":$file";
     }
 
     return $command;
@@ -1972,9 +1973,7 @@ sub extractLineRangeBeforeChange($)
 
     if (isSVN() && $string =~ /^(\d+)(,(\d+))?([acd])\d+(,\d+)?/) {
         $start = $1;
-        if($4 gt "a") {
-            $end = $3 || $1;
-        }
+        $end = $3 || $1 if $4 ne "a";
     } elsif (isGit() && $string =~ /^@@ -(\d+)(,(\d+))? \+\d+(,\d+)? @@/) {
         $start = $1;
         $end = defined($3) ? $3 + $1 - 1 : $1;