* Scripts/check-dom-results: Special case 100% to say something nice.
authordarin <darin@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 31 Aug 2005 06:12:14 +0000 (06:12 +0000)
committerdarin <darin@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 31 Aug 2005 06:12:14 +0000 (06:12 +0000)
        * Scripts/cvs-apply: Added "--merge" which automatically rolls back the tree before
        applying the patch (need a better name).
        * Scripts/find-extra-includes: Added. Experimental tool to find unneeded includes.

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

WebKitTools/ChangeLog
WebKitTools/Scripts/check-dom-results
WebKitTools/Scripts/cvs-apply
WebKitTools/Scripts/find-extra-includes [new file with mode: 0755]

index 32b5cb45ac807c778deae8903a5bd2ae5a2db702..8965d8808e125162f70995d98cdafcdafe74a394 100644 (file)
@@ -1,3 +1,10 @@
+2005-08-30  Darin Adler  <darin@apple.com>
+
+        * Scripts/check-dom-results: Special case 100% to say something nice.
+        * Scripts/cvs-apply: Added "--merge" which automatically rolls back the tree before
+        applying the patch (need a better name).
+        * Scripts/find-extra-includes: Added. Experimental tool to find unneeded includes.
+
 2005-08-30  Darin Adler  <darin@apple.com>
 
         * Scripts/run-webkit-tests: Small formatting fix for leaks mode.
index 4c2c6a2c679f317055767b7acb2f16eec5b87d68..bba9373037915b5bc5d35aaade9b7780457bb1da 100755 (executable)
@@ -87,7 +87,11 @@ foreach my $suite (@suites) {
     my $successCount = $count - $failureCount - $disabledCount;
     my $percentage = (sprintf "%.1f", ($successCount * 100.0 / $count));
     
-    print "${name}: ${successCount} out of ${count} tests succeeded (${percentage}%)\n";
+    if ($percentage == 100) {
+        print "${name}: all ${count} tests succeeded\n";
+    } else {
+        print "${name}: ${successCount} out of ${count} tests succeeded (${percentage}%)\n";
+    }
     if ($verbose) {
        print "\n";
        if (@disabled) {
index 1b5b54d4f2c88b9ecb4dc6a5303f2716532031e8..f3839225be80f605116c39204d0ca39c4988ccaf 100755 (executable)
 
 # Differences from invoking "patch -p0":
 #
-#   Paths from Index: lines are used rather than the paths on the patch lines, which makes
+#   Paths from "Index:" lines are used rather than the paths on the patch lines, which makes
 #       patches generated by "cvs diff" work.
 #   Handles added files (does a cvs add). 
 #   Handles removed files (does a cvs rm). 
+#   Has mode where it will roll back to CVS version numbers in the patch file so CVS can do a 3-way merge.
 #
 # Missing features:
 #
-#   Use CVS version numbers in the patch file and do a 3-way merge.
 #   Handle binary files.
 #   When doing a removal, doesn't check that old file matches what's being removed.
+#   Notice a patch that's being applied at the "wrong level" and make it work anyway.
 
 use strict;
 use Cwd;
+use Getopt::Long;
+
+my $merge = 0;
+GetOptions("merge" => \$merge); 
 
 my $startDir = getcwd();
 
+my @patches;
+my %versions;
+
 my $indexPath;
 my $patch;
 while (<>) {
@@ -54,7 +62,7 @@ while (<>) {
     if (/^Index: (.+)/) {
         $indexPath = $1;
         if ($patch) {
-            patch($patch);
+            push @patches, $patch;
             $patch = "";
         }
     }
@@ -66,10 +74,25 @@ while (<>) {
             $indexPath = "";
         }
     }
+    if (/^diff.* -r(\d+(\.\d+)*) (.*)$/) {
+        $versions{$3} = $1;
+    }
     $patch .= $_;
     $patch .= "\n";
 }
-patch($patch);
+
+push @patches, $patch if $patch;
+
+if ($merge) {
+    for my $file (sort keys %versions) {
+        print "Getting version $versions{$file} of $file\n";
+        system "cvs update -r $versions{$file} $file";
+    }
+}
+
+for $patch (@patches) {
+    patch($patch);
+}
 
 sub patch
 {
diff --git a/WebKitTools/Scripts/find-extra-includes b/WebKitTools/Scripts/find-extra-includes
new file mode 100755 (executable)
index 0000000..955b3fc
--- /dev/null
@@ -0,0 +1,102 @@
+#!/usr/bin/perl -w
+
+# Copyright (C) 2005 Apple Computer, Inc.  All rights reserved.
+#
+# Redistribution and use in source and binary forms, with or without
+# modification, are permitted provided that the following conditions
+# are met:
+#
+# 1.  Redistributions of source code must retain the above copyright
+#     notice, this list of conditions and the following disclaimer. 
+# 2.  Redistributions in binary form must reproduce the above copyright
+#     notice, this list of conditions and the following disclaimer in the
+#     documentation and/or other materials provided with the distribution. 
+# 3.  Neither the name of Apple Computer, Inc. ("Apple") nor the names of
+#     its contributors may be used to endorse or promote products derived
+#     from this software without specific prior written permission. 
+#
+# THIS SOFTWARE IS PROVIDED BY APPLE AND ITS CONTRIBUTORS "AS IS" AND ANY
+# EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED
+# WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
+# DISCLAIMED. IN NO EVENT SHALL APPLE OR ITS CONTRIBUTORS BE LIABLE FOR ANY
+# DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES
+# (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES;
+# LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND
+# ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+# (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
+# THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+
+# "find-extra-includes" script for Web Kit Open Source Project
+
+use strict;
+use File::Find;
+
+find(\&wanted, @ARGV ? @ARGV : ".");
+
+my %paths;
+my %includes;
+
+sub wanted
+{
+    my $file = $_;
+
+    if ($file eq "icu") {
+        $File::Find::prune = 1;
+        return;
+    }
+
+    if ($file !~ /^\./ && $file =~ /.(h|cpp|c|mm|m)$/) {
+        $paths{$file} = $File::Find::name;
+        open FILE, $file or die;
+        while (<FILE>) {
+            if (m-^\s*#\s*(include|import)\s+["<]((\S+/)*)(\S+)[">]-) {
+                my $include = ($2 eq "sys/" ? $2 : "") . $4;
+                $includes{$file}{$include}++;
+            }
+        }
+        close FILE;
+    }
+}
+
+my %totalIncludes;
+
+sub fillOut
+{
+    my ($file) = @_;
+
+    return if defined $totalIncludes{$file};
+
+    for my $include (keys %{ $includes{$file} }) {
+        $totalIncludes{$file}{$include} = 1;
+        fillOut($include);
+        for my $i (keys %{ $totalIncludes{$include} }) {
+            $totalIncludes{$file}{$i} = 1;
+        }
+    }
+}
+
+sub check
+{
+    my ($file) = @_;
+
+    for my $include (keys %{ $includes{$file} }) {
+        fillOut($include);
+    }
+    for my $i1 (sort keys %{ $includes{$file} }) {
+        for my $i2 (keys %{ $includes{$file} }) {
+            next if $i1 eq $i2;
+            if ($totalIncludes{$i2}{$i1}) {
+                my $b1 = $i1;
+                my $b2 = $file;
+                $b1 =~ s/\..+$//;
+                $b2 =~ s/\..+$//;
+                print "$paths{$file} does not need to include $i1, because $i2 does\n" if $b1 ne $b2;
+                last;
+            }
+        }
+    }
+}
+
+for my $file (sort keys %includes) {
+    check($file);
+}