REGRESSION(r159599): webkitdirs.pm spits out warnings at lines 851 and 852
authorrego@igalia.com <rego@igalia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 21 Nov 2013 12:26:06 +0000 (12:26 +0000)
committerrego@igalia.com <rego@igalia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 21 Nov 2013 12:26:06 +0000 (12:26 +0000)
https://bugs.webkit.org/show_bug.cgi?id=124697

Reviewed by Ryosuke Niwa.

* Scripts/webkitdirs.pm:
(checkForArgumentAndRemoveFromARGVGettingValue): Fix check of array size
before trying to access to the first element.

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

Tools/ChangeLog
Tools/Scripts/webkitdirs.pm

index 3283731..dfbc5e2 100644 (file)
@@ -1,3 +1,14 @@
+2013-11-21  Manuel Rego Casasnovas  <rego@igalia.com>
+
+        REGRESSION(r159599): webkitdirs.pm spits out warnings at lines 851 and 852
+        https://bugs.webkit.org/show_bug.cgi?id=124697
+
+        Reviewed by Ryosuke Niwa.
+
+        * Scripts/webkitdirs.pm:
+        (checkForArgumentAndRemoveFromARGVGettingValue): Fix check of array size
+        before trying to access to the first element.
+
 2013-11-21  Carlos Garcia Campos  <cgarcia@igalia.com>
 
         [GTK] Mark all deprecated symbols in GObject DOM bindings
index 1f682d5..fd6e214 100755 (executable)
@@ -847,7 +847,7 @@ sub checkForArgumentAndRemoveFromARGVGettingValue($$)
 {
     my ($argToCheck, $valueRef) = @_;
     my @matchingIndices = findMatchingArguments($argToCheck, \@ARGV);
-    return 0 unless ($#matchingIndices != 1);
+    return 0 if ($#matchingIndices != 1);
     splice(@ARGV, $matchingIndices[0], 1);
     return $$valueRef = splice(@ARGV, $matchingIndices[0], 1);
 }