Changeset 171264 in webkit
- Timestamp:
- Jul 19, 2014 11:07:43 AM (10 years ago)
- Location:
- trunk/Source/WebCore
- Files:
-
- 3 edited
Legend:
- Unmodified
- Added
- Removed
-
trunk/Source/WebCore/ChangeLog
r171261 r171264 1 2014-07-19 Mark Rowe <mrowe@apple.com> 2 3 <https://webkit.org/b/135085> Ensure that make_names.pl generates the same result when run multiple times. 4 5 Perl 5.18 introduced hash randomization. This results in the iteration order of hashes being different 6 from one run to the next. To ensure identical output we can iterate over the hash keys in sorted order. 7 8 Reviewed by Alexey Proskuryakov. 9 10 * bindings/scripts/StaticString.pm: 11 (GenerateStrings): 12 (GenerateStringAsserts): 13 * dom/make_names.pl: 14 1 15 2014-07-19 Zan Dobersek <zdobersek@igalia.com> 2 16 -
trunk/Source/WebCore/bindings/scripts/StaticString.pm
r166120 r171264 34 34 my @result = (); 35 35 36 while ( my ($name, $value) = each %strings) {37 push(@result, "static const LChar ${name}String8[] = \"$ {value}\";\n");36 for my $name (sort keys %strings) { 37 push(@result, "static const LChar ${name}String8[] = \"$strings{$name}\";\n"); 38 38 } 39 39 40 40 push(@result, "\n"); 41 41 42 while ( my ($name, $value) = each %strings ) { 42 for my $name (sort keys %strings) { 43 my $value = $strings{$name}; 43 44 my $length = length($value); 44 45 my $hash = Hasher::GenerateHashValue($value); … … 67 68 push(@result, "#ifndef NDEBUG\n"); 68 69 69 while ( my ($name, $value) = each %strings) {70 for my $name (sort keys %strings) { 70 71 push(@result, " reinterpret_cast<StringImpl*>(&${name}Data)->assertHashIsCorrect();\n"); 71 72 } -
trunk/Source/WebCore/dom/make_names.pl
r166128 r171264 120 120 print F StaticString::GenerateStrings(\%parameters); 121 121 122 while ( my ($name, $identifier) = each %parameters) {122 for my $name (sort keys %parameters) { 123 123 print F "DEFINE_GLOBAL(AtomicString, $name)\n"; 124 124 } … … 129 129 print F StaticString::GenerateStringAsserts(\%parameters); 130 130 131 while ( my ($name, $identifier) = each %parameters) {131 for my $name (sort keys %parameters) { 132 132 # FIXME: Would like to use static_cast here, but there are differences in const 133 133 # depending on whether SKIP_STATIC_CONSTRUCTORS_ON_GCC is used, so stick with a
Note: See TracChangeset
for help on using the changeset viewer.