Bug 21831 – Fix create_hash_table for Perl 5.10

• Component: JavaScriptCore
• Status: RESOLVED FIXED

[David Kilzer (:ddkilzer)]

Square brackets need to be escaped in a print() statement to make Perl 5.10 happy when running create_hash_table.  Patch forthcoming.

[David Kilzer (:ddkilzer)]

Created attachment 24603 [details]
Patch v1

Just fix it.

[Sam Weinig]

Comment on attachment 24603 [details]
Patch v1

> commit 9ad34c84b18ab388ac50ad70fe7aac879ad74f6d
> Author: David Kilzer <[e-mail]>
> Date:   Thu Oct 23 11:16:14 2008 -0700
> 
>             Bug 21831: Fix create_hash_table for Perl 5.10
>     
>             <https://bugs.webkit.org/show_bug.cgi?id=21831>
>     
>             Reviewed by NOBODY (OOPS!).
>     
>             * kjs/create_hash_table: Escaped square brackets so that Perl 5.10
>             doesn't try to use @nameEntries.
> 
> diff --git a/JavaScriptCore/ChangeLog b/JavaScriptCore/ChangeLog
> index 3fbfd0c..50742fd 100644
> --- a/JavaScriptCore/ChangeLog
> +++ b/JavaScriptCore/ChangeLog
> @@ -1,3 +1,14 @@
> [phone]  David Kilzer  <[e-mail]>
> +
> +        Bug 21831: Fix create_hash_table for Perl 5.10
> +
> +        <https://bugs.webkit.org/show_bug.cgi?id=21831>
> +
> +        Reviewed by NOBODY (OOPS!).
> +
> +        * kjs/create_hash_table: Escaped square brackets so that Perl 5.10
> +        doesn't try to use @nameEntries.
> +
>  2008-10-22  Darin Adler  <[e-mail]>
>  
>          Reviewed by Sam Weinig.
> diff --git a/JavaScriptCore/kjs/create_hash_table b/JavaScriptCore/kjs/create_hash_table
> index d98cc30..829a024 100755
> --- a/JavaScriptCore/kjs/create_hash_table
> +++ b/JavaScriptCore/kjs/create_hash_table
> @@ -215,7 +215,7 @@ sub output() {
>          print "\nnamespace JSC {\n";
>      }
>      my $count = scalar @keys + 1;
> -    print "\nstatic const struct HashTableValue ${nameEntries}[$count] = {\n";
> +    print "\nstatic const struct HashTableValue ${nameEntries}\[$count\] = {\n";
>      my $i = 0;
>      foreach my $key (@keys) {
>          my $firstValue = "";

[David Kilzer (:ddkilzer)]

Committed r37819

http://trac.webkit.org/changeset/37819