Bug 175697 – Add a Unicode test to RexBench that matches non-BMP characters

• Component: JavaScriptCore
• Status: RESOLVED FIXED

[Michael Saboff]

Currently all the Unicode sub tests of RexBench only process BMP characters.  We should add a test that goes beyond BMP characters.

[Radar WebKit Bug Importer]

<rdar://problem/33952865>

[Michael Saboff]

Created attachment 318441 [details]
Patch

[JF Bastien]

Comment on attachment 318441 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=318441&action=review

I didn't check the poker logic. I saw you play, looked about right.

r=me

> PerformanceTests/RexBench/unicode_poker_benchmark.js:51
> +    runUnicodePokerBenchmark = makeBenchmarkRunner(sources, "UnicodePokerBenchmark", 20);

What's the 20 here? numIterations?

> PerformanceTests/RexBench/unicode_poker_benchmark.js:55
> +    name: "Unicode Poker",

UniPoker or Po🂾er

;-)

> PerformanceTests/RexBench/UnicodePoker/benchmark.js:36
> +        Math.random = (function() {

Ha nice :)

> PerformanceTests/RexBench/UnicodePoker/benchmark.js:54
> +        this._players.push(new Player("Player 4"));

Better player names! You could immortalized 4 people in benchmark fame!

> PerformanceTests/RexBench/UnicodePoker/poker.js:71
> +        return cardRank

ASI on purpose here?

> PerformanceTests/RexBench/UnicodePoker/poker.js:128
> +        });

Arrow function here!

[Michael Saboff]

Committed r220893: <http://trac.webkit.org/changeset/220893>

[Michael Saboff]

(In reply to JF Bastien from comment #3)
> Comment on attachment 318441 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=318441&action=review
> 
> I didn't check the poker logic. I saw you play, looked about right.
> 
> r=me
> 
> > PerformanceTests/RexBench/unicode_poker_benchmark.js:51
> > +    runUnicodePokerBenchmark = makeBenchmarkRunner(sources, "UnicodePokerBenchmark", 20);
> 
> What's the 20 here? numIterations?

Yes.

> > PerformanceTests/RexBench/unicode_poker_benchmark.js:55
> > +    name: "Unicode Poker",
> 
> UniPoker or Po🂾er
> 
> ;-)

I changed it to UniPoker.

> > PerformanceTests/RexBench/UnicodePoker/benchmark.js:36
> > +        Math.random = (function() {
> 
> Ha nice :)
> 
> > PerformanceTests/RexBench/UnicodePoker/benchmark.js:54
> > +        this._players.push(new Player("Player 4"));
> 
> Better player names! You could immortalized 4 people in benchmark fame!
> 
> > PerformanceTests/RexBench/UnicodePoker/poker.js:71
> > +        return cardRank
> 
> ASI on purpose here?

I added the missing semicolon.

> > PerformanceTests/RexBench/UnicodePoker/poker.js:128
> > +        });
> 
> Arrow function here!

Done.

[hebon48]

The community aspect is also a big plus. Winspirit AU Sat has a great community of players from all over, but I’ve met quite a few fellow https://winspirit-aus.com/ who are just as passionate about the games as I am. We share tips, talk about our favorite games, and keep each other motivated.