Avoid triggering rebuilds for minor changes of CSSProperties.json
authorsimon.fraser@apple.com <simon.fraser@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 8 Jan 2017 06:02:25 +0000 (06:02 +0000)
committersimon.fraser@apple.com <simon.fraser@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 8 Jan 2017 06:02:25 +0000 (06:02 +0000)
https://bugs.webkit.org/show_bug.cgi?id=166810

Post-review cleanup suggested by Darin Adler.

* css/makeprop.pl:

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

Source/WebCore/ChangeLog
Source/WebCore/css/makeprop.pl

index ad9e137..4cc0287 100644 (file)
@@ -3,6 +3,15 @@
         Avoid triggering rebuilds for minor changes of CSSProperties.json
         https://bugs.webkit.org/show_bug.cgi?id=166810
 
+        Post-review cleanup suggested by Darin Adler.
+
+        * css/makeprop.pl:
+
+2017-01-07  Simon Fraser  <simon.fraser@apple.com>
+
+        Avoid triggering rebuilds for minor changes of CSSProperties.json
+        https://bugs.webkit.org/show_bug.cgi?id=166810
+
         Reviewed by Daniel Bates.
 
         We expect to make lots of metadata-related changes to CSSProperties.json, and
index c558f11..85d9377 100755 (executable)
@@ -180,7 +180,7 @@ sub sortByDescendingPriorityAndName
 sub replaceFileIfChanged($$)
 {
     my ($tempFile, $file) = @_;
-    
+
     if (compare($tempFile, $file) != 0) {
         copy($tempFile, $file) or die "Failed to copy $tempFile to $file: $!";
     }
@@ -189,7 +189,7 @@ sub replaceFileIfChanged($$)
 
 my $gperfTempFile = "CSSPropertyNames.gperf.tmp";
 
-open GPERF, ">$gperfTempFile" || die "Could not open $gperfTempFile for writing";
+open GPERF, ">", $gperfTempFile or die "Could not open $gperfTempFile for writing";
 print GPERF << "EOF";
 %{
 /* This file is automatically generated from $inputFile by makeprop, do not edit */
@@ -348,9 +348,9 @@ close GPERF;
 
 replaceFileIfChanged($gperfTempFile, "CSSPropertyNames.gperf");
 
-my $properyNamesHeaderTempFile = "CSSPropertyNames.h.tmp";
+my $propertyNamesHeaderTempFile = "CSSPropertyNames.h.tmp";
 
-open HEADER, ">$properyNamesHeaderTempFile" || die "Could not open $properyNamesHeaderTempFile for writing";
+open HEADER, ">", $propertyNamesHeaderTempFile or die "Could not open $propertyNamesHeaderTempFile for writing";
 print HEADER << "EOF";
 /* This file is automatically generated from $inputFile by makeprop, do not edit */
 
@@ -423,7 +423,7 @@ EOF
 
 close HEADER;
 
-replaceFileIfChanged($properyNamesHeaderTempFile, "CSSPropertyNames.h");
+replaceFileIfChanged($propertyNamesHeaderTempFile, "CSSPropertyNames.h");
 
 #
 # StyleBuilder.cpp generator.
@@ -916,7 +916,7 @@ sub generateValueSetter {
 
 my $styleBuilderTempFile = "StyleBuilder.cpp.tmp";
 
-open STYLEBUILDER, ">$styleBuilderTempFile" || die "Could not open $styleBuilderTempFile for writing";
+open STYLEBUILDER, ">", $styleBuilderTempFile or die "Could not open $styleBuilderTempFile for writing";
 print STYLEBUILDER << "EOF";
 /* This file is automatically generated from $inputFile by makeprop, do not edit */
 
@@ -996,7 +996,7 @@ replaceFileIfChanged($styleBuilderTempFile, "StyleBuilder.cpp");
 
 my $stylePropertyShorthandFunctionsHeaderTempFile = "StylePropertyShorthandFunctions.h.tmp";
 
-open SHORTHANDS_H, ">$stylePropertyShorthandFunctionsHeaderTempFile" || die "Could not open $stylePropertyShorthandFunctionsHeaderTempFile for writing";
+open SHORTHANDS_H, ">", $stylePropertyShorthandFunctionsHeaderTempFile or die "Could not open $stylePropertyShorthandFunctionsHeaderTempFile for writing";
 print SHORTHANDS_H << "EOF";
 /* This file is automatically generated from $inputFile by makeprop, do not edit */
 
@@ -1028,7 +1028,7 @@ replaceFileIfChanged($stylePropertyShorthandFunctionsHeaderTempFile, "StylePrope
 
 my $stylePropertyShorthandFunctionsTempFile = "StylePropertyShorthandFunctions.cpp.tmp";
 
-open SHORTHANDS_CPP, ">$stylePropertyShorthandFunctionsTempFile" || die "Could not open $stylePropertyShorthandFunctionsTempFile for writing";
+open SHORTHANDS_CPP, ">", $stylePropertyShorthandFunctionsTempFile or die "Could not open $stylePropertyShorthandFunctionsTempFile for writing";
 print SHORTHANDS_CPP << "EOF";
 /* This file is automatically generated from $inputFile by makeprop, do not edit */
 
@@ -1146,4 +1146,4 @@ replaceFileIfChanged($stylePropertyShorthandFunctionsTempFile, "StylePropertySho
 if (not $gperf) {
     $gperf = $ENV{GPERF} ? $ENV{GPERF} : "gperf";
 }
-system("\"$gperf\" --key-positions=\"*\" -D -n -s 2 CSSPropertyNames.gperf --output-file=CSSPropertyNames.cpp") == 0 || die "calling gperf failed: $?";
+system("\"$gperf\" --key-positions=\"*\" -D -n -s 2 CSSPropertyNames.gperf --output-file=CSSPropertyNames.cpp") == 0 or die "calling gperf failed: $?";